[blockly] fix item.getAttributes (#1254)

Signed-off-by: Stefan Höhn <stefan@andreaundstefanhoehn.de>

bundles/org.openhab.ui/web/src/assets/definitions/blockly/blocks-items.js

@@ -87,25 +87,80 @@ export default function (f7) {
     return [code, 0]
   }
 
+  /*
+  * Provides all attributes from an item
+  * - name: String
+  * - label: String
+  * - state: State
+  * - category: String
+  * - tags: Array
+  * - groups: Array
+  * - type: String
+  * Blockly part
+  */
   Blockly.Blocks['oh_getitem_attribute'] = {
     init: function () {
+      let thisBlock = this
+      let dropdown = new Blockly.FieldDropdown(
+        [['name', 'Name'], ['label', 'Label'], ['state', 'State'], ['category', 'Category'], ['tags', 'Tags'], ['groups', 'GroupNames'], ['type', 'Type']],
+        function (newMode) {
+          thisBlock.updateType_(newMode)
+        })
       this.appendValueInput('item')
         .appendField('get ')
-        .appendField(new Blockly.FieldDropdown([['name', 'Name'], ['label', 'Label'], ['state', 'State'], ['category', 'Category'], ['tags', 'Tags'], ['groups', 'GroupNames'], ['type', 'Type']]), 'attributeName')
+        .appendField(dropdown, 'attributeName')
         .appendField('of item')
         .setCheck('oh_itemtype')
       this.setInputsInline(false)
-      this.setOutput(true, 'any')
+      this.setOutput(true, 'String')
       this.setColour(0)
-      this.setTooltip('Retrieve a specific attribute from the item')
+      this.setTooltip('Retrieve a specific attribute from the item. Note that groups and tags return a list and should be used with the loops-block \'for each item ... in list\'. ')
       this.setHelpUrl('https://www.openhab.org/docs/configuration/items.html')
+    },
+    /**
+    * Modify this block to have the correct output type based on the attribute.
+    */
+    updateType_: function (newAttributeName) {
+      let attributeName = this.getFieldValue('attributeName')
+      if (newAttributeName === 'Tags' || newAttributeName === 'GroupNames') {
+        this.outputConnection.setCheck('Array')
+      } else {
+        this.outputConnection.setCheck('String')
+      }
+    },
+    /**
+    * Create XML to represent the input and output types.
+    * @return {!Element} XML storage element.
+    * @this {Blockly.Block}
+    */
+    mutationToDom: function () {
+      let container = Blockly.utils.xml.createElement('mutation')
+      container.setAttribute('attributeName', this.getFieldValue('attributeName'))
+      return container
+    },
+    /**
+    * Parse XML to restore the input and output types.
+    * @param {!Element} xmlElement XML storage element.
+    * @this {Blockly.Block}
+    */
+    domToMutation: function (xmlElement) {
+      this.updateType_(xmlElement.getAttribute('attributeName'))
     }
   }
 
+  /*
+  * Provides all attributes from an item
+  * Code part
+  */
   Blockly.JavaScript['oh_getitem_attribute'] = function (block) {
     const theItem = Blockly.JavaScript.valueToCode(block, 'item', Blockly.JavaScript.ORDER_ATOMIC)
     const attributeName = block.getFieldValue('attributeName')
-    let code = `${theItem}.get${attributeName}()`
+    let code = ''
+    if (attributeName === 'Tags' || attributeName === 'GroupNames') {
+      code = `Java.from(${theItem}.get${attributeName}())`
+    } else {
+      code = `${theItem}.get${attributeName}()`
+    }
     return [code, 0]
   }
 }

bundles/org.openhab.ui/web/src/assets/definitions/blockly/blocks-things.js

@@ -29,7 +29,7 @@ export default function defineOHBlocks (f7) {
   Blockly.Blocks['oh_getthing_state'] = {
     init: function () {
       this.appendValueInput('thingUid')
-        .appendField('get thing state')
+        .appendField('get thing status')
         .setCheck('String')
       this.setInputsInline(false)
       this.setOutput(true, 'String')

bundles/org.openhab.ui/web/src/pages/settings/rules/script/blockly-editor.vue

@@ -344,7 +344,7 @@
           <sep gap="48" />
           <block type="oh_thing" />
           <block type="oh_getthing_state">
-            <value name="itemName">
+            <value name="thingUid">
               <shadow type="oh_thing" />
             </value>
           </block>