A CodeBlock should not be allowed to attach to a node that has a script attached already

[tindrew]

What Happened

I discovered this purely by accident, but it should highlight some new functionality that needs to be added.

If there's a script on a node, you can still add a BlockCode to it. I assume that is not behavior that should happen, especially since it causes problems.

If I had to pick a fix for it, I'd say if you to attach a BlockCode to a node that has a script already, you should get a popup saying "You can't add a BlockCode to a node with an existing script!" (and also not let you add it of course).

How to Reproduce

1. Add any node to a scene
2. Add a script such as the starter player script
3. Add a block to the same node

Block Coding Plugin Version

v0.9.0

Godot Engine Version

v4.3.0 Stable

Operating System

Windows 10

Logs or command-line output

Anything else?

No response

[manuq]

Good point, the user should be informed at least. A configuration warning ⚠ should be added to the Block Coding node, saying something like "This Block Coding will override the text script".

[wjt]

I think that the warning that was added in #361 is too eager to fire. Take a look at the pong example:

The warning fires for both of the paddles because they have a script attached (namely simple_character.gd) and also a BlockCode node as a child. However the warning is false: the BlockCode node is smart enough to generate code that extends SimpleCharacter so there is no problem here.

[manuq]

Oh good catch @wjt, my fault, thanks for noting this.