Fix #3412 (Android compat issue)

Author: cowtowncoder

release-notes/CREDITS-2.x

@@ -1423,3 +1423,8 @@ Krishna Ghimire (Krishnaghimir@github)
 Christoph Dreis (dreis2211@github)
   * Suggested #3293: Use Method.getParameterCount() where possible
   (2.13.2)
+
+Matthieu Finiasz (finiasz@github)
+  * Reported #3412: Version 2.13.2 uses `Method.getParameterCount()` which is not
+   supported on Android before API 26
+  (2.13.2)

release-notes/VERSION-2.x

@@ -4,6 +4,12 @@ Project: jackson-databind
 === Releases === 
 ------------------------------------------------------------------------
 
+2.13.3 (not yet released)
+
+#3412: Version 2.13.2 uses `Method.getParameterCount()` which is not
+  supported on Android before API 26
+ (reported by Matthew F)
+
 2.13.2 (06-Mar-2022)
 
 #3293: Use Method.getParameterCount() where possible

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

@@ -961,8 +961,8 @@ protected SettableBeanProperty _resolveInnerClassValuedProperty(DeserializationC
                 // and is inner class of the bean class...
                 if ((enclosing != null) && (enclosing == _beanType.getRawClass())) {
                     for (Constructor<?> ctor : valueClass.getConstructors()) {
-                        if (ctor.getParameterCount() == 1) {
-                            Class<?>[] paramTypes = ctor.getParameterTypes();
+                        Class<?>[] paramTypes = ctor.getParameterTypes();
+                        if (paramTypes.length == 1) {
                             if (enclosing.equals(paramTypes[0])) {
                                 if (ctxt.canOverrideAccessModifiers()) {
                                     ClassUtil.checkAndFixAccess(ctor, ctxt.isEnabled(MapperFeature.OVERRIDE_PUBLIC_ACCESS_MODIFIERS));

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

@@ -169,7 +169,7 @@ public Object getValue(Object pojo)
 
     @Override
     public String toString() {
-        final int argCount = _constructor.getParameterCount();
+        final int argCount = _constructor.getParameterTypes().length;
         return String.format("[constructor for %s (%d arg%s), annotations: %s",
                 ClassUtil.nameOf(_constructor.getDeclaringClass()), argCount,
                 (argCount == 1) ? "" : "s", _annotations);

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

@@ -352,7 +352,7 @@ protected AnnotatedConstructor constructNonDefaultConstructor(ClassUtil.Ctor cto
     protected AnnotatedMethod constructFactoryCreator(Method m,
             TypeResolutionContext typeResCtxt, Method mixin)
     {
-        final int paramCount = m.getParameterCount();
+        final int paramCount = m.getParameterTypes().length;
         if (_intr == null) { // when annotation processing is disabled
             return new AnnotatedMethod(typeResCtxt, m, _emptyAnnotationMap(),
                     _emptyAnnotationMaps(paramCount));

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

@@ -184,7 +184,7 @@ private static boolean _isIncludableMemberMethod(Method m)
         }
         // also, for now we have no use for methods with more than 2 arguments:
         // (2 argument methods for "any setter", fwtw)
-        return (m.getParameterCount() <= 2);
+        return (m.getParameterTypes().length <= 2);
     }
 
     private final static class MethodBuilder {

src/main/java/com/fasterxml/jackson/databind/util/ClassUtil.java

@@ -1448,7 +1448,7 @@ public Constructor<?> getConstructor() {
         public int getParamCount() {
             int c = _paramCount;
             if (c < 0) {
-                c = _ctor.getParameterCount();
+                c = _ctor.getParameterTypes().length;
                 _paramCount = c;
             }
             return c;