This should make node-lookups faster. Previously, every page would be iterated to look for a node object, but as node-ids can no longer be duplicated/re-used across pages, we can just create a Map to look them up directly. This also has the advantage of running getBaseComponentId() 0 or 1 times, instead of once-per-page for pages that don't have the node in them.
I also removed some unused code in node traversal.
Related Issue(s)
2699
2688
Verification/QA
Manual functionality testing
[ ] I have tested these changes manually
[ ] Creator of the original issue (or service owner) has been contacted for manual testing (or will be contacted when released in alpha)
[x] No testing done/necessary
Automated tests
[ ] Unit test(s) have been added/updated
[ ] Cypress E2E test(s) have been added/updated
[x] No automatic tests are needed here (no functional changes/additions)
Description
This should make node-lookups faster. Previously, every page would be iterated to look for a node object, but as node-ids can no longer be duplicated/re-used across pages, we can just create a Map to look them up directly. This also has the advantage of running
getBaseComponentId()
0 or 1 times, instead of once-per-page for pages that don't have the node in them.I also removed some unused code in node traversal.
Related Issue(s)
2699
2688
Verification/QA
kind/*
label to this PR for proper release notes grouping