Updating job shown as possibility although unsupported #243

Author: m-mohr

src/components/JobPanel.vue

@@ -38,7 +38,21 @@ export default {
 	},
 	data() {
 		return {
-			columns: {
+			watchers: {},
+			jobUpdater: null,
+			runId: 0
+		};
+	},
+	mounted() {
+		this.listen('replaceProcess', this.replaceProcess);
+	},
+	computed: {
+		...Utils.mapState(['connection']),
+		...Utils.mapGetters(['supports', 'supportsBilling', 'supportsBillingPlans']),
+		...Utils.mapGetters('editor', ['hasProcess']),
+		...Utils.mapState('editor', ['process']),
+		columns() {
+			return {
 				id: {
 					name: 'ID',
 					primaryKey: true,
@@ -48,7 +62,7 @@ export default {
 					name: 'Batch Job',
 					computedValue: row => Utils.getResourceTitle(row),
 					format: value => Utils.formatIdOrTitle(value),
-					edit: this.updateTitle
+					edit: this.supportsUpdate ? this.updateTitle : null
 				},
 				status: {
 					name: 'Status',
@@ -68,20 +82,8 @@ export default {
 					filterable: false,
 					sort: false
 				}
-			},
-			watchers: {},
-			jobUpdater: null,
-			runId: 0
-		};
-	},
-	mounted() {
-		this.listen('replaceProcess', this.replaceProcess);
-	},
-	computed: {
-		...Utils.mapState(['connection']),
-		...Utils.mapGetters(['supports', 'supportsBilling', 'supportsBillingPlans']),
-		...Utils.mapGetters('editor', ['hasProcess']),
-		...Utils.mapState('editor', ['process']),
+			};
+		},
 		supportsStart() {
 			return this.supports('startJob');
 		},

src/components/ServicePanel.vue

@@ -33,7 +33,16 @@ export default {
 	},
 	data() {
 		return {
-			columns: {
+			createdQuickViews: {}
+		};
+	},
+	computed: {
+		...Utils.mapState('editor', ['process']),
+		...Utils.mapGetters('editor', ['hasProcess']),
+		...Utils.mapState(['serviceTypes']),
+		...Utils.mapGetters(['supports', 'supportsBilling', 'supportsBillingPlans']),
+		columns() {
+			return {
 				id: {
 					name: 'ID',
 					primaryKey: true,
@@ -43,15 +52,15 @@ export default {
 					name: 'Web Service',
 					computedValue: row => Utils.getResourceTitle(row),
 					format: value => Utils.formatIdOrTitle(value),
-					edit: this.updateTitle
+					edit: this.supportsUpdate ? this.updateTitle : null
 				},
 				type: {
 					name: 'Type',
 					format: value => typeof value === 'string' ? value.toUpperCase() : value,
 				},
 				enabled: {
 					name: 'Enabled',
-					edit: this.toggleEnabled
+					edit: this.supportsUpdate ? this.toggleEnabled : null
 				},
 				created: {
 					name: 'Submitted',
@@ -63,15 +72,8 @@ export default {
 					filterable: false,
 					sort: false
 				}
-			},
-			createdQuickViews: {}
-		};
-	},
-	computed: {
-		...Utils.mapState('editor', ['process']),
-		...Utils.mapGetters('editor', ['hasProcess']),
-		...Utils.mapState(['serviceTypes']),
-		...Utils.mapGetters(['supports', 'supportsBilling', 'supportsBillingPlans']),
+			};
+		},
 		canShare() {
 			return Array.isArray(this.$config.supportedBatchJobSharingServices) && this.$config.supportedBatchJobSharingServices.length > 0;
 		},

src/components/WorkPanelMixin.js

@@ -8,7 +8,6 @@ export default (namespace, singular, plural, loadInitially = true) => {
 		},
 		data() {
 			return {
-				columns: {},
 				syncTimer: null,
 				lastSyncTime: null
 			};