fix double calls for opsrequest (#968)

SamiulSourav · SamiulSourav · commit 93755881070c · 2026-01-28T16:53:57.000+06:00
Signed-off-by: Samiul &lt;samiul@appscode.com&gt;
diff --git a/charts/opskubedbcom-druidopsrequest-editor/ui/functions.js b/charts/opskubedbcom-druidopsrequest-editor/ui/functions.js
@@ -335,7 +335,6 @@ export const useFunc = (model) => {
   )
 
   // Initialize on load
-  getDbDetails()
   showAndInitOpsRequestType()
 
   // ============================================================
@@ -457,9 +456,14 @@ export const useFunc = (model) => {
     } else return {}
   }
 
-   function isTlsEnabled() {
+  function isTlsEnabled() {
     const dbDetails = getValue(discriminator, '/dbDetails')
-    return (dbDetails?.spec?.sslMode && dbDetails?.spec?.sslMode !== 'disabled' && dbDetails?.spec?.sslMode !== 'disable') || dbDetails?.spec?.tls
+    return (
+      (dbDetails?.spec?.sslMode &&
+        dbDetails?.spec?.sslMode !== 'disabled' &&
+        dbDetails?.spec?.sslMode !== 'disable') ||
+      dbDetails?.spec?.tls
+    )
   }
 
   async function getDbVersions() {
@@ -851,7 +855,8 @@ export const useFunc = (model) => {
                 memory: machineData.limits.memory,
               },
             }
-          } else return { text: machine, value: { machine, cpu: limits.cpu, memory: limits.memory } }
+          } else
+            return { text: machine, value: { machine, cpu: limits.cpu, memory: limits.memory } }
         }
       })
     } else {
@@ -1746,7 +1751,7 @@ export const useFunc = (model) => {
         return getIssuer(url)
       }
       return clusterIssuers
-    }else if (!kind) {
+    } else if (!kind) {
       commit('wizard/model$delete', '/spec/tls/issuerRef/name')
       return []
     }
@@ -2171,6 +2176,6 @@ export const useFunc = (model) => {
     getNestedValue,
     hasTopologyType,
     getResourceConfig,
-    isTlsEnabled
+    isTlsEnabled,
   }
 }
diff --git a/charts/opskubedbcom-elasticsearchopsrequest-editor/ui/functions.js b/charts/opskubedbcom-elasticsearchopsrequest-editor/ui/functions.js
@@ -326,7 +326,6 @@ export const useFunc = (model) => {
   )
 
   // Initialize on load
-  getDbDetails()
   showAndInitOpsRequestType()
 
   // =====================================================
@@ -482,7 +481,12 @@ export const useFunc = (model) => {
 
   function isTlsEnabled() {
     const dbDetails = getValue(discriminator, '/dbDetails')
-    return (dbDetails?.spec?.sslMode && dbDetails?.spec?.sslMode !== 'disabled' && dbDetails?.spec?.sslMode !== 'disable') || dbDetails?.spec?.tls
+    return (
+      (dbDetails?.spec?.sslMode &&
+        dbDetails?.spec?.sslMode !== 'disabled' &&
+        dbDetails?.spec?.sslMode !== 'disable') ||
+      dbDetails?.spec?.tls
+    )
   }
 
   function initDatabaseRef() {
@@ -903,7 +907,8 @@ export const useFunc = (model) => {
                 memory: machineData.limits.memory,
               },
             }
-          } else return { text: machine, value: { machine, cpu: limits.cpu, memory: limits.memory } }
+          } else
+            return { text: machine, value: { machine, cpu: limits.cpu, memory: limits.memory } }
         }
       })
     } else {
@@ -2078,6 +2083,6 @@ export const useFunc = (model) => {
     setApplyToIfReady,
     hasResourceValue,
     hasVolumeExpansion,
-    isTlsEnabled
+    isTlsEnabled,
   }
 }
diff --git a/charts/opskubedbcom-ferretdbopsrequest-editor/ui/functions.js b/charts/opskubedbcom-ferretdbopsrequest-editor/ui/functions.js
@@ -315,7 +315,6 @@ export const useFunc = (model) => {
     store.state,
   )
 
-  getDbDetails()
   showAndInitOpsRequestType()
   async function fetchJsons({ axios, itemCtx }) {
     let ui = {}
@@ -353,7 +352,12 @@ export const useFunc = (model) => {
 
   function isTlsEnabled() {
     const dbDetails = getValue(discriminator, '/dbDetails')
-    return (dbDetails?.spec?.sslMode && dbDetails?.spec?.sslMode !== 'disabled' && dbDetails?.spec?.sslMode !== 'disable') || dbDetails?.spec?.tls
+    return (
+      (dbDetails?.spec?.sslMode &&
+        dbDetails?.spec?.sslMode !== 'disabled' &&
+        dbDetails?.spec?.sslMode !== 'disable') ||
+      dbDetails?.spec?.tls
+    )
   }
 
   function isRancherManaged() {
@@ -749,7 +753,8 @@ export const useFunc = (model) => {
                 memory: machineData.limits.memory,
               },
             }
-          } else return { text: machine, value: { machine, cpu: limits.cpu, memory: limits.memory } }
+          } else
+            return { text: machine, value: { machine, cpu: limits.cpu, memory: limits.memory } }
         }
       })
     } else {
diff --git a/charts/opskubedbcom-kafkaopsrequest-editor/ui/functions.js b/charts/opskubedbcom-kafkaopsrequest-editor/ui/functions.js
@@ -318,7 +318,6 @@ export const useFunc = (model) => {
     store.state,
   )
 
-  getDbDetails()
   showAndInitOpsRequestType()
   async function fetchJsons({ axios, itemCtx }) {
     let ui = {}
@@ -356,7 +355,12 @@ export const useFunc = (model) => {
 
   function isTlsEnabled() {
     const dbDetails = getValue(discriminator, '/dbDetails')
-    return (dbDetails?.spec?.sslMode && dbDetails?.spec?.sslMode !== 'disabled' && dbDetails?.spec?.sslMode !== 'disable') || dbDetails?.spec?.tls
+    return (
+      (dbDetails?.spec?.sslMode &&
+        dbDetails?.spec?.sslMode !== 'disabled' &&
+        dbDetails?.spec?.sslMode !== 'disable') ||
+      dbDetails?.spec?.tls
+    )
   }
 
   function isRancherManaged() {
@@ -773,7 +777,8 @@ export const useFunc = (model) => {
                 memory: machineData.limits.memory,
               },
             }
-          } else return { text: machine, value: { machine, cpu: limits.cpu, memory: limits.memory } }
+          } else
+            return { text: machine, value: { machine, cpu: limits.cpu, memory: limits.memory } }
         }
       })
     } else {
diff --git a/charts/opskubedbcom-mariadbopsrequest-editor/ui/functions.js b/charts/opskubedbcom-mariadbopsrequest-editor/ui/functions.js
@@ -317,7 +317,6 @@ export const useFunc = (model) => {
     store.state,
   )
 
-  getDbDetails()
   showAndInitOpsRequestType()
   async function fetchJsons({ axios, itemCtx }) {
     let ui = {}
diff --git a/charts/opskubedbcom-memcachedopsrequest-editor/ui/functions.js b/charts/opskubedbcom-memcachedopsrequest-editor/ui/functions.js
@@ -317,7 +317,6 @@ export const useFunc = (model) => {
     store.state,
   )
 
-  getDbDetails()
   showAndInitOpsRequestType()
 
   async function fetchJsons({ axios, itemCtx }) {
diff --git a/charts/opskubedbcom-mongodbopsrequest-editor/ui/functions.js b/charts/opskubedbcom-mongodbopsrequest-editor/ui/functions.js
@@ -328,7 +328,6 @@ export const useFunc = (model) => {
     store.state,
   )
 
-  getDbDetails()
   showAndInitOpsRequestType()
   async function fetchJsons({ axios, itemCtx }) {
     let ui = {}
diff --git a/charts/opskubedbcom-mssqlserveropsrequest-editor/ui/functions.js b/charts/opskubedbcom-mssqlserveropsrequest-editor/ui/functions.js
@@ -317,7 +317,6 @@ export const useFunc = (model) => {
     store.state,
   )
 
-  getDbDetails()
   showAndInitOpsRequestType()
 
   async function fetchJsons({ axios, itemCtx }) {
diff --git a/charts/opskubedbcom-mysqlopsrequest-editor/ui/functions.js b/charts/opskubedbcom-mysqlopsrequest-editor/ui/functions.js
@@ -317,7 +317,6 @@ export const useFunc = (model) => {
     store.state,
   )
 
-  getDbDetails()
   showAndInitOpsRequestType()
   async function fetchJsons({ axios, itemCtx }) {
     let ui = {}
diff --git a/charts/opskubedbcom-perconaxtradbopsrequest-editor/ui/functions.js b/charts/opskubedbcom-perconaxtradbopsrequest-editor/ui/functions.js
@@ -318,7 +318,6 @@ export const useFunc = (model) => {
     store.state,
   )
 
-  getDbDetails()
   showAndInitOpsRequestType()
   async function fetchJsons({ axios, itemCtx }) {
     let ui = {}
diff --git a/charts/opskubedbcom-pgbounceropsrequest-editor/ui/functions.js b/charts/opskubedbcom-pgbounceropsrequest-editor/ui/functions.js
@@ -318,7 +318,6 @@ export const useFunc = (model) => {
     store.state,
   )
 
-  getDbDetails()
   showAndInitOpsRequestType()
   async function fetchJsons({ axios, itemCtx }) {
     let ui = {}
diff --git a/charts/opskubedbcom-pgpoolopsrequest-editor/ui/functions.js b/charts/opskubedbcom-pgpoolopsrequest-editor/ui/functions.js
@@ -318,7 +318,6 @@ export const useFunc = (model) => {
     store.state,
   )
 
-  getDbDetails()
   showAndInitOpsRequestType()
 
   async function fetchJsons({ axios, itemCtx }) {
diff --git a/charts/opskubedbcom-postgresopsrequest-editor/ui/functions.js b/charts/opskubedbcom-postgresopsrequest-editor/ui/functions.js
@@ -316,7 +316,6 @@ export const useFunc = (model) => {
     store.state,
   )
 
-  getDbDetails()
   showAndInitOpsRequestType()
 
   async function fetchJsons({ axios, itemCtx }) {
diff --git a/charts/opskubedbcom-proxysqlopsrequest-editor/ui/functions.js b/charts/opskubedbcom-proxysqlopsrequest-editor/ui/functions.js
@@ -315,7 +315,6 @@ export const useFunc = (model) => {
     store.state,
   )
 
-  getDbDetails()
   showAndInitOpsRequestType()
   async function fetchJsons({ axios, itemCtx }) {
     let ui = {}
@@ -353,7 +352,12 @@ export const useFunc = (model) => {
 
   function isTlsEnabled() {
     const dbDetails = getValue(discriminator, '/dbDetails')
-    return (dbDetails?.spec?.sslMode && dbDetails?.spec?.sslMode !== 'disabled' && dbDetails?.spec?.sslMode !== 'disable') || dbDetails?.spec?.tls
+    return (
+      (dbDetails?.spec?.sslMode &&
+        dbDetails?.spec?.sslMode !== 'disabled' &&
+        dbDetails?.spec?.sslMode !== 'disable') ||
+      dbDetails?.spec?.tls
+    )
   }
 
   function isRancherManaged() {
@@ -731,7 +735,8 @@ export const useFunc = (model) => {
                 memory: machineData.limits.memory,
               },
             }
-          } else return { text: machine, value: { machine, cpu: limits.cpu, memory: limits.memory } }
+          } else
+            return { text: machine, value: { machine, cpu: limits.cpu, memory: limits.memory } }
         }
       })
     } else {
diff --git a/charts/opskubedbcom-rabbitmqopsrequest-editor/ui/functions.js b/charts/opskubedbcom-rabbitmqopsrequest-editor/ui/functions.js
@@ -317,7 +317,6 @@ export const useFunc = (model) => {
     store.state,
   )
 
-  getDbDetails()
   showAndInitOpsRequestType()
 
   async function fetchJsons({ axios, itemCtx }) {
diff --git a/charts/opskubedbcom-redisopsrequest-editor/ui/functions.js b/charts/opskubedbcom-redisopsrequest-editor/ui/functions.js
@@ -317,7 +317,6 @@ export const useFunc = (model) => {
     store.state,
   )
 
-  getDbDetails()
   showAndInitOpsRequestType()
   async function fetchJsons({ axios, itemCtx }) {
     let ui = {}
diff --git a/charts/opskubedbcom-singlestoreopsrequest-editor/ui/functions.js b/charts/opskubedbcom-singlestoreopsrequest-editor/ui/functions.js
@@ -317,7 +317,6 @@ export const useFunc = (model) => {
     store.state,
   )
 
-  getDbDetails()
   showAndInitOpsRequestType()
   async function fetchJsons({ axios, itemCtx }) {
     let ui = {}
@@ -355,7 +354,12 @@ export const useFunc = (model) => {
 
   function isTlsEnabled() {
     const dbDetails = getValue(discriminator, '/dbDetails')
-    return (dbDetails?.spec?.sslMode && dbDetails?.spec?.sslMode !== 'disabled' && dbDetails?.spec?.sslMode !== 'disable') || dbDetails?.spec?.tls
+    return (
+      (dbDetails?.spec?.sslMode &&
+        dbDetails?.spec?.sslMode !== 'disabled' &&
+        dbDetails?.spec?.sslMode !== 'disable') ||
+      dbDetails?.spec?.tls
+    )
   }
 
   function isRancherManaged() {
@@ -749,7 +753,8 @@ export const useFunc = (model) => {
                 memory: machineData.limits.memory,
               },
             }
-          } else return { text: machine, value: { machine, cpu: limits.cpu, memory: limits.memory } }
+          } else
+            return { text: machine, value: { machine, cpu: limits.cpu, memory: limits.memory } }
         }
       })
     } else {
diff --git a/charts/opskubedbcom-solropsrequest-editor/ui/functions.js b/charts/opskubedbcom-solropsrequest-editor/ui/functions.js
@@ -317,7 +317,6 @@ export const useFunc = (model) => {
     store.state,
   )
 
-  getDbDetails()
   showAndInitOpsRequestType()
 
   // for config secret
@@ -359,7 +358,12 @@ export const useFunc = (model) => {
 
   function isTlsEnabled() {
     const dbDetails = getValue(discriminator, '/dbDetails')
-    return (dbDetails?.spec?.sslMode && dbDetails?.spec?.sslMode !== 'disabled' && dbDetails?.spec?.sslMode !== 'disable') || dbDetails?.spec?.tls
+    return (
+      (dbDetails?.spec?.sslMode &&
+        dbDetails?.spec?.sslMode !== 'disabled' &&
+        dbDetails?.spec?.sslMode !== 'disable') ||
+      dbDetails?.spec?.tls
+    )
   }
 
   function isRancherManaged() {
diff --git a/charts/opskubedbcom-zookeeperopsrequest-editor/ui/functions.js b/charts/opskubedbcom-zookeeperopsrequest-editor/ui/functions.js
@@ -318,7 +318,6 @@ export const useFunc = (model) => {
     store.state,
   )
 
-  getDbDetails()
   showAndInitOpsRequestType()
 
   function returnFalse() {

Original file line number	Diff line number	Diff line change
`@@ -335,7 +335,6 @@ export const useFunc = (model) => {`
`335`	`335`	`)`
`336`	`336`
`337`	`337`	`// Initialize on load`
`338`		`- getDbDetails()`
`339`	`338`	`showAndInitOpsRequestType()`
`340`	`339`
`341`	`340`	`// ============================================================`
`@@ -457,9 +456,14 @@ export const useFunc = (model) => {`
`457`	`456`	`} else return {}`
`458`	`457`	`}`
`459`	`458`
`460`		`- function isTlsEnabled() {`
	`459`	`+ function isTlsEnabled() {`
`461`	`460`	`const dbDetails = getValue(discriminator, '/dbDetails')`
`462`		`- return (dbDetails?.spec?.sslMode && dbDetails?.spec?.sslMode !== 'disabled' && dbDetails?.spec?.sslMode !== 'disable') \|\| dbDetails?.spec?.tls`
	`461`	`+ return (`
	`462`	`+ (dbDetails?.spec?.sslMode &&`
	`463`	`+ dbDetails?.spec?.sslMode !== 'disabled' &&`
	`464`	`+ dbDetails?.spec?.sslMode !== 'disable') \|\|`
	`465`	`+ dbDetails?.spec?.tls`
	`466`	`+ )`
`463`	`467`	`}`
`464`	`468`
`465`	`469`	`async function getDbVersions() {`
`@@ -851,7 +855,8 @@ export const useFunc = (model) => {`
`851`	`855`	`memory: machineData.limits.memory,`
`852`	`856`	`},`
`853`	`857`	`}`
`854`		`- } else return { text: machine, value: { machine, cpu: limits.cpu, memory: limits.memory } }`
	`858`	`+ } else`
	`859`	`+ return { text: machine, value: { machine, cpu: limits.cpu, memory: limits.memory } }`
`855`	`860`	`}`
`856`	`861`	`})`
`857`	`862`	`} else {`
`@@ -1746,7 +1751,7 @@ export const useFunc = (model) => {`
`1746`	`1751`	`return getIssuer(url)`
`1747`	`1752`	`}`
`1748`	`1753`	`return clusterIssuers`
`1749`		`- }else if (!kind) {`
	`1754`	`+ } else if (!kind) {`
`1750`	`1755`	`commit('wizard/model$delete', '/spec/tls/issuerRef/name')`
`1751`	`1756`	`return []`
`1752`	`1757`	`}`
`@@ -2171,6 +2176,6 @@ export const useFunc = (model) => {`
`2171`	`2176`	`getNestedValue,`
`2172`	`2177`	`hasTopologyType,`
`2173`	`2178`	`getResourceConfig,`
`2174`		`- isTlsEnabled`
	`2179`	`+ isTlsEnabled,`
`2175`	`2180`	`}`
`2176`	`2181`	`}`
Original file line number	Diff line number	Diff line change
`@@ -326,7 +326,6 @@ export const useFunc = (model) => {`
`326`	`326`	`)`
`327`	`327`
`328`	`328`	`// Initialize on load`
`329`		`- getDbDetails()`
`330`	`329`	`showAndInitOpsRequestType()`
`331`	`330`
`332`	`331`	`// =====================================================`
`@@ -482,7 +481,12 @@ export const useFunc = (model) => {`
`482`	`481`
`483`	`482`	`function isTlsEnabled() {`
`484`	`483`	`const dbDetails = getValue(discriminator, '/dbDetails')`
`485`		`- return (dbDetails?.spec?.sslMode && dbDetails?.spec?.sslMode !== 'disabled' && dbDetails?.spec?.sslMode !== 'disable') \|\| dbDetails?.spec?.tls`
	`484`	`+ return (`
	`485`	`+ (dbDetails?.spec?.sslMode &&`
	`486`	`+ dbDetails?.spec?.sslMode !== 'disabled' &&`
	`487`	`+ dbDetails?.spec?.sslMode !== 'disable') \|\|`
	`488`	`+ dbDetails?.spec?.tls`
	`489`	`+ )`
`486`	`490`	`}`
`487`	`491`
`488`	`492`	`function initDatabaseRef() {`
`@@ -903,7 +907,8 @@ export const useFunc = (model) => {`
`903`	`907`	`memory: machineData.limits.memory,`
`904`	`908`	`},`
`905`	`909`	`}`
`906`		`- } else return { text: machine, value: { machine, cpu: limits.cpu, memory: limits.memory } }`
	`910`	`+ } else`
	`911`	`+ return { text: machine, value: { machine, cpu: limits.cpu, memory: limits.memory } }`
`907`	`912`	`}`
`908`	`913`	`})`
`909`	`914`	`} else {`
`@@ -2078,6 +2083,6 @@ export const useFunc = (model) => {`
`2078`	`2083`	`setApplyToIfReady,`
`2079`	`2084`	`hasResourceValue,`
`2080`	`2085`	`hasVolumeExpansion,`
`2081`		`- isTlsEnabled`
	`2086`	`+ isTlsEnabled,`
`2082`	`2087`	`}`
`2083`	`2088`	`}`
Original file line number	Diff line number	Diff line change
`@@ -317,7 +317,6 @@ export const useFunc = (model) => {`
`317`	`317`	`store.state,`
`318`	`318`	`)`
`319`	`319`
`320`		`- getDbDetails()`
`321`	`320`	`showAndInitOpsRequestType()`
`322`	`321`	`async function fetchJsons({ axios, itemCtx }) {`
`323`	`322`	`let ui = {}`
Original file line number	Diff line number	Diff line change
`@@ -328,7 +328,6 @@ export const useFunc = (model) => {`
`328`	`328`	`store.state,`
`329`	`329`	`)`
`330`	`330`
`331`		`- getDbDetails()`
`332`	`331`	`showAndInitOpsRequestType()`
`333`	`332`	`async function fetchJsons({ axios, itemCtx }) {`
`334`	`333`	`let ui = {}`
Original file line number	Diff line number	Diff line change
`@@ -318,7 +318,6 @@ export const useFunc = (model) => {`
`318`	`318`	`store.state,`
`319`	`319`	`)`
`320`	`320`
`321`		`- getDbDetails()`
`322`	`321`	`showAndInitOpsRequestType()`
`323`	`322`	`async function fetchJsons({ axios, itemCtx }) {`
`324`	`323`	`let ui = {}`