Allow using @JsonPropertyOrder with any-property (@JsonAnyGetter)

src/main/java/com/fasterxml/jackson/databind/introspect/POJOPropertiesCollector.java

@@ -554,8 +554,10 @@ protected void _addFields(Map<String, POJOPropertyBuilder> props)
                         _anySetterField = new LinkedList<>();
                     }
                     _anySetterField.add(f);
+                    // 07-Feb-2025: [databind#4775]: Skip the rest of processing, but only
+                    //    for "any-setter', not any-getter
+                    continue;
                 }
-                continue;
             }
             String implName = ai.findImplicitPropertyName(f);
             if (implName == null) {
@@ -1125,18 +1127,19 @@ protected void _addGetterMethod(Map<String, POJOPropertyBuilder> props,
                 _anyGetters = new LinkedList<>();
             }
             _anyGetters.add(m);
-            return;
+            // 07-Feb-2025: [databind#4775] Do not stop processing here
+            //   (used to return)
         }
         // @JsonKey?
-        if (Boolean.TRUE.equals(ai.hasAsKey(_config, m))) {
+        else if (Boolean.TRUE.equals(ai.hasAsKey(_config, m))) {
             if (_jsonKeyAccessors == null) {
                 _jsonKeyAccessors = new LinkedList<>();
             }
             _jsonKeyAccessors.add(m);
             return;
         }
         // @JsonValue?
-        if (Boolean.TRUE.equals(ai.hasAsValue(m))) {
+        else if (Boolean.TRUE.equals(ai.hasAsValue(m))) {
             if (_jsonValueAccessors == null) {
                 _jsonValueAccessors = new LinkedList<>();
             }

src/main/java/com/fasterxml/jackson/databind/ser/AnyGetterWriter.java

@@ -12,23 +12,31 @@
  * for serializing {@link com.fasterxml.jackson.annotation.JsonAnyGetter} annotated
  * (Map) properties
  */
-public class AnyGetterWriter
+public class AnyGetterWriter extends BeanPropertyWriter
+    implements java.io.Serializable // since 2.19
 {
+    // As of 2.19
+    private static final long serialVersionUID = 1L;
+
     protected final BeanProperty _property;
 
     /**
-     * Method (or field) that represents the "any getter"
+     * Method (or Field) that represents the "any getter"
      */
     protected final AnnotatedMember _accessor;
 
     protected JsonSerializer<Object> _serializer;
 
     protected MapSerializer _mapSerializer;
 
+    /**
+     * @since 2.19
+     */
     @SuppressWarnings("unchecked")
-    public AnyGetterWriter(BeanProperty property,
+    public AnyGetterWriter(BeanPropertyWriter parent, BeanProperty property,
             AnnotatedMember accessor, JsonSerializer<?> serializer)
     {
+        super(parent);
         _accessor = accessor;
         _property = property;
         _serializer = (JsonSerializer<Object>) serializer;
@@ -37,9 +45,20 @@ public AnyGetterWriter(BeanProperty property,
         }
     }
 
+    /**
+     * @deprecated Since 2.19, use one that takes {@link BeanPropertyWriter} instead.
+     */
+    @Deprecated
+    public AnyGetterWriter(BeanProperty property,
+            AnnotatedMember accessor, JsonSerializer<?> serializer)
+    {
+        this(null, property, accessor, serializer);
+    }
+
     /**
      * @since 2.8.3
      */
+    @Override
     public void fixAccess(SerializationConfig config) {
         _accessor.fixAccess(
                 config.isEnabled(MapperFeature.OVERRIDE_PUBLIC_ACCESS_MODIFIERS));
@@ -65,6 +84,11 @@ public void getAndSerialize(Object bean, JsonGenerator gen, SerializerProvider p
         _serializer.serialize(value, gen, provider);
     }
 
+    @Override
+    public void serializeAsField(Object bean, JsonGenerator gen, SerializerProvider prov) throws Exception {
+        getAndSerialize(bean, gen, prov);
+    }
+
     /**
      * @since 2.3
      */

src/main/java/com/fasterxml/jackson/databind/ser/BeanSerializer.java

@@ -147,7 +147,6 @@ protected BeanSerializerBase asArraySerializer()
          * - have per-property filters
          */
         if ((_objectIdWriter == null)
-                && (_anyGetterWriter == null)
                 && (_propertyFilterId == null)
                 ) {
             return new BeanAsArraySerializer(this);

src/main/java/com/fasterxml/jackson/databind/ser/BeanSerializerFactory.java

@@ -26,11 +26,7 @@
 import com.fasterxml.jackson.databind.ser.std.StdDelegatingSerializer;
 import com.fasterxml.jackson.databind.ser.std.ToEmptyObjectSerializer;
 import com.fasterxml.jackson.databind.type.ReferenceType;
-import com.fasterxml.jackson.databind.util.BeanUtil;
-import com.fasterxml.jackson.databind.util.ClassUtil;
-import com.fasterxml.jackson.databind.util.Converter;
-import com.fasterxml.jackson.databind.util.IgnorePropertiesUtil;
-import com.fasterxml.jackson.databind.util.NativeImageUtil;
+import com.fasterxml.jackson.databind.util.*;
 
 /**
  * Factory class that can provide serializers for any regular Java beans
@@ -464,7 +460,33 @@ protected JsonSerializer<Object> constructBeanOrAddOnSerializer(SerializerProvid
             PropertyName name = PropertyName.construct(anyGetter.getName());
             BeanProperty.Std anyProp = new BeanProperty.Std(name, valueType, null,
                     anyGetter, PropertyMetadata.STD_OPTIONAL);
-            builder.setAnyGetter(new AnyGetterWriter(anyProp, anyGetter, anySer));
+
+            // Check if there is an accessor exposed for the anyGetter
+            BeanPropertyWriter anyGetterProp = null;
+            int anyGetterIndex = -1;
+            for (int i = 0; i < props.size(); i++) {
+                BeanPropertyWriter prop = props.get(i);
+                // Either any-getter as field...
+                if (Objects.equals(prop.getName(), anyGetter.getName())
+                    // or as method
+                    || Objects.equals(prop.getMember().getMember(), anyGetter.getMember()))
+                {
+                    anyGetterProp = prop;
+                    anyGetterIndex = i;
+                    break;
+                }
+            }
+            if (anyGetterIndex != -1) {
+                // There is prop is already in place, just need to replace it
+                AnyGetterWriter anyGetterWriter = new AnyGetterWriter(anyGetterProp, anyProp, anyGetter, anySer);
+                props.set(anyGetterIndex, anyGetterWriter);
+            } else {
+                // Otherwise just add it at the end, but won't be sorted...
+                // This is case where JsonAnyGetter is private/protected,
+                BeanPropertyDefinition anyGetterPropDef = SimpleBeanPropertyDefinition.construct(config, anyGetter, name);
+                BeanPropertyWriter anyPropWriter = _constructWriter(prov, anyGetterPropDef, new PropertyBuilder(config, beanDesc), staticTyping, anyGetter);
+                props.add(new AnyGetterWriter(anyPropWriter, anyProp, anyGetter, anySer));
+            }
         }
         // Next: need to gather view information, if any:
         processViews(config, builder);

src/main/java/com/fasterxml/jackson/databind/ser/impl/BeanAsArraySerializer.java

@@ -214,10 +214,6 @@ protected final void serializeAsArray(Object bean, JsonGenerator gen, Serializer
                     prop.serializeAsElement(bean, gen, provider);
                 }
             }
-            // NOTE: any getters cannot be supported either
-            //if (_anyGetterWriter != null) {
-            //    _anyGetterWriter.getAndSerialize(bean, gen, provider);
-            //}
         } catch (Exception e) {
             wrapAndThrow(provider, e, bean, props[i].getName());
         } catch (StackOverflowError e) {

src/main/java/com/fasterxml/jackson/databind/ser/impl/SimpleBeanPropertyFilter.java

@@ -218,6 +218,8 @@ public void serializeAsField(Object pojo, JsonGenerator jgen,
             writer.serializeAsField(pojo, jgen, provider);
         } else if (!jgen.canOmitFields()) { // since 2.3
             writer.serializeAsOmittedField(pojo, jgen, provider);
+        } else if (writer instanceof AnyGetterWriter) {
+            ((AnyGetterWriter) writer).getAndFilter(pojo, jgen, provider, this);
         }
     }
 

src/main/java/com/fasterxml/jackson/databind/ser/std/BeanSerializerBase.java

@@ -65,12 +65,6 @@ public abstract class BeanSerializerBase
      */
     final protected BeanPropertyWriter[] _filteredProps;
 
-    /**
-     * Handler for {@link com.fasterxml.jackson.annotation.JsonAnyGetter}
-     * annotated properties
-     */
-    final protected AnyGetterWriter _anyGetterWriter;
-
     /**
      * Id of the bean property filter to use, if any; null if none.
      */
@@ -118,13 +112,11 @@ protected BeanSerializerBase(JavaType type, BeanSerializerBuilder builder,
             // 20-Sep-2019, tatu: Actually not just that but also "dummy" serializer for
             //     case of no bean properties, too
             _typeId = null;
-            _anyGetterWriter = null;
             _propertyFilterId = null;
             _objectIdWriter = null;
             _serializationShape = null;
         } else {
             _typeId = builder.getTypeId();
-            _anyGetterWriter = builder.getAnyGetter();
             _propertyFilterId = builder.getFilterId();
             _objectIdWriter = builder.getObjectIdWriter();
             final JsonFormat.Value format = builder.getBeanDescription().findExpectedFormat();
@@ -141,7 +133,6 @@ protected BeanSerializerBase(BeanSerializerBase src,
         _filteredProps = filteredProperties;
 
         _typeId = src._typeId;
-        _anyGetterWriter = src._anyGetterWriter;
         _objectIdWriter = src._objectIdWriter;
         _propertyFilterId = src._propertyFilterId;
         _serializationShape = src._serializationShape;
@@ -165,7 +156,6 @@ protected BeanSerializerBase(BeanSerializerBase src,
         _filteredProps = src._filteredProps;
 
         _typeId = src._typeId;
-        _anyGetterWriter = src._anyGetterWriter;
         _objectIdWriter = objectIdWriter;
         _propertyFilterId = filterId;
         _serializationShape = src._serializationShape;
@@ -209,7 +199,6 @@ protected BeanSerializerBase(BeanSerializerBase src, Set<String> toIgnore, Set<S
         _filteredProps = (fpropsOut == null) ? null : fpropsOut.toArray(new BeanPropertyWriter[fpropsOut.size()]);
 
         _typeId = src._typeId;
-        _anyGetterWriter = src._anyGetterWriter;
         _objectIdWriter = src._objectIdWriter;
         _propertyFilterId = src._propertyFilterId;
         _serializationShape = src._serializationShape;
@@ -401,9 +390,11 @@ public void resolve(SerializerProvider provider)
         }
 
         // also, any-getter may need to be resolved
-        if (_anyGetterWriter != null) {
-            // 23-Feb-2015, tatu: Misleading, as this actually triggers call to contextualization...
-            _anyGetterWriter.resolve(provider);
+        for (int i = 0; i < _props.length; i++) {
+            BeanPropertyWriter prop = _props[i];
+            if (prop instanceof AnyGetterWriter) {
+                ((AnyGetterWriter) prop).resolve(provider);
+            }
         }
     }
 
@@ -769,9 +760,6 @@ protected void serializeFields(Object bean, JsonGenerator gen, SerializerProvide
                     prop.serializeAsField(bean, gen, provider);
                 }
             }
-            if (_anyGetterWriter != null) {
-                _anyGetterWriter.getAndSerialize(bean, gen, provider);
-            }
         } catch (Exception e) {
             String name = (i == props.length) ? "[anySetter]" : props[i].getName();
             wrapAndThrow(provider, e, bean, name);
@@ -820,9 +808,6 @@ protected void serializeFieldsFiltered(Object bean, JsonGenerator gen,
                     filter.serializeAsField(bean, gen, provider, prop);
                 }
             }
-            if (_anyGetterWriter != null) {
-                _anyGetterWriter.getAndFilter(bean, gen, provider, filter);
-            }
         } catch (Exception e) {
             String name = (i == props.length) ? "[anySetter]" : props[i].getName();
             wrapAndThrow(provider, e, bean, name);