Allow @JsonAnySetter on Creators

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

@@ -508,12 +508,20 @@ private void _addSelectedPropertiesBasedCreator(DeserializationContext ctxt,
     {
         final int paramCount = candidate.paramCount();
         SettableBeanProperty[] properties = new SettableBeanProperty[paramCount];
+        boolean anySetterFound = false;
 
         for (int i = 0; i < paramCount; ++i) {
             JacksonInject.Value injectId = candidate.injection(i);
             AnnotatedParameter param = candidate.parameter(i);
             PropertyName name = candidate.paramName(i);
-            if (name == null) {
+            boolean isAnySetter = Boolean.TRUE.equals(ctxt.getAnnotationIntrospector().hasAnySetter(param));
+            if (isAnySetter) {
+                if (anySetterFound) {
+                    ctxt.reportBadTypeDefinition(beanDesc, "More than one 'any-setter' specified");
+                } else {
+                    anySetterFound = true;
+                }
+            } else if (name == null) {
                 // 21-Sep-2017, tatu: Looks like we want to block accidental use of Unwrapped,
                 //   as that will not work with Creators well at all
                 NameTransformer unwrapper = ctxt.getAnnotationIntrospector().findUnwrappingNameTransformer(param);

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

@@ -415,7 +415,9 @@ protected Object _deserializeUsingPropertyBased(final JsonParser p, final Deseri
         throws IOException
     {
         final PropertyBasedCreator creator = _propertyBasedCreator;
-        PropertyValueBuffer buffer = creator.startBuilding(p, ctxt, _objectIdReader);
+        PropertyValueBuffer buffer = (_anySetter != null)
+            ? creator.startBuildingWithAnySetter(p, ctxt, _objectIdReader, _anySetter)
+            : creator.startBuilding(p, ctxt, _objectIdReader);
         TokenBuffer unknown = null;
         final Class<?> activeView = _needViewProcesing ? ctxt.getActiveView() : null;
 

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

@@ -543,9 +543,9 @@ protected void addBeanProps(DeserializationContext ctxt,
         }
 
         // Also, do we have a fallback "any" setter?
-        AnnotatedMember anySetter = beanDesc.findAnySetterAccessor();
+        SettableAnyProperty anySetter = _resolveAnySetter(ctxt, beanDesc, creatorProps);
         if (anySetter != null) {
-            builder.setAnySetter(constructAnySetter(ctxt, beanDesc, anySetter));
+            builder.setAnySetter(anySetter);
         } 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 +661,29 @@ protected void addBeanProps(DeserializationContext ctxt,
         }
     }
 
+    // since 2.18
+    private SettableAnyProperty _resolveAnySetter(DeserializationContext ctxt,
+              BeanDescription beanDesc, SettableBeanProperty[] creatorProps)
+            throws JsonMappingException
+    {
+        // Find the regular method/field level any-setter
+        AnnotatedMember anySetter = beanDesc.findAnySetterAccessor();
+        if (anySetter != null) {
+            return constructAnySetter(ctxt, beanDesc, anySetter);
+        }
+        // else look for any-setter via @JsonCreator
+        if (creatorProps != null) {
+            for (SettableBeanProperty prop : creatorProps) {
+                AnnotatedMember member = prop.getMember();
+                if (member != null && Boolean.TRUE.equals(ctxt.getAnnotationIntrospector().hasAnySetter(member))) {
+                    return constructAnySetter(ctxt, beanDesc, member);
+                }
+            }
+        }
+        // not found, that's fine, too
+        return null;
+    }
+
     private boolean _isSetterlessType(Class<?> rawType) {
         // May also need to consider getters
         // for Map/Collection properties; but with lowest precedence
@@ -808,6 +831,9 @@ protected SettableAnyProperty constructAnySetter(DeserializationContext ctxt,
         JavaType keyType;
         JavaType valueType;
         final boolean isField = mutator instanceof AnnotatedField;
+        // [databind#562] Allow @JsonAnySetter on Creator constructor
+        final boolean isParameter = mutator instanceof AnnotatedParameter;
+        int parameterIndex = -1;
 
         if (mutator instanceof AnnotatedMethod) {
             // we know it's a 2-arg method, second arg is the value
@@ -848,11 +874,39 @@ protected SettableAnyProperty constructAnySetter(DeserializationContext ctxt,
                         "Unsupported type for any-setter: %s -- only support `Map`s, `JsonNode` and `ObjectNode` ",
                         ClassUtil.getTypeDescription(fieldType)));
             }
+        } else if (isParameter) {
+            AnnotatedParameter af = (AnnotatedParameter) mutator;
+            JavaType fieldType = af.getType();
+            parameterIndex = af.getIndex();
+
+            if (fieldType.hasRawClass(JsonNode.class) || fieldType.hasRawClass(ObjectNode.class)) {
+                fieldType = resolveMemberAndTypeAnnotations(ctxt, mutator, fieldType);
+                // Deserialize is individual values of ObjectNode, not full ObjectNode, so:
+                valueType = ctxt.constructType(JsonNode.class);
+                prop = new BeanProperty.Std(PropertyName.construct(mutator.getName()),
+                        fieldType, null, mutator, PropertyMetadata.STD_OPTIONAL);
+
+                // Unlike with more complicated types, here we do not allow any annotation
+                // overrides etc but instead short-cut handling:
+                return SettableAnyProperty.constructForJsonNodeParameter(ctxt, prop, mutator, valueType,
+                        ctxt.findRootValueDeserializer(valueType), parameterIndex);
+            } else if (fieldType.isMapLikeType()) {
+                fieldType = resolveMemberAndTypeAnnotations(ctxt, mutator, fieldType);
+                keyType = fieldType.getKeyType();
+                valueType = fieldType.getContentType();
+                prop = new BeanProperty.Std(PropertyName.construct(mutator.getName()),
+                        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 {
             return ctxt.reportBadDefinition(beanDesc.getType(), String.format(
                     "Unrecognized mutator type for any-setter: %s",
                     ClassUtil.nameOf(mutator.getClass())));
         }
+
         // First: see if there are explicitly specified
         // and then possible direct deserializer override on accessor
         KeyDeserializer keyDeser = findKeyDeserializerFromAnnotation(ctxt, mutator);
@@ -880,6 +934,10 @@ protected SettableAnyProperty constructAnySetter(DeserializationContext ctxt,
             return SettableAnyProperty.constructForMapField(ctxt,
                     prop, mutator, valueType, keyDeser, deser, typeDeser);
         }
+        if (isParameter) {
+            return SettableAnyProperty.constructForMapParameter(ctxt,
+                    prop, mutator, valueType, keyDeser, deser, typeDeser, parameterIndex);
+        }
         return SettableAnyProperty.constructForMethod(ctxt,
                 prop, mutator, valueType, keyDeser, deser, typeDeser);
     }

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

@@ -1,8 +1,10 @@
 package com.fasterxml.jackson.databind.deser;
 
 import java.io.IOException;
+import java.util.HashMap;
 import java.util.LinkedHashMap;
 import java.util.Map;
+import java.util.Objects;
 
 import com.fasterxml.jackson.core.*;
 import com.fasterxml.jackson.databind.*;
@@ -117,6 +119,31 @@ public static SettableAnyProperty constructForJsonNodeField(DeserializationConte
                 ctxt.getNodeFactory());
     }
 
+    /**
+     * @since 2.18
+     */
+    public static SettableAnyProperty constructForMapParameter(DeserializationContext ctxt,
+            BeanProperty property, AnnotatedMember field, JavaType valueType, KeyDeserializer keyDeser,
+            JsonDeserializer<Object> valueDeser, TypeDeserializer typeDeser, int parameterIndex
+    ) {
+        Class<?> mapType = field.getRawType();
+        // 02-Aug-2022, tatu: Ideally would be resolved to a concrete type by caller but
+        //    alas doesn't appear to happen. Nor does `BasicDeserializerFactory` expose method
+        //    for finding default or explicit mappings.
+        if (mapType == Map.class) {
+            mapType = LinkedHashMap.class;
+        }
+        ValueInstantiator vi = JDKValueInstantiators.findStdValueInstantiator(ctxt.getConfig(), mapType);
+        return new MapParameterAnyProperty(property, field, valueType, keyDeser, valueDeser, typeDeser, vi, parameterIndex);
+    }
+
+    public static SettableAnyProperty constructForJsonNodeParameter(DeserializationContext ctxt, BeanProperty prop,
+            AnnotatedMember mutator, JavaType valueType, JsonDeserializer<Object> valueDeser, int parameterIndex
+    ) {
+        return new JsonNodeParameterAnyProperty(prop, mutator, valueType, valueDeser, ctxt.getNodeFactory(), parameterIndex);
+    }
+
+
     // Abstract @since 2.14
     public abstract SettableAnyProperty withValueDeserializer(JsonDeserializer<Object> deser);
 
@@ -159,6 +186,23 @@ Object readResolve() {
      */
     public String getPropertyName() { return _property.getName(); }
 
+    /**
+     * Accessor for parameterIndex.
+     * @return -1 if not a parameterized setter, otherwise index of parameter
+     *
+     * @since 2.18
+     */
+    public int getParameterIndex() { return -1; }
+
+    /**
+     * Create an instance of value to set any properties.
+     *
+     * @since 2.18
+     */
+    public Object createParameterObject() {
+        throw new UnsupportedOperationException("Cannot call createParameterObject() on " + getClass().getName());
+    }
+
     /*
     /**********************************************************
     /* Public API, deserialization
@@ -437,4 +481,99 @@ public SettableAnyProperty withValueDeserializer(JsonDeserializer<Object> deser)
             return this;
         }
     }
+
+
+    /**
+     * [databind#562] Allow @JsonAnySetter on Creator constructor
+     *
+     * @since 2.18
+     */
+    protected static class MapParameterAnyProperty extends SettableAnyProperty
+        implements java.io.Serializable
+    {
+        private static final long serialVersionUID = 1L;
+
+        protected final ValueInstantiator _valueInstantiator;
+
+        protected final int _parameterIndex;
+
+        public MapParameterAnyProperty(BeanProperty property, AnnotatedMember field, JavaType valueType,
+                KeyDeserializer keyDeser, JsonDeserializer<Object> valueDeser, TypeDeserializer typeDeser,
+                ValueInstantiator inst, int parameterIndex)
+        {
+            super(property, field, valueType, keyDeser, valueDeser, typeDeser);
+            _valueInstantiator = Objects.requireNonNull(inst, "ValueInstantiator for MapParameterAnyProperty cannot be `null`");
+            _parameterIndex = parameterIndex;
+        }
+
+        @Override
+        public SettableAnyProperty withValueDeserializer(JsonDeserializer<Object> deser)
+        {
+            return new MapParameterAnyProperty(_property, _setter, _type, _keyDeserializer, deser,
+                    _valueTypeDeserializer, _valueInstantiator, _parameterIndex);
+        }
+
+        @Override
+        protected void _set(Object instance, Object propName, Object value)
+        {
+            ((Map<Object, Object>) instance).put(propName, value);
+        }
+
+        @Override
+        public int getParameterIndex() { return _parameterIndex; }
+
+        @Override
+        public Object createParameterObject() { return new HashMap<>(); }
+
+    }
+
+    /**
+     * [databind#562] Allow @JsonAnySetter on Creator constructor
+     *
+     * @since 2.18
+     */
+    protected static class JsonNodeParameterAnyProperty extends SettableAnyProperty
+        implements java.io.Serializable
+    {
+        private static final long serialVersionUID = 1L;
+
+        protected final JsonNodeFactory _nodeFactory;
+
+        protected final int _parameterIndex;
+
+        public JsonNodeParameterAnyProperty(BeanProperty property, AnnotatedMember field, JavaType valueType,
+                JsonDeserializer<Object> valueDeser, JsonNodeFactory nodeFactory, int parameterIndex)
+        {
+            super(property, field, valueType, null, valueDeser, null);
+            _nodeFactory = nodeFactory;
+            _parameterIndex = parameterIndex;
+        }
+
+        // Let's override since this is much simpler with JsonNodes
+        @Override
+        public Object deserialize(JsonParser p, DeserializationContext ctxt)
+                throws IOException
+        {
+            return _valueDeserializer.deserialize(p, ctxt);
+        }
+
+        @Override
+        protected void _set(Object instance, Object propName, Object value)
+                throws Exception
+        {
+            ((ObjectNode) instance).set((String) propName, (JsonNode) value);
+        }
+
+        // Should not get called but...
+        @Override
+        public SettableAnyProperty withValueDeserializer(JsonDeserializer<Object> deser) { return this; }
+
+        @Override
+        public int getParameterIndex() { return _parameterIndex; }
+
+        @Override
+        public Object createParameterObject() { return _nodeFactory.objectNode(); }
+
+    }
+
 }

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

@@ -6,6 +6,7 @@
 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.ValueInstantiator;
 
@@ -194,7 +195,18 @@ public SettableBeanProperty findCreatorProperty(int propertyIndex) {
      */
     public PropertyValueBuffer startBuilding(JsonParser p, DeserializationContext ctxt,
             ObjectIdReader oir) {
-        return new PropertyValueBuffer(p, ctxt, _propertyCount, oir);
+        return new PropertyValueBuffer(p, ctxt, _propertyCount, oir, null);
+    }
+
+    /**
+     * Method called when starting to build a bean instance.
+     *
+     * @since 2.18 (added SettableAnyProperty parameter)
+     */
+    public PropertyValueBuffer startBuildingWithAnySetter(JsonParser p, DeserializationContext ctxt,
+            ObjectIdReader oir, SettableAnyProperty anySetter
+    ) {
+        return new PropertyValueBuffer(p, ctxt, _propertyCount, oir, anySetter);
     }
 
     public Object build(DeserializationContext ctxt, PropertyValueBuffer buffer) throws IOException