JsonConverter for struct not used for nullable fields when passed through JsonSerializerSettings

[Whathecode]

JsonConverter for struct not used for nullable fields when passed through JsonSerializerSettings

[Whathecode]

Related to #2245.

@elgonzo

Yes and no. (I actually read this issue before posting.)

Yes, in the responses the OP seems to basically suggest the desired behavior I outline here. The behavior which works, but only in case the converter is registered through an attribute. So maybe this has been implemented since, or the OP was unaware.

No, in that the OP started from the outset with a broader (non-generic) converter, and incorrectly implemented CanConvert.

The discrepancy in behavior between the two ways of registering is what I primarily want to highlight here. Documentation-wise (or the limited source code I browsed), I see no indication this is intentional. But, it strikes me as extremely odd (and even unlikely) such an old established, widely used, library would contain a bug in such a core feature.

Out-of-the-box support for things like this is why serialization frameworks exist. 🤔 So I still feel I may be overlooking something.

[elgonzo]

Related to #2245. As commented on in that issue, the workaround for this limitation is using the non-generic JsonConverter base class and overriding the CanConvert method in your converter and compare the provided type object against both Test and Nullable<Test>. If you need to do this with several nullable value types, it is probably advisable to create an intermediate generic abstract class inheriting from JsonConverter class which would then serve as generic base class for the various nullable-capable json converters for desired value types.

Also keep in mind that WriteJson method of your converter would need to deal with Nullable<T> instances representing null in whatever way is desired, with the ReadJson method likely also needing to deal with the json input possibly containing null values instead of data for a Test instance.

(P.S. I am just a user and not associated with the Newtonsoft.Json project)