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

Add checks for scene type reimports #89428

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Jonathan-Jay-TB
Copy link

-Removes duplicate scene that is added during reimport when it is safe to do so.
-Added some safeties to the set_edited_scene() function of the EditorNode script (doesn't check if the signal exists when disconnecting it, and same issue as the next point)
-Adds a safety to the _add_nodes() function of the SceneTreeEditor script (would print an error if the node had its parent changed from the viewport/null, which can happen in plugin scripts)

Could be improved by properly selecting the node directly to restore the node tree, instead of using set_edited_scene() on line 1063 of editor_node.cpp (possible slight efficiency gain)

Fixes #87451

-Mainly affects gltf and other custom importable files
-Removes duplicate that is created
-Adds safeties to certain parts of the EditorNode script
@Jonathan-Jay-TB Jonathan-Jay-TB requested review from a team as code owners March 12, 2024 19:22
@Jonathan-Jay-TB Jonathan-Jay-TB changed the title Added checks for scene type reimports Added checks for scene type reimports to avoid duplicate and error messages in specific cases Mar 12, 2024
@AThousandShips AThousandShips changed the title Added checks for scene type reimports to avoid duplicate and error messages in specific cases Add checks for scene type reimports Mar 13, 2024
@AThousandShips AThousandShips modified the milestones: 4.x, 4.3 Mar 13, 2024
@KoBeWi KoBeWi modified the milestones: 4.3, 4.4 Aug 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants