Browse Source

Merge pull request #109444 from thederickff/ba-transform-before-esk

Fix `BoneAttachment3D` getting global transform of external skeleton before it `is_inside_tree()`
Thaddeus Crews 1 month ago
parent
commit
7791eed739
1 changed files with 6 additions and 1 deletions
  1. 6 1
      scene/3d/bone_attachment_3d.cpp

+ 6 - 1
scene/3d/bone_attachment_3d.cpp

@@ -301,7 +301,12 @@ void BoneAttachment3D::on_skeleton_update() {
 		if (sk) {
 			if (!override_pose) {
 				if (use_external_skeleton) {
-					set_global_transform(sk->get_global_transform() * sk->get_bone_global_pose(bone_idx));
+					if (sk->is_inside_tree()) {
+						set_global_transform(sk->get_global_transform() * sk->get_bone_global_pose(bone_idx));
+						// Else, do nothing, the transform will be set when the skeleton enters the tree:
+						// Skeleton3D::_notification(NOTIFICATION_ENTER_TREE) -> calls Skeleton3D::_notification(NOTIFICATION_UPDATE_SKELETON)
+						// -> emits skeleton_updated signal -> connected to BoneAttachment3D::on_skeleton_update()
+					}
 				} else {
 					set_transform(sk->get_bone_global_pose(bone_idx));
 				}