From bea29e64e1efc0e3c37d08f407c3763eae065fbd Mon Sep 17 00:00:00 2001
From: stefan-hoehn <mail@stefanhoehn.com>
Date: Thu, 27 Jul 2023 23:41:04 +0200
Subject: [PATCH] Blockly: Support String/Number on eventcontext state/command
 (#1992)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Fixes #1991.

Support conversion of event context types itemState, oldItemState,
itemCommand as String or Number by supplying a mutating block with a
choise of Number / String.

Also-by: Florian Hotze <florianh_dev@icloud.com>
Signed-off-by: Stefan Höhn <mail@stefanhoehn.com>
(cherry picked from commit edc124e4f89f4710ac5506edae979f6f76391a14)
---
 .../definitions/blockly/blocks-scripts.js     | 52 +++++++++++++++++--
 1 file changed, 48 insertions(+), 4 deletions(-)

diff --git a/bundles/org.openhab.ui/web/src/assets/definitions/blockly/blocks-scripts.js b/bundles/org.openhab.ui/web/src/assets/definitions/blockly/blocks-scripts.js
index 53da520aaa..4929a5de58 100644
--- a/bundles/org.openhab.ui/web/src/assets/definitions/blockly/blocks-scripts.js
+++ b/bundles/org.openhab.ui/web/src/assets/definitions/blockly/blocks-scripts.js
@@ -171,7 +171,7 @@ export default function defineOHBlocks_Scripts (f7, isGraalJs, scripts) {
           ['received command', 'itemCommand'],
           ['triggered channel', 'channel'],
           ['triggered event', 'event']
-        ]),
+        ], this.handleTypeSelection.bind(this)),
         'contextInfo')
       this.contextInfo = this.getFieldValue('contextInfo')
       this.setInputsInline(true)
@@ -195,15 +195,49 @@ export default function defineOHBlocks_Scripts (f7, isGraalJs, scripts) {
       this.setHelpUrl('https://www.openhab.org/docs/configuration/blockly/rules-blockly-run-and-process.html#retrieve-rule-context-information')
     },
     onchange: function (event) {
-      let contextInfo = this.getFieldValue('contextInfo')
+      const contextInfo = this.getFieldValue('contextInfo')
+      const asType = this.getFieldValue('asType')
       if (this.contextInfo !== contextInfo) {
         this.contextInfo = contextInfo
         if (contextInfo === 'itemName') {
           this.setOutput(true, 'oh_item')
-          console.log('type = oh_item')
         } else {
           this.setOutput(true, 'String')
-          console.log('type = State String')
+        }
+      }
+
+      if (this.asType !== asType) {
+        this.asType = asType
+        if (this.methodName === 'itemState' || this.methodName === 'oldItemState' || this.methodName === 'itemCommand') {
+          if (asType === 'asNumber') {
+            this.setOutput(true, 'Number')
+          } else if (asType === 'asQuantity') {
+            this.setOutput(true, 'oh_quantity')
+          } else {
+            this.setOutput(true, 'String')
+          }
+        }
+      }
+    },
+    handleTypeSelection: function (methodName) {
+      if (this.methodName !== methodName) {
+        this.methodName = methodName
+        this.updateShape()
+      }
+    },
+    updateShape: function () {
+      if (this.methodName === 'itemState' || this.methodName === 'oldItemState' || this.methodName === 'itemCommand') {
+        if (!this.getInput('asTypeInput')) {
+          this.appendDummyInput('asTypeInput').appendField(new Blockly.FieldDropdown([
+            ['as String', 'asString'],
+            ['as Number', 'asNumber'],
+            ['as Quantity', 'asQuantity']
+          ]),
+          'asType')
+        }
+      } else {
+        if (this.getInput('asTypeInput')) {
+          this.removeInput('asTypeInput')
         }
       }
     }
@@ -211,7 +245,17 @@ export default function defineOHBlocks_Scripts (f7, isGraalJs, scripts) {
 
   javascriptGenerator['oh_context_info'] = function (block) {
     const contextInfo = block.getFieldValue('contextInfo')
+    const type = block.getFieldValue('asType')
     if (contextInfo === 'ruleUID') return ['ctx.ruleUID', javascriptGenerator.ORDER_ATOMIC]
+    if (contextInfo === 'itemState' || contextInfo === 'oldItemState' || contextInfo === 'itemCommand') {
+      if (type === 'asNumber') {
+        return [`parseFloat(event.${contextInfo}.toString())`, javascriptGenerator.ORDER_ATOMIC]
+      } else if (type === 'asQuantity') {
+        return [`Quantity(event.${contextInfo}.toString())`, javascriptGenerator.ORDER_ATOMIC]
+      } else {
+        return [`event.${contextInfo}.toString()`, javascriptGenerator.ORDER_ATOMIC]
+      }
+    }
     return [`event.${contextInfo}`, javascriptGenerator.ORDER_ATOMIC]
   }