mapstruct / mapstruct/mapstruct-idea

Wrong usage of mappers factory should be able to remove static initializer

Offen
#183 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
Java
Sterne
168
Forks
41
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

Why reviewing PR #179 @hduelme pointed out that the Wrong usage of mappers factory does not work (i.e. it does not remove it) with the following code:

```java
@Mapper(componentModel = "spring")
abstract class WrongComponentModelMapper {

static {
Mappers.getMapper(WrongComponentModelMapper.class);
}

Target map(Target source);
}
```

or
```java
@Mapper(config = MyConfig.class, componentModel = "spring")
interface WrongComponentModelMapper {

WrongComponentModelMapper INSTANCE = () -> {
return Mappers.getMapper(WrongComponentModelMapper.class);
};

Target map(Target source);
}
```

We need to see if we can support this.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne damit, die Beispiele für statische Initialisierer und Schnittstellenfelder aus dem Issue zu reproduzieren und nachzuverfolgen, wie das IntelliJ plugin derzeit die Verwendung von Mappers.getMapper behandelt. Sieh dir zur Einordnung die Diskussion rund um PR #179 an. Als abgeschlossen gilt die Aufgabe, wenn festgestellt wurde, ob diese Muster unterstützt werden können, und, falls ja, das Verhalten mit Abdeckung für beide Beispiele validiert wurde.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
java
Bereich
tooling
Issue-Typ
Feature
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.