diff options
author | Bastien Montagne <bastien@blender.org> | 2022-03-15 17:00:19 +0300 |
---|---|---|
committer | Bastien Montagne <bastien@blender.org> | 2022-03-15 17:02:39 +0300 |
commit | 2156841c8ca8e754e073ceb14912c7529af891bf (patch) | |
tree | f1148174b797ddd219a771b26687765c8b8cdc3b /source/blender/editors/armature | |
parent | 256d36683b755fbf4c3e4a80e3935888643719ea (diff) |
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.
Diffstat (limited to 'source/blender/editors/armature')
-rw-r--r-- | source/blender/editors/armature/armature_relations.c | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/source/blender/editors/armature/armature_relations.c b/source/blender/editors/armature/armature_relations.c index 45d64807e65..be4829c02be 100644 --- a/source/blender/editors/armature/armature_relations.c +++ b/source/blender/editors/armature/armature_relations.c @@ -377,6 +377,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); |