[Bf-blender-cvs] [bb95895b190] ui-asset-view-template: LibOverride: Fix missing update after an override reset.

Bastien Montagne noreply at git.blender.org
Sun Mar 21 16:11:58 CET 2021


Commit: bb95895b190dd1b0fd8808937cc840cc50453b58
Author: Bastien Montagne
Date:   Thu Mar 18 15:58:20 2021 +0100
Branches: ui-asset-view-template
https://developer.blender.org/rBbb95895b190dd1b0fd8808937cc840cc50453b58

LibOverride: Fix missing update after an override reset.

We need to brute-force reset IDs with `ID_RECALC_ALL` here...

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

M	source/blender/blenkernel/intern/lib_override.c

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

diff --git a/source/blender/blenkernel/intern/lib_override.c b/source/blender/blenkernel/intern/lib_override.c
index 4ec130f8388..3b5566c6cdc 100644
--- a/source/blender/blenkernel/intern/lib_override.c
+++ b/source/blender/blenkernel/intern/lib_override.c
@@ -2240,10 +2240,11 @@ void BKE_lib_override_library_update(Main *bmain, ID *local)
 
   local->tag |= LIB_TAG_OVERRIDE_LIBRARY_REFOK;
 
-  /* Full rebuild of Depsgraph! */
-  /* Note: this is really brute force, in theory updates from RNA should have handled this already,
-   * but for now let's play it safe. */
-  DEG_id_tag_update_ex(bmain, local, ID_RECALC_COPY_ON_WRITE);
+  /* Note: Since we reload full content from linked ID here, potentially from edited local
+   * override, we do not really have a way to know *what* is changed, so we need to rely on the
+   * massive destruction weapon of `ID_RECALC_ALL` here. */
+  DEG_id_tag_update_ex(bmain, local, ID_RECALC_ALL);
+  /* For same reason as above, also assume that the relationships between IDs changed. */
   DEG_relations_tag_update(bmain);
 }



More information about the Bf-blender-cvs mailing list