CraterCrash / godot-orchestrator

Orchestrator: Unleashing Creativity with Visual Scripting
https://www.cratercrash.com/orchestrator
Apache License 2.0
981 stars 53 forks source link

Godot's `get_current_scene` can return a nullptr, causing a crash #796

Closed Naros closed 1 month ago

Naros commented 1 month ago

Describe the bug

The PropertyGet, PropertySet, ShowMessage, and PrintString nodes all use the get_tree()->get_current_scene() API. While we have not seen any issues with its usage in our demos, some users have reported crashes.

Expected behavior

No response

Actual behavior

No response

How to Reproduce?

No response

Godot full version

No response

Orchestrator version

2.1.1.stable / 2.0.3.stable

Additional information

The workaround appears to use get_tree()->get_root() instead.