Fix for material/node tree localization
authorSergey Sharybin <sergey.vfx@gmail.com>
Thu, 3 May 2018 09:07:09 +0000 (11:07 +0200)
committerSergey Sharybin <sergey.vfx@gmail.com>
Thu, 3 May 2018 09:25:47 +0000 (11:25 +0200)
The workaround removal was wrong, the whole id->adt of a local
copy is to be NULL.

But now instead of modifying original datablock, we tell library
manager to not copy animation data.

source/blender/blenkernel/BKE_library.h
source/blender/blenkernel/intern/library.c
source/blender/blenkernel/intern/material.c
source/blender/blenkernel/intern/node.c

index f3df8b9b363cc9dfc20b9d26aef705a19f9b5977..c34571fdd7a982c1d964ddd0dce44fb151ac63c8 100644 (file)
@@ -78,6 +78,7 @@ enum {
        /* XXX TODO Do we want to keep that? would rather try to get rid of it... */
        LIB_ID_COPY_ACTIONS            = 1 << 19,  /* EXCEPTION! Deep-copy actions used by animdata of copied ID. */
        LIB_ID_COPY_KEEP_LIB           = 1 << 20,  /* Keep the library pointer when copying datablock outside of bmain. */
+       LIB_ID_COPY_NO_ANIMDATA        = 1 << 21,  /* Don't copy id->adt, used by ID datablock localization routines. */
 };
 
 void BKE_libblock_copy_ex(struct Main *bmain, const struct ID *id, struct ID **r_newid, const int flag);
index fa38e3c441aaee195ad97c68984fff6222f57650..d23d48752b6ad64245d485651d5a9a3742490b7f 100644 (file)
@@ -1429,8 +1429,14 @@ void BKE_libblock_copy_ex(Main *bmain, const ID *id, ID **r_newid, const int fla
 #endif
 
        /* the duplicate should get a copy of the animdata */
-       BLI_assert((flag & LIB_ID_COPY_ACTIONS) == 0 || (flag & LIB_ID_CREATE_NO_MAIN) == 0);
-       id_copy_animdata(bmain, new_id, (flag & LIB_ID_COPY_ACTIONS) != 0 && (flag & LIB_ID_CREATE_NO_MAIN) == 0);
+       if ((flag & LIB_ID_COPY_NO_ANIMDATA) == 0) {
+               BLI_assert((flag & LIB_ID_COPY_ACTIONS) == 0 || (flag & LIB_ID_CREATE_NO_MAIN) == 0);
+               id_copy_animdata(bmain, new_id, (flag & LIB_ID_COPY_ACTIONS) != 0 && (flag & LIB_ID_CREATE_NO_MAIN) == 0);
+       }
+       else if (id_can_have_animdata(new_id)) {
+               IdAdtTemplate *iat = (IdAdtTemplate *)new_id;
+               iat->adt = NULL;
+       }
 
        if ((flag & LIB_ID_CREATE_NO_DEG_TAG) == 0 && (flag & LIB_ID_CREATE_NO_MAIN) == 0) {
                DEG_id_type_tag(bmain, GS(new_id->name));
index 725a7b0719471baf6bd012427341cad1ba151320..131d79c249f062895c595eb9a10208846b080371 100644 (file)
@@ -187,15 +187,18 @@ Material *BKE_material_localize(Material *ma)
         * ... Once f*** nodes are fully converted to that too :( */
 
        Material *man;
-       
-       man = BKE_libblock_copy_nolib(&ma->id, false);
+
+       BKE_id_copy_ex(
+               NULL, &ma->id, (ID **)&man,
+               (LIB_ID_CREATE_NO_MAIN |
+                LIB_ID_CREATE_NO_USER_REFCOUNT |
+                LIB_ID_COPY_NO_PREVIEW |
+                LIB_ID_COPY_NO_ANIMDATA),
+               false);
 
        man->texpaintslot = NULL;
        man->preview = NULL;
-       
-       if (ma->nodetree)
-               man->nodetree = ntreeLocalize(ma->nodetree);
-       
+
        BLI_listbase_clear(&man->gpumaterial);
 
        /* TODO Duplicate Engine Settings and set runtime to NULL */
index f3fd8b69b2c046d08dc06c43230940613331599a..3b51ce9366f0ed8c8cdcd74eb9af940b527ed59c 100644 (file)
@@ -2001,8 +2001,13 @@ bNodeTree *ntreeLocalize(bNodeTree *ntree)
                /* Make full copy outside of Main database.
                 * Note: previews are not copied here.
                 */
-               BKE_id_copy_ex(G.main, (ID *)ntree, (ID **)&ltree,
-                              LIB_ID_CREATE_NO_MAIN | LIB_ID_CREATE_NO_USER_REFCOUNT | LIB_ID_COPY_NO_PREVIEW, false);
+               BKE_id_copy_ex(
+                       NULL, &ntree->id, (ID **)&ltree,
+                       (LIB_ID_CREATE_NO_MAIN |
+                        LIB_ID_CREATE_NO_USER_REFCOUNT |
+                        LIB_ID_COPY_NO_PREVIEW |
+                        LIB_ID_COPY_NO_ANIMDATA),
+                       false);
                ltree->flag |= NTREE_IS_LOCALIZED;
 
                for (node = ltree->nodes.first; node; node = node->next) {