Remove org.apache.commons.beanutils2.MethodUtils.getAccessibleMethod(Class,

String, Class) in favor vararg variant

Author: garydgregory

src/changes/changes.xml

@@ -50,7 +50,8 @@
       <action dev="ggregory" type="remove" due-to="Gary Gregory">Remove unused org.apache.commons.beanutils2.MethodUtils.toNonPrimitiveClass(Class) in favor of org.apache.commons.lang3.ClassUtils.primitiveToWrapper(Class).</action>
       <action dev="ggregory" type="remove" due-to="Gary Gregory">Remove unused org.apache.commons.beanutils2.MethodUtils.getPrimitiveType(Class) in favor of org.apache.commons.lang3.ClassUtils.wrapperToPrimitive(Class).</action>
       <action dev="ggregory" type="remove" due-to="Gary Gregory">Remove org.apache.commons.beanutils2.MethodUtils.getPrimitiveWrapper(Class) in favor of org.apache.commons.lang3.ClassUtils.primitiveToWrapper(Class).</action>
-      <action dev="ggregory" type="remove" due-to="Gary Gregory">Remove org.apache.commons.beanutils2.MethodUtils.isAssignmentCompatible(Class toClass, Class cls) in favor oforg.apache.commons.lang3.ClassUtils.isAssignable(Class cls, Class toClass).</action>
+      <action dev="ggregory" type="remove" due-to="Gary Gregory">Remove org.apache.commons.beanutils2.MethodUtils.isAssignmentCompatible(Class toClass, Class cls) in favor of org.apache.commons.lang3.ClassUtils.isAssignable(Class cls, Class toClass).</action>
+      <action dev="ggregory" type="remove" due-to="Gary Gregory">Remove org.apache.commons.beanutils2.MethodUtils.getAccessibleMethod(Class, String, Class) in favor vararg variant.</action>
     </release>
     <release version="2.0.0-M2" date="2025-05-25" description="This is a major release and requires Java 8.">
       <!-- FIX -->

src/main/java/org/apache/commons/beanutils2/MethodUtils.java

@@ -208,22 +208,6 @@ public static Method getAccessibleMethod(Class<?> clazz, Method method) {
         return method;
     }
 
-    /**
-     * <p>
-     * Return an accessible method (that is, one that can be invoked via reflection) with given name and a single parameter. If no such method can be found,
-     * return {@code null}. Basically, a convenience wrapper that constructs a {@code Class} array for you.
-     * </p>
-     *
-     * @param clazz         get method from this class
-     * @param methodName    get method with this name
-     * @param parameterType taking this type of parameter
-     * @return The accessible method
-     */
-    public static Method getAccessibleMethod(final Class<?> clazz, final String methodName, final Class<?> parameterType) {
-        final Class<?>[] parameterTypes = { parameterType };
-        return getAccessibleMethod(clazz, methodName, parameterTypes);
-    }
-
     /**
      * <p>
      * Return an accessible method (that is, one that can be invoked via reflection) with given name and parameters. If no such method can be found, return
@@ -235,7 +219,7 @@ public static Method getAccessibleMethod(final Class<?> clazz, final String meth
      * @param parameterTypes with these parameters types
      * @return The accessible method
      */
-    public static Method getAccessibleMethod(final Class<?> clazz, final String methodName, final Class<?>[] parameterTypes) {
+    public static Method getAccessibleMethod(final Class<?> clazz, final String methodName, final Class<?>... parameterTypes) {
         try {
             final MethodDescriptor md = new MethodDescriptor(clazz, methodName, parameterTypes, true);
             // Check the cache first