blender
/
blender
136
413
Fork 597
Code Issues 6.3k Pull Requests 770 Projects 19 Wiki Activity

Fix #98923 : Remove the ability to offset a linked collection asset not instantiated #104958

Open
Colin Marmond wants to merge 2 commits from Kdaf/blender:fix-98923-collection-asset-instance-offset_ into blender-v3.5-release
pull from: Kdaf/blender:fix-98923-collection-asset-instance-offset_
merge into: blender:blender-v3.5-release

When changing the target branch, be careful to rebase the branch in your fork to match. See documentation.
Conversation 0 Commits 2 Files Changed 1 +5 -3
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 875924d74b - Show all commits

8
source/blender/editors/object/object_add.cc
View File

@ -1859,16 +1859,18 @@ static int collection_drop_exec(bContext *C, wmOperator *op)
/* Disable location and rotation controls for the user when the collection
* is linked and not instantiated */
static bool collection_drop_poll_property(const bContext *C, wmOperator *op, const PropertyRNA *prop)
static bool collection_drop_poll_property(const bContext *C,
wmOperator *op,
const PropertyRNA *prop)
{
if (RNA_boolean_get(op->ptr, "use_instance"))
Julian Eisel commented 2023-05-31 17:48:32 +02:00
Review
Copy Link

Style: Always use braces https://wiki.blender.org/wiki/Style_Guide/C_Cpp#Braces

Style: Always use braces https://wiki.blender.org/wiki/Style_Guide/C_Cpp#Braces
return true;
const char *prop_name = RNA_property_ui_name(prop);
Julian Eisel commented 2023-05-31 17:52:05 +02:00
Review
Copy Link

This shouldn't use the UI name but the internal identifier, see RNA_property_identifier(). UI names are supposed to be changeable without compatibility concerns, plus they are translated. For example this code will break for languages that have "Rotation" or "Location" translated.

This shouldn't use the UI name but the internal identifier, see `RNA_property_identifier()`. UI names are supposed to be changeable without compatibility concerns, plus they are translated. For example this code will break for languages that have "Rotation" or "Location" translated.
if (STR_ELEM(prop_name, "Location", "Rotation")) {
Collection *collection = static_cast<Collection *>(
BLI_findlink(&CTX_data_main(C)->collections, RNA_enum_get(op->ptr, "collection")));
BLI_findlink(&CTX_data_main(C)->collections, RNA_enum_get(op->ptr, "collection")));
Julian Eisel commented 2023-05-31 18:09:26 +02:00
Review
Copy Link

This is not in sync with how collection_drop_exec() gets the collection, that first attempts to get the collection from the session UUID and the name, see collection_add_info_get_from_op().

This is not in sync with how `collection_drop_exec()` gets the collection, that first attempts to get the collection from the session UUID and the name, see `collection_add_info_get_from_op()`.
if (!ID_IS_LINKED(&collection->id))
Julian Eisel commented 2023-05-31 18:04:53 +02:00
Review
Copy Link

I'd omit the negation to make this more readable / less error-prone:

if (ID_IS_LINKED(&collection->id)) {
  return false;
}
return true;
I'd omit the negation to make this more readable / less error-prone: ``` if (ID_IS_LINKED(&collection->id)) { return false; } return true; ```
return true;
return false;