Support `@JsonAnySetter` on setter method (single `Map` parameter)

[cowtowncoder]

Discussed in #4888

^{Originally posted by chrylis January 7, 2025}
Currently, @JsonAnySetter can be used on a physical field of type Map, but unlike @JsonProperty, it will be silently dropped if used on a method of arity other than two (POJOPropertiesCollector#_addMethods). I would like to be able to use @JsonAnySetter on a method representing a virtual property, like this:

@Slf4j
class WebhookDto {
  @JsonAnySetter
  void setUnmappedProperties(Map<String, Object> unmapped) {
    log.warn("unmapped properties on webhook: {}", unmapped)
  }
}
```</div>

[alixpeigue]

Hi, is it still available ? I'd like to tackle it as part of a software engineering course, we'll be multiple people working on it but I'll serve as intermediary not to burden you with that.

[cowtowncoder]

@JooHyukKim What do you think about this wrt feasibility/need? You have been working most on any-getter/any-setter support.

@alixpeigue Yes, should be free: if someone was working on it, they'd be expected to add a note here.

[JooHyukKim]

@cowtowncoder
Hmmm this usage seems more like workaround, honestly. Adding custom handling for un-expected property isn't what @JsonAnySetter is for I think?

@alixpeigue
Speaking of "need" along with "priority", we have couple of other JsonAnySetter related issues, if you trying to work on somethiing.

[cowtowncoder]

@JooHyukKim I think use of @JsonAnySetter for properties for which there is no Field or Setter (hence, "unmapped") is pretty much the main use case?

[JooHyukKim]

@JooHyukKim I think use of @JsonAnySetter for properties for which there is no Field or Setter (hence, "unmapped") is pretty much the main use case?

Yeah, we can see it that way makes sense.
My idea was we might be better off creating hook to trigger when facing unknown properties. That way we dont even need JsonAnySetter or limit our selves to a builder like

JsonMapper.builder().onUnknownProperty(logger::error).build()

[cowtowncoder]

My idea was we might be better off creating hook to trigger when facing unknown properties. That way we dont even need JsonAnySetter or limit our selves to a builder like

There is DeserializationProblemHandler to register for doing this.

[goutamadwant]

@cowtowncoder added a fix for this issue in #6075. Let me know if you have any suggestions. thanks!