block-plus-minus plugin: field named "TEXT" not found in "text_join" block

Category

• Plugins

Component

block-plus-minus

Describe the bug

block-plus-minus plugin, when used with Blockly 8.0.1 and the text_join block results in this warning in the console:

field named "TEXT" not found in "text_join" block (id="|rQF{.!!]-+H=?7ai!O0")

The warning seems to be coming from here: https://github.com/google/blockly/blob/master/blocks/text.js#L682

To Reproduce

1. Install @blockly/block-plus-minus

2. Import @blockly/block-plus-minus

import '@blockly/block-plus-minus';

1. Add text_join block to toolbox:

        {
          kind: 'block',
          type: 'text_join',
        },

Now load Blockly and drag the text_join block into the workspace.

Expected behavior

No warning should appear.

Screenshots

Additional context

[BeksOmega]

@johnnyoshika yeah it should be fine =)

The issue is that in core, we mix the quoting functionality directly into the text_create_with block, while in the plus/minus plugin we apply the extension. This means that on the plus/minus block we trigger quoting, but in core we do not.

If you look at the quoteField_ definition, it doesn’t do anything unless it actually finds a matching field. So the warning just means “hey I didn’t do anything”.

So I think it should be fine to put in production, but definitely double check my logic for yourself =)

[johnnyoshika]

@BeksOmega Given that this is only a warning, is it safe to deploy this to a live production environment?