mapstruct / mapstruct/mapstruct-idea
Find usages and renaming for constructor fields
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Java
- Sterne
- 168
- Forks
- 41
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
I've been trying to make this work. However, there are some problems with the custom ConstructorParametersReferencesSearcher that I've implemented. Find usages works properly. However, that causes some other problems:
- When I rename the constructor parameter in the constructor method then my element in the annotation is not renamed. This is tested by
MapstructMethodUsagesSearcherTest.testRenameConstructorTargetReferenceMethod - When I rename the element in the annotation and the class is in the same file as my annotation there is an error happening. This is tested in
RenameHandlerTest.testRenameConstructorTargetParameter. When I run the plugin in an IDE and the class is not in the same file as my annotation then the renaming is correct.
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
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 mit dem benutzerdefinierten ConstructorParametersReferencesSearcher und den beiden genannten Tests: MapstructMethodUsagesSearcherTest.testRenameConstructorTargetReferenceMethod und RenameHandlerTest.testRenameConstructorTargetParameter. Reproduzieren Sie beide Szenarien zur Umbenennung von Konstruktorparametern und untersuchen Sie deren Fehlschläge. Die Aufgabe ist erledigt, wenn beide Tests bestehen, einschließlich der Umbenennung eines Annotationselements, wenn sich die Klasse in derselben Datei befindet, ohne einen Fehler.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- java
- Bereich
- devtools
- Issue-Typ
- Bug
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100