redhat-developer / redhat-developer/vscode-java
Imports are not included in the document symbols returned from "getDocumentSymbols" API
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
Hi
I am writing a vscode extension and I want to get the symbols of a JAVA file.
Environment
Operating System: windows 10
Visual Studio Code version: 1.50.0
Steps To Reproduce
If I run this sample code :
`const vscodeCDSExtension = extensions.getExtension("redhat.java");
const javaDocParams : DocumentSymbolParams = {
textDocument: {
uri: Uri.file(filePath).toString()
}
};
if (vscodeCDSExtension && vscodeCDSExtension.isActive) {
const symbols = await vscodeCDSExtension.exports.getDocumentSymbols(javaDocParams);
console.log(symbols.length);
}`
Current Result
The "imports" of the file are not included in the returned symbols (only "package", classes and methods are returned)
Expected Result
The "imports" also should be included in the result
The JAVA file is attached
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
Beginnen Sie damit, den getDocumentSymbols API-Aufruf aus dem Issue mit der angehängten Java-Datei CatalogServiceHandler.zip zu reproduzieren. Verfolgen Sie, wie Pakete, Klassen und Methoden zurückgegeben werden, überprüfen Sie anschließend, dass Importdeklarationen im Symbolergebnis enthalten sind, und ergänzen oder aktualisieren Sie die Abdeckung, falls das Repository einen passenden Test bereitstellt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- java, typescript, vscode
- Bereich
- api, developer-experience, tooling
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 38/100