Skip to content

Commit 80e84c0

Browse files
authored
Fixed problem in StdDelegatingSerializer#serializeWithType where final serializer lookup was done on the pre-converted value when _delegateSerializer was null. (#4749)
1 parent 03de9a1 commit 80e84c0

File tree

3 files changed

+8
-1
lines changed

3 files changed

+8
-1
lines changed

release-notes/CREDITS-2.x

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1840,3 +1840,7 @@ Rikkarth (rikkarth@github)
18401840
Maxim Valeev (@MaximValeev)
18411841
* Reported #4508: Deserialized JsonAnySetter field in Kotlin data class is null
18421842
(2.18.1)
1843+
1844+
wrongwrong (@k163377)
1845+
* Contributed #4749: Fixed problem in StdDelegatingSerializer#serializeWithType where final serializer lookup was done
1846+
on the pre-converted value when _delegateSerializer was null

release-notes/VERSION-2.x

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,9 @@ Project: jackson-databind
66

77
2.18.1 (WIP-2024)
88

9+
#4749: Fixed a problem with `StdDelegatingSerializer#serializeWithType` looking up the serializer
10+
with the wrong argument
11+
(fix by wrongwrong)
912
#4508: Deserialized JsonAnySetter field in Kotlin data class is null
1013
(reported by @MaximValeev)
1114
(fix by Joo-Hyuk K)

src/main/java/com/fasterxml/jackson/databind/ser/std/StdDelegatingSerializer.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -181,7 +181,7 @@ public void serializeWithType(Object value, JsonGenerator gen, SerializerProvide
181181
}
182182
JsonSerializer<Object> ser = _delegateSerializer;
183183
if (ser == null) {
184-
ser = _findSerializer(value, provider);
184+
ser = _findSerializer(delegateValue, provider);
185185
}
186186
ser.serializeWithType(delegateValue, gen, provider, typeSer);
187187
}

0 commit comments

Comments
 (0)