redhat-developer / redhat-developer/vscode-java
"Find / Peek All references" on a base class doesn't find overwritten invocations
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 2.3k
- Forks
- 546
- Ø Merge
- 20 Std. 1 Min.
- Gemergte PRs (30 T.)
- 11
Beschreibung
When I search for all invocations / references on a method, I would expect that also the invocations will be found if the method was overwritten by another class. These are correct references on this method. But these references are missing in the results, which can lead to missing circumstances.
The other direction, if I search for references for the method on the lower class which overwrites, I see also only the direct invocations, which is correct in this case in my impression (but should also include all invocations of overwrites of this class of course).
The only workaround I found is the call hierarchy, but it is hard to pick all the references out of this representation.
I am not sure if it is a bug, so I marked it as feature request.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
In der Issue sind keine Dateien oder Tests angegeben. Beginne damit, die Referenzsuche in VS Code Java mit einer Basismethode und einer überschreibenden Methode zu reproduzieren, und untersuche anschließend die Einstiegspunkte für die Referenzsuche und die Aufrufhierarchie. Als abgeschlossen gilt die Suche, wenn die Suche nach der Basismethode Aufrufe von Überschreibungen einschließt, während die Suche nach der Methode der Unterklasse ihr direktes Referenzverhalten beibehält.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- java, typescript
- Bereich
- developer-experience, search
- Issue-Typ
- Feature
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 38/100