Fix error when actionAnalyzerItems value is a string (#2159)

Fixes #2158. Only join `actionAnalyzerItems` values if the variable is an Array. Signed-off-by: Thomas Wunschel <[email protected]>
openhab · Nov 4, 2023 · f4bbba0 · f4bbba0
1 parent f1e2c39
commit f4bbba0
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/bundles/org.openhab.ui/web/src/components/widgets/widget-actions.js b/bundles/org.openhab.ui/web/src/components/widgets/widget-actions.js
@@ -253,10 +253,13 @@ export const actionsMixin = {
           break
         case 'analyze':
         case 'analyzer':
-          const actionAnalyzerItems = actionConfig[prefix + 'actionAnalyzerItems']
+          let actionAnalyzerItems = actionConfig[prefix + 'actionAnalyzerItems']
           const actionAnalyzerChartType = actionConfig[prefix + 'actionAnalyzerChartType']
           const actionAnalyzerCoordSystem = actionConfig[prefix + 'actionAnalyzerCoordSystem']
-          this.$f7.views.main.router.navigate(`/analyzer/?items=${actionAnalyzerItems.join(',')}&chartType=${actionAnalyzerChartType || ''}&coordSystem=${actionAnalyzerCoordSystem || ''}`)
+          if (Array.isArray(actionAnalyzerItems)) {
+            actionAnalyzerItems = actionAnalyzerItems.join(',')
+          }
+          this.$f7.views.main.router.navigate(`/analyzer/?items=${actionAnalyzerItems || ''}&chartType=${actionAnalyzerChartType || ''}&coordSystem=${actionAnalyzerCoordSystem || ''}`)
           console.log('Opening the analyzer')
           break
         case 'url':