ioBroker.javascript icon indicating copy to clipboard operation
ioBroker.javascript copied to clipboard

Fix all Blockly definitions of custom blocks

Open klein0r opened this issue 1 year ago • 9 comments

klein0r avatar Jun 07 '24 10:06 klein0r

See https://github.com/google/blockly/releases/tag/blockly-v10.0.0 / https://github.com/google/blockly/pull/7077

  • Blockly.WidgetDiv.DIV -> Blockly.WidgetDiv.getDiv()
  • Blockly.JavaScript.variableDB_ -> Blockly.JavaScript.nameDB_
  • use Blockly.JavaScript.quote_() to add quoted strings which escapes single quotes ' (optional)
  • Mutator-Blocks must be updated:
if (typeof Blockly.icons === 'object') {
    this.setMutator(new Blockly.icons.MutatorIcon(['telegram_ask_mutator'], this));
} else {
    // Blockly 9.x
    this.setMutator(new Blockly.Mutator(['telegram_ask_mutator'], this));
}

klein0r avatar Jun 07 '24 13:06 klein0r

imap -> is fixed in next release >= 0.2.0 commit

Lucky-ESA avatar Jun 14 '24 16:06 Lucky-ESA

@Lucky-ESA Thanks, just updated the generated code to fit the code style of all other blockly blocks: https://github.com/Lucky-ESA/ioBroker.imap/pull/75

klein0r avatar Jun 14 '24 18:06 klein0r

@klein0r was muss angepasst werden? Adapter: LoRaWan

BenAhrdt avatar Jun 27 '24 15:06 BenAhrdt

@klein0r Wenn ein Adapter nicht angepasst werden kann wird da ab irgendwann was passieren das er nicht mhr geht?

Apollon77 avatar Jun 27 '24 15:06 Apollon77

Eigentlich muss nichts angepasst werden. Nur Bausteine mit Mutator-Blöcken. Das sind aber super wenige Adapter.

klein0r avatar Jun 28 '24 17:06 klein0r

Eigentlich muss nichts angepasst werden. Nur Bausteine mit Mutator-Blöcken. Das sind aber super wenige Adapter.

@klein0r Ich dachte es müsste was erledigt werden, da der Adapter oben in deiner Liste steht. Also muss dazu nichts angepasst werden? Ich habe im Adapter auch nur custom sendTo Blöcke drin.

BenAhrdt avatar Jun 28 '24 18:06 BenAhrdt

@BenAhrdt Ich habe einfach nur vor, alle Blockly-Bausteine mal anzuschauen und im gleichen Stil aufzubauen. Mehr nicht. 95% Kosmetik.

klein0r avatar Jun 28 '24 19:06 klein0r

@klein0r discord fixed in latest release v2.3.1

https://github.com/crycode-de/ioBroker.discord/commit/2dc6b9a527ecb767b266db4d1c56e9ff82c6d166

crycode-de avatar Jun 29 '24 19:06 crycode-de