Allow @JsonAnySetter on Creator constructors

src/main/java/com/fasterxml/jackson/databind/deser/BasicDeserializerFactory.java

@@ -879,8 +879,14 @@ protected void _addExplicitPropertyCreator(DeserializationContext ctxt,
                     */
                 }
                 name = candidate.findImplicitParamName(i);
-                _validateNamedPropertyParameter(ctxt, beanDesc, candidate, i,
+                // [databind#562] Any setter can be used...
+                Boolean hasAnySetter = ctxt.getAnnotationIntrospector().hasAnySetter(param);
+                if (hasAnySetter != null && hasAnySetter) {
+                    // no-op
+                } else {
+                    _validateNamedPropertyParameter(ctxt, beanDesc, candidate, i,
                         name, injectId);
+                }
             }
             properties[i] = constructCreatorProperty(ctxt, beanDesc, name, i, param, injectId);
         }

src/main/java/com/fasterxml/jackson/databind/deser/BeanDeserializer.java

@@ -497,7 +497,7 @@ protected Object _deserializeUsingPropertyBased(final JsonParser p, final Deseri
             // "any property"?
             if (_anySetter != null) {
                 try {
-                    buffer.bufferAnyProperty(_anySetter, propName, _anySetter.deserialize(p, ctxt));
+                    buffer.bufferMapProperty(propName, _anySetter.deserialize(p, ctxt));
                 } catch (Exception e) {
                     wrapAndThrow(e, _beanType.getRawClass(), propName, ctxt);
                 }
@@ -523,10 +523,15 @@ protected Object _deserializeUsingPropertyBased(final JsonParser p, final Deseri
         // We hit END_OBJECT, so:
         Object bean;
         try {
-            bean = creator.build(ctxt, buffer);
+            if (_anySetter != null) {
+                bean = creator.buildSimple(ctxt, buffer);
+            } else {
+                bean = creator.build(ctxt, buffer);
+            }
         } catch (Exception e) {
             return wrapInstantiationProblem(e, ctxt);
         }
+
         // 13-Apr-2020, tatu: [databind#2678] need to handle injection here
         if (_injectables != null) {
             injectValues(ctxt, bean);

src/main/java/com/fasterxml/jackson/databind/deser/BeanDeserializerFactory.java

@@ -544,8 +544,11 @@ protected void addBeanProps(DeserializationContext ctxt,
 
         // Also, do we have a fallback "any" setter?
         AnnotatedMember anySetter = beanDesc.findAnySetterAccessor();
+        AnnotatedMember creatorPropWithAnySetter = _findCreatorPropWithAnySetter(ctxt, creatorProps);
         if (anySetter != null) {
             builder.setAnySetter(constructAnySetter(ctxt, beanDesc, anySetter));
+        } else if (creatorPropWithAnySetter != null) {
+            builder.setAnySetter(constructAnySetter(ctxt, beanDesc, creatorPropWithAnySetter));
         } else {
             // 23-Jan-2018, tatu: although [databind#1805] would suggest we should block
             //   properties regardless, for now only consider unless there's any setter...
@@ -661,6 +664,23 @@ protected void addBeanProps(DeserializationContext ctxt,
         }
     }
 
+    private AnnotatedMember _findCreatorPropWithAnySetter(DeserializationContext ctxt, SettableBeanProperty[] creatorProps) {
+        if (creatorProps != null) {
+            AnnotationIntrospector ai = ctxt.getAnnotationIntrospector();
+            for (SettableBeanProperty prop : creatorProps) {
+                AnnotatedMember m = prop.getMember();
+                if (m != null) {
+                    Boolean hasAnySetter = ai.hasAnySetter(m);
+                    if (hasAnySetter != null && hasAnySetter) {
+                        return prop.getMember();
+                    }
+                }
+
+            }
+        }
+        return null;
+    }
+
     private boolean _isSetterlessType(Class<?> rawType) {
         // May also need to consider getters
         // for Map/Collection properties; but with lowest precedence
@@ -819,6 +839,22 @@ protected SettableAnyProperty constructAnySetter(DeserializationContext ctxt,
                     valueType, null, mutator,
                     PropertyMetadata.STD_OPTIONAL);
 
+        } else if (mutator instanceof AnnotatedParameter){
+            AnnotatedParameter af = (AnnotatedParameter) mutator;
+            // get the type from the content type of the map object
+            JavaType fieldType = af.getType();
+            // 31-Jul-2022, tatu: Not just Maps any more but also JsonNode, so:
+            if (fieldType.isMapLikeType()) {
+                fieldType = resolveMemberAndTypeAnnotations(ctxt, mutator, fieldType);
+                keyType = fieldType.getKeyType();
+                valueType = fieldType.getContentType();
+                prop = new BeanProperty.Std(PropertyName.construct("stuff"),
+                    fieldType, null, mutator, PropertyMetadata.STD_OPTIONAL);
+            } else {
+                return ctxt.reportBadDefinition(beanDesc.getType(), String.format(
+                    "Unsupported type for any-setter: %s -- only support `Map`s, `JsonNode` and `ObjectNode` ",
+                    ClassUtil.getTypeDescription(fieldType)));
+            }
         } else if (isField) {
             AnnotatedField af = (AnnotatedField) mutator;
             // get the type from the content type of the map object

src/main/java/com/fasterxml/jackson/databind/deser/impl/PropertyBasedCreator.java

@@ -197,6 +197,12 @@ public PropertyValueBuffer startBuilding(JsonParser p, DeserializationContext ct
         return new PropertyValueBuffer(p, ctxt, _propertyCount, oir);
     }
 
+    public Object buildSimple(DeserializationContext ctxt, PropertyValueBuffer buffer) throws IOException
+    {
+        return  _valueInstantiator.createFromObjectWith(ctxt,
+            _allProperties, buffer);
+    }
+
     public Object build(DeserializationContext ctxt, PropertyValueBuffer buffer) throws IOException
     {
         Object bean = _valueInstantiator.createFromObjectWith(ctxt,

src/main/java/com/fasterxml/jackson/databind/deser/impl/PropertyValueBuffer.java

@@ -2,12 +2,15 @@
 
 import java.io.IOException;
 import java.util.BitSet;
+import java.util.HashMap;
+import java.util.Map;
 
 import com.fasterxml.jackson.core.JsonParser;
 
 import com.fasterxml.jackson.databind.*;
 import com.fasterxml.jackson.databind.deser.SettableAnyProperty;
 import com.fasterxml.jackson.databind.deser.SettableBeanProperty;
+import com.fasterxml.jackson.databind.deser.UnresolvedForwardReference;
 import com.fasterxml.jackson.databind.introspect.AnnotatedMember;
 
 /**
@@ -176,6 +179,34 @@ public Object[] getParameters(SettableBeanProperty[] props)
                 }
             }
         }
+
+        // [databind#562]: Respect @JsonAnySetter in @JsonCreator
+        // check if we have anySetter Param
+        AnnotationIntrospector ai = _context.getAnnotationIntrospector();
+        for (int i = 0; i < props.length; i++) {
+            SettableBeanProperty prop = props[i];
+            AnnotatedMember member = prop.getMember();
+            if (member == null) {
+                continue;
+            }
+            Boolean hasAnySetter = ai.hasAnySetter(member);
+            if (hasAnySetter == null || !hasAnySetter) {
+                continue;
+            }
+            // So we have prop with anySetter. Should be Map-like, so let's assign such?
+            // Assign all remaining values to the map
+            Map<String, Object> param = new HashMap<>();
+            for (PropertyValue next = buffered(); next != null; next = next.next) {
+                try {
+                    next.assign(param);
+                } catch (IOException e) {
+                    _context.reportInputMismatch(prop, e.getMessage());
+                }
+            }
+            // assign it then return
+            _creatorParameters[i] = param;
+            break;
+        }
         return _creatorParameters;
     }
 

src/test/java/com/fasterxml/jackson/failing/AnySetterForCreator562Test.java

@@ -1,6 +1,6 @@
 package com.fasterxml.jackson.failing;
 
-import java.util.Collections;
+import java.util.HashMap;
 import java.util.Map;
 
 import org.junit.jupiter.api.Test;
@@ -36,12 +36,16 @@ public POJO562(@JsonProperty("a") String a,
     @Test
     public void testAnySetterViaCreator562() throws Exception
     {
+        Map<String, Object> expected = new HashMap<>();
+        expected.put("b", Integer.valueOf(42));
+        expected.put("c", Integer.valueOf(111));
+
         POJO562 pojo = MAPPER.readValue(a2q(
-                "{'a':'value', 'b':42}"
+                "{'a':'value', 'b':42, 'c': 111}"
                 ),
                 POJO562.class);
-        assertEquals(pojo.a, "value");
-        assertEquals(Collections.singletonMap("b", Integer.valueOf(42)),
-                pojo.stuff);
+
+        assertEquals("value", pojo.a);
+        assertEquals(expected, pojo.stuff);
     }
 }