eclipse-platform / eclipse-platform/eclipse.platform
[prefs] Better define behaviour of #nodeExists for project nodes
- Vorherrschende Sprache
- Java
- Sterne
- 165
- Forks
- 174
- Ø Merge
- 2 T. 8 Std.
- Gemergte PRs (30 T.)
- 22
Beschreibung
This is a migration of [Bug 90500](https://bugs.eclipse.org/bugs/show_bug.cgi?id=90500):
> DJ Houghton CLA Friend 2005-04-06 13:51:36 EDT
>
> We need to better define the behaviour of #nodeExists for the nodes in the
> project scope.
>
> Currently we over-ride super-class impl such that the node does exist if the
> super-class says that it does or if you are at /project and you are asking about
> a single segment name and that name is the name of a project that exists in the
> workspace.
>
> The impl doesn't work for long paths though since super accesses the cache of
> children. (e.g. nodeExists("myProject/qualifier/nodeName")
>
> Investigate the interaction between #nodeExists and #getChild, #getChildren, and
> #childnames.
which is still referenced at
https://github.com/eclipse-platform/eclipse.platform.resources/blob/88f58177bf0ad15355d3d1136c5d8b4aac7be751/bundles/org.eclipse.core.resources/src/org/eclipse/core/internal/resources/CharsetManager.java#L363-L384
we either should fix this or remove the reference from the code
Beitragsleitfaden
Rechercherichtung
Beginnen Sie mit CharsetManager.java bei den referenzierten Zeilen und verfolgen Sie die Implementierung des Präferenzbereichs von nodeExists, getChild, getChildren und childnames. Reproduzieren Sie den Fall mit dem langen Pfad wie nodeExists("myProject/qualifier/nodeName") und bestimmen Sie anschließend, ob das Verhalten von nodeExists korrigiert oder der Verweis auf CharsetManager entfernt werden sollte; abgeschlossen bedeutet, dass das Verhalten des Projektknotens ausdrücklich definiert und durch die relevanten Ressourcentests abgedeckt ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- java
- Bereich
- tooling
- Issue-Typ
- Bug
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 28/100