Fix for Make Local -> All not working correct with multy-user datablocks
authorSergey Sharybin <sergey.vfx@gmail.com>
Mon, 1 Jul 2013 08:53:45 +0000 (08:53 +0000)
committerSergey Sharybin <sergey.vfx@gmail.com>
Mon, 1 Jul 2013 08:53:45 +0000 (08:53 +0000)
Make Local operator uses BKE_library_make_local function if all the
datablocks needs to be made local. And this function was calling
id_clear_lib_data for every datablock, which only clears library
data. But this function doesn't work correct for datablocks which
areshared by multiple users (this is also mentioned in comment
for this function).

This lead to situations when two datablocks shares the same runtime
data leading to crashes later. For example making everythig local in
scales cycles scene from durian ends up in a crash when toggling
rig edit mode.

Solved by using id_make_local instead of id_clear_lib_data, which
will ensure all the data are nicely expanded and made local.

Checked by Brecht, thanks fr the review!

source/blender/blenkernel/intern/library.c

index c1715ada7bdb5ea611ee39b2b0c3f7d13f93242f..5b5d4f3c318917e82f27aca3f98a778cf96e9a5d 100644 (file)
@@ -1573,7 +1573,7 @@ void BKE_library_make_local(Main *bmain, Library *lib, bool untagged_only)
                        {
                                if (lib == NULL || id->lib == lib) {
                                        if (id->lib) {
-                                               id_clear_lib_data(bmain, id); /* sets 'id->flag' */
+                                               id_make_local(id, false);
 
                                                /* why sort alphabetically here but not in
                                                 * id_clear_lib_data() ? - campbell */