-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
fix: set FieldPath for SequenceNode elements
The FieldPath was not being set for nodes underneath a SequenceNode during fieldspec's traversal. This is in part because handleSequence uses VisitElements in contrast to a PathGetter as is done by handleMap. The accuracy of FieldPath is more relevant now with the recently added support of mutation trackers in filtersutil. This change fixes the case where a mutation tracker callback is called on a SequenceNode element and the node does not have an accurate FieldPath value.
- Loading branch information
Showing
2 changed files
with
84 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters