From d49579b038df558ddd70605f07e2000d4a42c2f8 Mon Sep 17 00:00:00 2001 From: robfram Date: Fri, 9 Mar 2018 19:05:04 +0100 Subject: [PATCH] Fix overwriting all common properties when using `Change Type` tool If you change the type of an existing node, it checks if you have modified the initial value of their properties before overwriting their values in the new node. For example, if you created a `Label` and changed it to `LineEdit`, the `mouse_filter` property was created as `Ignore` for the original `Label` node, and was maintained after changing it to `LineEdit` causing not to work as expected. Now it checks if `Ignore` is the default value for `Label` nodes, and as it is, the property value is left unchanged, maintaining the default value for `LineEdit`, which is `Stop`. Fix #13955 and alike. (cherry picked from commit 8ea4ea0d53e772673dea69a9df83aa8445ad49ea) --- editor/scene_tree_dock.cpp | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/editor/scene_tree_dock.cpp b/editor/scene_tree_dock.cpp index 45622d43e8a..9a4bd9a567d 100644 --- a/editor/scene_tree_dock.cpp +++ b/editor/scene_tree_dock.cpp @@ -1402,6 +1402,8 @@ void SceneTreeDock::_create() { Node *newnode = Object::cast_to(c); ERR_FAIL_COND(!newnode); + Node *default_oldnode = Object::cast_to(ClassDB::instance(n->get_class())); + List pinfo; n->get_property_list(&pinfo); @@ -1410,10 +1412,13 @@ void SceneTreeDock::_create() { continue; if (E->get().name == "__meta__") continue; - newnode->set(E->get().name, n->get(E->get().name)); + if (default_oldnode->get(E->get().name) != n->get(E->get().name)) { + newnode->set(E->get().name, n->get(E->get().name)); + } } editor->push_item(NULL); + memdelete(default_oldnode); //reconnect signals List sl;