Merge pull request #39448 from mrushyendra/duplicate_signals
[3.2] Fix signal duplication bug when duplicating node with instanced children
This commit is contained in:
commit
0eaefa73a1
1 changed files with 42 additions and 33 deletions
|
@ -2247,17 +2247,24 @@ void Node::_duplicate_and_reown(Node *p_new_parent, const Map<Node *, Node *> &p
|
|||
// if the emitter node comes later in tree order than the receiver
|
||||
void Node::_duplicate_signals(const Node *p_original, Node *p_copy) const {
|
||||
|
||||
if (this != p_original && (get_owner() != p_original && get_owner() != p_original->get_owner()))
|
||||
if ((this != p_original) && !(p_original->is_a_parent_of(this))) {
|
||||
return;
|
||||
}
|
||||
|
||||
List<const Node *> process_list;
|
||||
process_list.push_back(this);
|
||||
while (!process_list.empty()) {
|
||||
|
||||
const Node *n = process_list.front()->get();
|
||||
process_list.pop_front();
|
||||
|
||||
List<Connection> conns;
|
||||
get_all_signal_connections(&conns);
|
||||
n->get_all_signal_connections(&conns);
|
||||
|
||||
for (List<Connection>::Element *E = conns.front(); E; E = E->next()) {
|
||||
|
||||
if (E->get().flags & CONNECT_PERSIST) {
|
||||
//user connected
|
||||
NodePath p = p_original->get_path_to(this);
|
||||
NodePath p = p_original->get_path_to(n);
|
||||
Node *copy = p_copy->get_node(p);
|
||||
|
||||
Node *target = Object::cast_to<Node>(E->get().target);
|
||||
|
@ -2268,12 +2275,13 @@ void Node::_duplicate_signals(const Node *p_original, Node *p_copy) const {
|
|||
|
||||
Node *copytarget = target;
|
||||
|
||||
// Atempt to find a path to the duplicate target, if it seems it's not part
|
||||
// Attempt to find a path to the duplicate target, if it seems it's not part
|
||||
// of the duplicated and not yet parented hierarchy then at least try to connect
|
||||
// to the same target as the original
|
||||
|
||||
if (p_copy->has_node(ptarget))
|
||||
if (p_copy->has_node(ptarget)) {
|
||||
copytarget = p_copy->get_node(ptarget);
|
||||
}
|
||||
|
||||
if (copy && copytarget && !copy->is_connected(E->get().signal, copytarget, E->get().method)) {
|
||||
copy->connect(E->get().signal, copytarget, E->get().method, E->get().binds, E->get().flags);
|
||||
|
@ -2281,8 +2289,9 @@ void Node::_duplicate_signals(const Node *p_original, Node *p_copy) const {
|
|||
}
|
||||
}
|
||||
|
||||
for (int i = 0; i < get_child_count(); i++) {
|
||||
get_child(i)->_duplicate_signals(p_original, p_copy);
|
||||
for (int i = 0; i < n->get_child_count(); i++) {
|
||||
process_list.push_back(n->get_child(i));
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in a new issue