Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Output error when select a TileMap node in the editor #83185

Closed
pinellos opened this issue Oct 12, 2023 · 7 comments
Closed

Output error when select a TileMap node in the editor #83185

pinellos opened this issue Oct 12, 2023 · 7 comments

Comments

@pinellos
Copy link

Godot version

4.x

System information

Windows, WebEditor

Issue description

The output print this error everytime a TileMap node is selected in editor:

editor/editor_node.cpp:8223 - Condition "plugins_list.has(p_plugin)" is true.

Steps to reproduce

Add a TileMap node and add a tileset to it, then select the TileMap node in tree.

Minimal reproduction project

N/A

@AThousandShips
Copy link
Member

AThousandShips commented Oct 12, 2023

What version are you on? 4.x could be any of 8 different stable versions, and 3 dev branches

@pinellos
Copy link
Author

This happens on every 4.x version i've tried, 4.1.2 / 4.2 / WebEditor from Godot website...

@Chaosus Chaosus changed the title Outuput error when select a Tilemap node in editor Output error when select a TileMap node in the editor Oct 12, 2023
@leowise
Copy link

leowise commented Dec 3, 2023

Similar Issue here!

Environment:
OS Name Microsoft Windows 11 Pro
Version 10.0.22621 Build 22621

Godot:
v4.1.3.stable.official [f06b6836a]

Issue description:

TileMap > TileSet > Tiles Box with Tileset selected and Setup Menu selected AFTER adding tileset png, if select
TileSet on TileMap in Inspector:

The output print this error everytime:

editor/editor_node.cpp:8224 - Condition "plugins_list.has(p_plugin)" is true.

If attempt to grab and drop tile onto TileMap in editor window nothing is added.

Steps to reproduce
Add a TileMap node and add a tileset to it, then click on the TileSet on the Inspector.
Attempt to add a tile to editor window.

@jakemeinershagen
Copy link

jakemeinershagen commented Dec 9, 2023

Same here
Environment:
Linux Mint 21.1 Cinnamon

Godot:
v4.1.3.stable.official [f06b6836a]

Reproduce:
In my normal project it happens when I click on the tilemap.

I was able to reproduce it a blank project by following steps:

  1. New project
  2. Create Node2d
  3. Add tilemap
  4. Click on <empty> in the inspector to add new tileset.
  5. Click on your newly added tileset (error should print now)
  6. Error will now print every time you click off and back on to the tilemap

I did notice some weird behaviors that change when the error pops up:

  • If I create the tilemap and add the tileset, but make sure that I don't click on the tileset, I can click back and forth between the node 2d and tilemap without the error appearing. I can even close godot and reopen and still not get the error. However, as soon as I have clicked the tileset once I get the error every time I click the tilemap afterward.
  • The above is only true in the case that the tilemap is new and the tileset on that map has not been clicked. If I create the tilemap, add a tileset, click the tileset (which gives error), remove the tileset, add new tileset (just the act of adding the tileset now creates the error)

Looks like there is a previous issue that was marked as closed for something just like this #74543 (closed by #74717).

@AThousandShips
Copy link
Member

Can you please try this in 4.2? It has been most likely resolved there

@jakemeinershagen
Copy link

Yep, looks like it is resolved in 4.2. Should have checked that. Thanks

@AThousandShips
Copy link
Member

Closing as per the above, a cherry picked fix might be added to 4.1

@AThousandShips AThousandShips added this to the 4.2 milestone Dec 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants