schema compatibility strategy fix

mfortman11 · mfortman11 · commit e2c05b336fb9 · 2022-06-13T12:04:57.000-05:00
diff --git a/dashboard/src/components/namespaces/NamespaceOverviewTab.vue b/dashboard/src/components/namespaces/NamespaceOverviewTab.vue
@@ -618,7 +618,7 @@ export default {
         const newStrat = this.namespacesConfig.data[this.$route.params.id].schema_compatibility_strategy
 
         if (!newStrat || newStrat === 'UNDEFINED') {
-          return oldStrat.toUpperCase()
+          return oldStrat ? oldStrat.toUpperCase() : 'UNDEFINED'
         }
         return newStrat
       }
diff --git a/dashboard/src/store/modules/pulsar.js b/dashboard/src/store/modules/pulsar.js
@@ -1279,7 +1279,7 @@ const actions = {
           const newStrat = state.namespacesConfig.data[nsIdx].schema_compatibility_strategy
 
           if (!newStrat || newStrat === 'UNDEFINED') {
-            compatMode = oldStrat.toUpperCase()
+            compatMode = oldStrat ? oldStrat.toUpperCase() : 'UNDEFINED'
           } else {
             compatMode = newStrat
           }

Original file line number	Diff line number	Diff line change
`@@ -618,7 +618,7 @@ export default {`
`618`	`618`	`const newStrat = this.namespacesConfig.data[this.$route.params.id].schema_compatibility_strategy`
`619`	`619`
`620`	`620`	`if (!newStrat \|\| newStrat === 'UNDEFINED') {`
`621`		`- return oldStrat.toUpperCase()`
	`621`	`+ return oldStrat ? oldStrat.toUpperCase() : 'UNDEFINED'`
`622`	`622`	`}`
`623`	`623`	`return newStrat`
`624`	`624`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1279,7 +1279,7 @@ const actions = {`
`1279`	`1279`	`const newStrat = state.namespacesConfig.data[nsIdx].schema_compatibility_strategy`
`1280`	`1280`
`1281`	`1281`	`if (!newStrat \|\| newStrat === 'UNDEFINED') {`
`1282`		`- compatMode = oldStrat.toUpperCase()`
	`1282`	`+ compatMode = oldStrat ? oldStrat.toUpperCase() : 'UNDEFINED'`
`1283`	`1283`	`} else {`
`1284`	`1284`	`compatMode = newStrat`
`1285`	`1285`	`}`