diff options
author | Jacques Lucke <jacques@blender.org> | 2022-02-09 17:50:03 +0300 |
---|---|---|
committer | Jacques Lucke <jacques@blender.org> | 2022-02-09 17:50:03 +0300 |
commit | 7313a84c5acb401817f1c44cfd8bf22428f34424 (patch) | |
tree | 0c7037cd8a953357cd613aaeefeecbb386e1d753 /source/blender/modifiers/intern | |
parent | d74bb7be1916744ae56347b49333eac22ebb7339 (diff) |
Fix T95612: only overwrite existing attributes with matching domain and type
Also fixes T95611 and T95610.
Differential Revision: https://developer.blender.org/D14051
Diffstat (limited to 'source/blender/modifiers/intern')
-rw-r--r-- | source/blender/modifiers/intern/MOD_nodes.cc | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/source/blender/modifiers/intern/MOD_nodes.cc b/source/blender/modifiers/intern/MOD_nodes.cc index 7e65774b86e..c5ee4b9fb52 100644 --- a/source/blender/modifiers/intern/MOD_nodes.cc +++ b/source/blender/modifiers/intern/MOD_nodes.cc @@ -996,7 +996,11 @@ static void store_computed_output_attributes( { for (const OutputAttributeToStore &store : attributes_to_store) { GeometryComponent &component = geometry.get_component_for_write(store.component_type); - if (component.attribute_exists(store.name)) { + const CustomDataType data_type = blender::bke::cpp_type_to_custom_data_type(store.data.type()); + const std::optional<AttributeMetaData> meta_data = component.attribute_get_meta_data( + store.name); + if (meta_data.has_value() && meta_data->domain == store.domain && + meta_data->data_type == data_type) { /* Copy the data into an existing attribute. */ blender::bke::WriteAttributeLookup write_attribute = component.attribute_try_get_for_write( store.name); @@ -1010,6 +1014,10 @@ static void store_computed_output_attributes( MEM_freeN(store.data.data()); } else { + /* Replace the existing attribute with the new data. */ + if (meta_data.has_value()) { + component.attribute_try_delete(store.name); + } component.attribute_try_create(store.name, store.domain, blender::bke::cpp_type_to_custom_data_type(store.data.type()), |