You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When multiple inheritance is used, it is not sufficient that the node id just contains the URI of the resource. VS Code will throw an error when trying to insert a node with the same id multiple times in the tree.
The problem is a risk for the user experience and to prevent regressions in future versions there should be a unit test for this: the parent id should always be included in the node id.
As a side effect that would make the implementation of the the getParent method much easier as the URI is already encoded in the id.
The text was updated successfully, but these errors were encountered:
When multiple inheritance is used, it is not sufficient that the node id just contains the URI of the resource. VS Code will throw an error when trying to insert a node with the same id multiple times in the tree.
The problem is a risk for the user experience and to prevent regressions in future versions there should be a unit test for this: the parent id should always be included in the node id.
As a side effect that would make the implementation of the the getParent method much easier as the URI is already encoded in the id.
The text was updated successfully, but these errors were encountered: