redhat-developer / redhat-developer/vscode-java
Peek/Go To Implementations points to method with same name
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- TypeScript
- Estrellas
- 2.3k
- Forks
- 546
- Merge medio
- 20 h 1 min
- PR fusionados (30 d)
- 11
Descripción
When I try to navigate to implementations of a method it will point me to a method with same name in wrapper class, but that wrapper class does not implement the interface of that method.
Environment
- Operating System: Windows 11
- JDK version: 21
- Visual Studio Code version: 1.94.2
- Java extension version: 1.35.1
Steps To Reproduce
- Clone https://github.com/eclipse-jdtls/eclipse.jdt.ls
- Go to JavaClientConnection.java
- Navigate to sendActionableNotification of the interface JavaLanguageClient line 59
- Try to Go To Implementations, it will point to method in wrapper class JavaClientConnection line 159, however the JavaClientConnection does not implement JavaLanguageClient
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza por JavaClientConnection.java y la interfaz JavaLanguageClient, centrándote en sendActionableNotification y en el comportamiento de Go To Implementations descrito en la reproducción. Compara el resultado informado con la relación real entre la interfaz y la implementación; se considera terminado cuando la navegación ya no selecciona el método con el mismo nombre en JavaClientConnection cuando este no implementa JavaLanguageClient.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- java, typescript, vscode
- Área
- developer-experience, tooling
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100