Skip to content

Commit 271281d

Browse files
committed
Complete the fix, testing impovements
1 parent a6d3a52 commit 271281d

File tree

2 files changed

+8
-3
lines changed

2 files changed

+8
-3
lines changed

src/main/java/tools/jackson/databind/ValueSerializer.java

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
import java.util.Set;
55

66
import com.fasterxml.jackson.annotation.JsonFormat;
7+
import com.fasterxml.jackson.annotation.JsonTypeInfo.As;
78

89
import tools.jackson.core.*;
910
import tools.jackson.databind.jsonFormatVisitors.JsonFormatVisitable;
@@ -252,6 +253,13 @@ public void serializeWithType(T value, JsonGenerator gen, SerializationContext c
252253
TypeSerializer typeSer)
253254
throws JacksonException
254255
{
256+
// 07-Dec-2025, tatu: [databind#1654] Check for "no-op" type serializer
257+
// indirectly
258+
if (typeSer.getTypeInclusion() == As.NOTHING) {
259+
serialize(value, gen, ctxt);
260+
return;
261+
}
262+
255263
Class<?> clz = handledType();
256264
if (clz == null) {
257265
clz = value.getClass();

src/main/java/tools/jackson/databind/ser/jdk/CollectionSerializer.java

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111
import tools.jackson.databind.SerializationContext;
1212
import tools.jackson.databind.ValueSerializer;
1313
import tools.jackson.databind.jsontype.TypeSerializer;
14-
import tools.jackson.databind.jsontype.impl.NoOpTypeSerializer;
1514
import tools.jackson.databind.ser.impl.PropertySerializerMap;
1615
import tools.jackson.databind.ser.std.AsArraySerializerBase;
1716
import tools.jackson.databind.ser.std.StdContainerSerializer;
@@ -172,8 +171,6 @@ public void serializeContentsUsing(Collection<?> value, JsonGenerator g, Seriali
172171
} else {
173172
if (typeSer == null) {
174173
ser.serialize(elem, g, provider);
175-
} else if (typeSer instanceof NoOpTypeSerializer) {
176-
ser.serialize(elem, g, provider);
177174
} else {
178175
ser.serializeWithType(elem, g, provider, typeSer);
179176
}

0 commit comments

Comments
 (0)