Browse Source

Fix editor-only visibility for lights

* Update visibility again for editor-only lights if owner changes.

Fixes #26399, supersedes #52327
reduz 3 years ago
parent
commit
d69e3791bf
6 changed files with 20 additions and 0 deletions
  1. 5 0
      scene/2d/light_2d.cpp
  2. 2 0
      scene/2d/light_2d.h
  3. 5 0
      scene/3d/light_3d.cpp
  4. 2 0
      scene/3d/light_3d.h
  5. 5 0
      scene/main/node.cpp
  6. 1 0
      scene/main/node.h

+ 5 - 0
scene/2d/light_2d.cpp

@@ -30,6 +30,11 @@
 
 #include "light_2d.h"
 
+void Light2D::owner_changed_notify() {
+	// For cases where owner changes _after_ entering tree (as example, editor editing).
+	_update_light_visibility();
+}
+
 void Light2D::_update_light_visibility() {
 	if (!is_inside_tree()) {
 		return;

+ 2 - 0
scene/2d/light_2d.h

@@ -73,6 +73,8 @@ private:
 
 	void _update_light_visibility();
 
+	virtual void owner_changed_notify() override;
+
 protected:
 	_FORCE_INLINE_ RID _get_light() const { return canvas_light; }
 	void _notification(int p_what);

+ 5 - 0
scene/3d/light_3d.cpp

@@ -176,6 +176,11 @@ Ref<Texture2D> Light3D::get_projector() const {
 	return projector;
 }
 
+void Light3D::owner_changed_notify() {
+	// For cases where owner changes _after_ entering tree (as example, editor editing).
+	_update_visibility();
+}
+
 void Light3D::_update_visibility() {
 	if (!is_inside_tree()) {
 		return;

+ 2 - 0
scene/3d/light_3d.h

@@ -85,6 +85,8 @@ private:
 
 	// bind helpers
 
+	virtual void owner_changed_notify() override;
+
 protected:
 	RID light;
 

+ 5 - 0
scene/main/node.cpp

@@ -420,6 +420,9 @@ void Node::move_child_notify(Node *p_child) {
 	// to be used when not wanted
 }
 
+void Node::owner_changed_notify() {
+}
+
 void Node::set_physics_process(bool p_process) {
 	if (data.physics_process == p_process) {
 		return;
@@ -1544,6 +1547,8 @@ void Node::_set_owner_nocheck(Node *p_owner) {
 	data.owner = p_owner;
 	data.owner->data.owned.push_back(this);
 	data.OW = data.owner->data.owned.back();
+
+	owner_changed_notify();
 }
 
 void Node::_release_unique_name_in_owner() {

+ 1 - 0
scene/main/node.h

@@ -208,6 +208,7 @@ protected:
 	virtual void add_child_notify(Node *p_child);
 	virtual void remove_child_notify(Node *p_child);
 	virtual void move_child_notify(Node *p_child);
+	virtual void owner_changed_notify();
 
 	void _propagate_replace_owner(Node *p_owner, Node *p_by_owner);