@Condition applied on wrong source parameter while mapping

Offen

#4,037 erstellt am 16.4.2026 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 3/5
Geschätzter Aufwand: 1-2 Tage
Anfängerfreundlichkeit: 52/100

Issue-Typ: Bug
Klarheit: Größtenteils klar
Aktivitätsstatus: Ruhig
Tech-Stack: java
Bereich: devtools

Rechercherichtung

Beginne mit der im Bericht verlinkten issue-4037-Reproduktion und untersuche den generierten Mapper für das Mapping mit zwei Quellparametern. Verfolge, wie @Condition ausgewählt wird, wenn nullValueCheckStrategy = NullValueCheckStrategy.ALWAYS gilt; als erledigt gilt die Aufgabe, wenn die Bedingung die aus requestBean abgeleitete Quelle für browserId statt slotId verwendet und ein Regressionstest diesen Fall abdeckt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

bug

Expected behavior

@Condition method should be correctly applied on correct source parameters when we are using nullValueCheckStrategy = NullValueCheckStrategy.ALWAYS

Actual behavior

When we have the mapping mapping with 2 source parameters, for eg:

@Mapping(target = "browserId", source = "requestBean.deviceInfo.browserId")
public abstract AudienceProfileRequest map(RequestBean requestBean, String slotId);

If we are mapping some Integer source field from RequestBean to AudienceProfileRequest then the condition is applied on slotId which is not expected. Following is the snippet of the generated code:

Integer browserId = requestBeanDeviceInfoBrowserId( requestBean );
if ( prechecksSupport.isNotBlank( slotId ) ) {
    audienceProfileRequest.browserId( browserId );
}

Steps to reproduce the problem

https://github.com/altaiezior/mapstruct-issues/tree/master/issue-4037

MapStruct Version

MapStruct 1.5.5

Vorherrschende Sprache: Java
Sterne: 7.7k
Forks: 1.1k
PR-Merge-Kennzahlen: Keine gemergten PRs in 30 T.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus mapstruct/mapstruct

ProcessorTests are currently unable to reload Mapper implementation class Offen

bug test

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100

mapstruct/mapstruct#4132 · Erstellt am 19.9.2026
NullValuePropertyMappingStrategy.IGNORE strategy ignored for Optional fields Offen

bug

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 68/100

mapstruct/mapstruct#4128 · Erstellt am 16.9.2026
Optional source property that is itself null throws NullPointerException on the generated isPresent() check; NullValueCheckStrategy.ALWAYS is ignored Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 68/100

mapstruct/mapstruct#4123 · 1 Kommentar · Erstellt am 09.9.2026
Collections no longer copied if marked non-null Offen

bug JSpecify

mapstruct/mapstruct#4118 · 1 zugewiesene Person · Erstellt am 25.8.2026
Optional<Map> / Optional<Collection> source property generates unguarded Optional#get, throwing NoSuchElementException when empty Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 72/100

mapstruct/mapstruct#4111 · 1 Kommentar · Erstellt am 14.8.2026

Alle Issues in mapstruct/mapstruct