Page MenuHome

Fix T89829: Wrong active context path check in spreadsheet.
ClosedPublic

Authored by Jacques Lucke (JacquesLucke) on Jul 23 2021, 12:06 PM.

Details

Summary

The problem was that the modifier was reevaluated all the time, even between showing the attribute search and clicking on the attribute name. This freed the data referenced by attribute search. The real bug here was that the dependency graph was tagged for update even though nothing changed. This was because the spreadsheet thought its active context has changed and it wanted to compute the new value to be shown in the spreadsheet.

The reason for the bug was that I confused how the tree-path of a node editor works. The second element in the tree path contains the name of the group node in the root tree that we're in (instead of the first element).

Diff Detail

Repository
rB Blender

Event Timeline

Jacques Lucke (JacquesLucke) requested review of this revision.Jul 23 2021, 12:06 PM
Jacques Lucke (JacquesLucke) created this revision.

Makes sense. At some point it would be nice to document the treepath a bit. The current comment in DNA is quite confusing, and incorrect, as far as I can tell.

This revision is now accepted and ready to land.Jul 23 2021, 2:54 PM