diff options
author | Sybren A. Stüvel <sybren@blender.org> | 2020-06-25 15:30:45 +0300 |
---|---|---|
committer | Sybren A. Stüvel <sybren@blender.org> | 2020-06-25 15:33:56 +0300 |
commit | baa0da3e69a1225cd18c075be5563c7d811b5347 (patch) | |
tree | 7415ab41b349faa6534ebfa8cb72435ca9273438 /source/blender/depsgraph/intern/builder/deg_builder_nodes.cc | |
parent | 890336849071dd716bc254edbbfa474ed9e61432 (diff) |
Fix T78071: Drivers reading object visibility not updating automatically
An object can be targeted by a driver that reads its `hide_viewport` or
`hide_render` property. The existence of such a driver will create a
relation between the 'sync base flags' depsgrpah node, and the datablock
containing the driver. When the object is hidden, however, it has no
base, and thus it had no 'sync base flags' depsgraph node. To support
such a driver, that depsgraph node is now always added, but for hidden
objects it will just be a no-op. If the node is not used by anything, it
will be automatically disconnected and have a negligible effect on
performance.
Diffstat (limited to 'source/blender/depsgraph/intern/builder/deg_builder_nodes.cc')
-rw-r--r-- | source/blender/depsgraph/intern/builder/deg_builder_nodes.cc | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/source/blender/depsgraph/intern/builder/deg_builder_nodes.cc b/source/blender/depsgraph/intern/builder/deg_builder_nodes.cc index d84ec9b1363..46dbe97ce23 100644 --- a/source/blender/depsgraph/intern/builder/deg_builder_nodes.cc +++ b/source/blender/depsgraph/intern/builder/deg_builder_nodes.cc @@ -667,8 +667,15 @@ void DepsgraphNodeBuilder::build_object_flags(int base_index, eDepsNode_LinkedState_Type linked_state) { if (base_index == -1) { + /* An object can be targeted by a driver that reads its `hide_viewport` or `hide_render` + * property. For visible objects, these properties are synced from the base flags, so that + * driver variable creates a relation from the OBJECT_FROM_LAYER component. However, when the + * object is hidden, it has base_index=-1. To support the aforementioned driver, the component + * should still exist, even if it's a no-op. See T78071. */ + add_operation_node(&object->id, NodeType::OBJECT_FROM_LAYER, OperationCode::OBJECT_BASE_FLAGS); return; } + Scene *scene_cow = get_cow_datablock(scene_); Object *object_cow = get_cow_datablock(object); const bool is_from_set = (linked_state == DEG_ID_LINKED_VIA_SET); |