LibOverride: Fixed bug in removing override op operator.
authorBastien Montagne <montagne29@wanadoo.fr>
Fri, 23 Aug 2019 13:39:19 +0000 (15:39 +0200)
committerBastien Montagne <montagne29@wanadoo.fr>
Fri, 23 Aug 2019 13:55:00 +0000 (15:55 +0200)
In case the property is a RNA pointer, `RNA_path_resolve()` will try to
resolve it and return that pointer, instead of returning expected
container... That is a bad inconsistency in the rNA path API, but no
proper way to solve it for now...

source/blender/editors/interface/interface_ops.c
source/blender/makesrna/intern/rna_access.c

index 819ec4fc2d38caa8e5f024efe215dc9ab9c11411..0afd67e5e66c6252a486ef11c37e42d28679aa51 100644 (file)
@@ -629,8 +629,9 @@ static int override_remove_button_exec(bContext *C, wmOperator *op)
   /* We need source (i.e. linked data) to restore values of deleted overrides...
    * If this is an override template, we obviously do not need to restore anything. */
   if (!is_template) {
+    PropertyRNA *src_prop;
     RNA_id_pointer_create(id->override_library->reference, &id_refptr);
-    if (!RNA_path_resolve(&id_refptr, oprop->rna_path, &src, NULL)) {
+    if (!RNA_path_resolve_property(&id_refptr, oprop->rna_path, &src, &src_prop)) {
       BLI_assert(0 && "Failed to create matching source (linked data) RNA pointer");
     }
   }
index d44020a52a18dd2e73dc27d385981ea18c7d503b..8327456f460f59b6ecfa5c2475ab498b6cd823d5 100644 (file)
@@ -5360,6 +5360,10 @@ static bool rna_path_parse(PointerRNA *ptr,
  * Resolve the given RNA Path to find the pointer and/or property
  * indicated by fully resolving the path.
  *
+ * \warning Unlike \a RNA_path_resolve_property(), that one *will* try to follow RNAPointers,
+ * e.g. the path 'parent' applied to a RNAObject \a ptr will return the object.parent in \a r_ptr,
+ * and a NULL \a r_prop...
+ *
  * \note Assumes all pointers provided are valid
  * \return True if path can be resolved to a valid "pointer + property" OR "pointer only"
  */