[oryan-block]

Description

Fixes #932

Follow-up to #931. A custom scalar class without visible properties still lost its value when Jackson wrote a type id or an object id for it:

@JsonTypeInfo(use = JsonTypeInfo.Id.CLASS, include = JsonTypeInfo.As.PROPERTY)
class TypedToken(private val value: String = "default")

@JsonIdentityInfo(generator = ObjectIdGenerators.IntSequenceGenerator::class)
class IdToken(private val value: String = "default")

These are written as {"@class": "..."} and {"@id": 1}, so the empty bean check from #931 didn't fire (it only looked at {}), and an Optional parameter or an input object field silently got TypedToken("default") or IdToken("default").

Now the check fires for any bean serializer without properties, whatever ids it wrote. So these scalars go down the embedded value path like the ones in #931. In an ObjectNode argument that gives a POJONode with the value. Everywhere else it fails, since Jackson's polymorphic and object id deserializers can't read an embedded value (missing type id property '@class', Cannot deserialize value of type Integer). I think an error is better than a silent default here.

I did have them working (native type ids on the embedded value, and binding the value to a generated object id), but it was a fair bit of Jackson plumbing with its own gaps, for scalars I don't think anyone has. So I dropped it. If someone runs into it, removing the annotation or adding a getter for the value works.

Release note: custom scalars without visible properties that Jackson writes with a type id (@JsonTypeInfo) or an object id (@JsonIdentityInfo) now fail with an error instead of reaching the resolver as a default instance. In ObjectNode arguments they're passed with their value.

Checklist

• Pull requests follows the contribution guide
• New or modified functionality is covered by tests

Both new cases in CustomScalarInputTest fail on master: the Optional, input object and list cases with @JsonTypeInfo / @JsonIdentityInfo return default or error inconsistently, and the ObjectNode case gets {"@id":1} or errors. mvn clean verify passes (432 tests). Also checked through the installed jar in a separate consumer project: on master the Optional parameters return typed default and id default, with this change they fail with the errors above.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud