asgardius/virtualx-engine
1
0
Fork 0
Code Issues Pull requests Projects Releases 1 Packages Wiki Activity Actions

Guard agains duplicate calling of _ready when instanced in _enter_tree

Browse source 
Fixes #6005

(cherry picked from commit 184173a9b9)
This commit is contained in:
Bojidar Marinov 2016-11-17 18:10:53 +02:00 committed by Rémi Verschelde
parent fd5e01d1cd
commit 13cdc2a6f5
2 changed files with 6 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
scene/main/node.cpp
View file

@ -140,6 +140,7 @@ void Node::_notification(int p_notification) {
void Node::_propagate_ready() { void Node::_propagate_ready() {
data.ready_notified=true;
data.blocked++; data.blocked++;
for (int i=0;i<data.children.size();i++) { for (int i=0;i<data.children.size();i++) {
@ -2005,7 +2006,9 @@ void Node::_set_tree(SceneTree *p_tree) {
_propagate_enter_tree(); _propagate_enter_tree();
_propagate_ready(); //reverse_notification(NOTIFICATION_READY); if (!data.parent || data.parent->data.ready_notified) { // No parent (root) or parent ready
_propagate_ready(); //reverse_notification(NOTIFICATION_READY);
}
tree_changed_b=data.tree; tree_changed_b=data.tree;
@ -2284,6 +2287,7 @@ Node::Node() {
data.fixed_process=false; data.fixed_process=false;
data.idle_process=false; data.idle_process=false;
data.inside_tree=false; data.inside_tree=false;
data.ready_notified=false;
data.owner=NULL; data.owner=NULL;
data.OW=NULL; data.OW=NULL;

1
scene/main/node.h
View file

@ -85,6 +85,7 @@ private:
StringName name; StringName name;
SceneTree *tree; SceneTree *tree;
bool inside_tree; bool inside_tree;
bool ready_notified;
#ifdef TOOLS_ENABLED #ifdef TOOLS_ENABLED
NodePath import_path; //path used when imported, used by scene editors to keep tracking NodePath import_path; //path used when imported, used by scene editors to keep tracking
#endif #endif