blender
/
blender
135
420
Fork 602
Code Issues 6.3k Pull Requests 764 Projects 19 Wiki Activity

Fix #119909: Unkeyable custom properties receive keyframes #119914

Merged
Christoph Lendenfeld merged 13 commits from ChrisLend/blender:fix_more_selective_when_keying_custom_props into main 2024-04-12 14:48:20 +02:00
Conversation 15 Commits 13 Files Changed 4 +26 -5
3 changed files with 26 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit e62142e9f3 - Show all commits

21
source/blender/editors/animation/keyframing.cc
View File

@ -247,8 +247,12 @@ static int insert_key_with_keyingset(bContext *C, wmOperator *op, KeyingSet *ks)
return OPERATOR_FINISHED;
}
static bool is_keyable_type(IDProperty *prop)
static bool is_keyable_type(IDProperty *prop, const PropertyRNA *property_rna)
mont29 marked this conversation as resolved Outdated
Bastien Montagne commented 2024-04-12 11:31:52 +02:00
Outdated
Review
Copy Link

This name is not specific enough. Should be something like is_idproperty_keyable or so (the _type part can be removed I think, since it's not only checking the IDP type anymore).

This name is not specific enough. Should be something like `is_idproperty_keyable` or so (the `_type` part can be removed I think, since it's not only checking the IDP type anymore).
{
if (RNA_property_is_runtime(property_rna)) {
return false;
}
if (ELEM(prop->type,
eIDPropertyType::IDP_BOOLEAN,
eIDPropertyType::IDP_INT,
@ -325,12 +329,19 @@ static blender::Vector<std::string> construct_rna_paths(PointerRNA *ptr)
if (insert_channel_flags & USER_ANIM_KEY_CHANNEL_CUSTOM_PROPERTIES) {
if (properties) {
LISTBASE_FOREACH (IDProperty *, prop, &properties->data.group) {
if (!is_keyable_type(prop)) {
continue;
}
char name_escaped[MAX_IDPROP_NAME * 2];
BLI_str_escape(name_escaped, prop->name, sizeof(name_escaped));
paths.append(fmt::format("[\"{}\"]", name_escaped));
std::string path = fmt::format("[\"{}\"]", name_escaped);
PointerRNA resolved_ptr;
PropertyRNA *resolved_prop;
const bool resolved_path = RNA_path_resolve_property(
mont29 marked this conversation as resolved Outdated
Sybren A. Stüvel commented 2024-04-12 12:09:53 +02:00
Outdated
Review
Copy Link

I think is_resolved (or simply ok) could be a better name than resolved_path, as this variable does not contain "the resolved path".

I think `is_resolved` (or simply `ok`) could be a better name than `resolved_path`, as this variable does not contain "the resolved path".
👍 1
ptr, path.c_str(), &resolved_ptr, &resolved_prop);
if (!resolved_path) {
continue;
}
if (is_keyable_type(prop, resolved_prop)) {
paths.append(path);
}
}
}
}

5
source/blender/makesrna/RNA_access.hh
View File

@ -288,6 +288,11 @@ int RNA_property_enum_bitflag_identifiers(
StructRNA *RNA_property_pointer_type(PointerRNA *ptr, PropertyRNA *prop);
bool RNA_property_pointer_poll(PointerRNA *ptr, PropertyRNA *prop, PointerRNA *value);
/**
* A property is a runtime property if the PROP_INTERN_RUNTIME flag is set on it.
* */
mont29 marked this conversation as resolved Outdated
Hans Goudey commented 2024-04-04 18:19:04 +02:00
Outdated
Review
Copy Link

* */ -> */

`* */` -> `*/`
👍 1
bool RNA_property_is_runtime(const PropertyRNA *prop);
mont29 marked this conversation as resolved
Bastien Montagne commented 2024-04-12 11:33:29 +02:00
Review
Copy Link

This should also be called by rna_Property_is_runtime_get then, to avoid code duplication.

This should also be called by `rna_Property_is_runtime_get` then, to avoid code duplication.
bool RNA_property_editable(const PointerRNA *ptr, PropertyRNA *prop);
/**
* Version of #RNA_property_editable that tries to return additional info in \a r_info

5
source/blender/makesrna/intern/rna_access.cc
View File

@ -2162,6 +2162,11 @@ static bool rna_property_editable_do(const PointerRNA *ptr,
return true;
}
bool RNA_property_is_runtime(const PropertyRNA *prop)
{
return prop->flag_internal & PROP_INTERN_RUNTIME;
}
bool RNA_property_editable(const PointerRNA *ptr, PropertyRNA *prop)
{
return rna_property_editable_do(ptr, prop, -1, nullptr);