Commit: 2fc52fb742396bf790395a0327cf77c1c97fd229
Author: Bastien Montagne
Date:   Tue Mar 15 15:00:19 2022 +0100
Branches: blender-v3.1-release
https://developer.blender.org/rB2fc52fb742396bf790395a0327cf77c1c97fd229

Fix T96452: Armature corrupted after undoing 'Join' operation.

Modified source Armature ID in the join operation was not properly
tagged as such for the depsgraph (and therefore memfile undo)..

Issue caused/revealed by rBe648e388874a.

Should be backported to 3.1 should we make a corrective release.

===================================================================

M       source/blender/editors/armature/armature_relations.c

===================================================================

diff --git a/source/blender/editors/armature/armature_relations.c 
b/source/blender/editors/armature/armature_relations.c
index eebe8a447f7..86a7b02bc3b 100644
--- a/source/blender/editors/armature/armature_relations.c
+++ b/source/blender/editors/armature/armature_relations.c
@@ -393,6 +393,15 @@ int ED_armature_join_objects_exec(bContext *C, wmOperator 
*op)
         BKE_pose_channels_hash_free(pose);
       }
 
+      /* Armature ID itself is not freed below, however it has been modified 
(and is now completely
+       * empty). This needs to be told to the depsgraph, it will also ensure 
that the global
+       * memfile undo system properly detects the change.
+       *
+       * FIXME: Modifying an existing obdata because we are joining an object 
using it into another
+       * object is a very questionable behavior, which also does not match 
with other object types
+       * joining. */
+      DEG_id_tag_update_ex(bmain, &curarm->id, ID_RECALC_GEOMETRY);
+
       /* Fix all the drivers (and animation data) */
       BKE_fcurves_main_cb(bmain, joined_armature_fix_animdata_cb, &afd);
       BLI_ghash_free(afd.names_map, MEM_freeN, NULL);

_______________________________________________
Bf-blender-cvs mailing list
[email protected]
List details, subscription details or unsubscribe:
https://lists.blender.org/mailman/listinfo/bf-blender-cvs

Reply via email to