commit a9610dc1d79166f66950e5c9f7a356e27764b3ce
author Romain Manni-Bucau <rmannibucau@apache.org> Wed May 16 22:52:43 2018 +0200
committer Matt Benson <mbenson@apache.org> Tue Oct 16 12:28:20 2018 -0500
tree da6b1f29a072a9cde8f21ef00f5bc56cb680b9d8
parent 21c1ed9c8c68df81237d4834fb05ef1c71d21d6a

ensure isGetter methods are consistent between them

bval-jsr/src/main/java/org/apache/bval/jsr/util/Methods.java

diff --git a/bval-jsr/src/main/java/org/apache/bval/jsr/util/Methods.java b/bval-jsr/src/main/java/org/apache/bval/jsr/util/Methods.java
index d0d5bdb..d8f2875 100644
--- a/bval-jsr/src/main/java/org/apache/bval/jsr/util/Methods.java
+++ b/bval-jsr/src/main/java/org/apache/bval/jsr/util/Methods.java
@@ -33,10 +33,10 @@
             return false;
         }
         // TODO look for capital letter after verb?
-        if (Boolean.TYPE.equals(m.getReturnType()) && m.getName().startsWith("is")) {
+        if (Boolean.TYPE.equals(m.getReturnType()) && isGetter(m.getName())) {
             return true;
         }
-        return !Void.TYPE.equals(m.getReturnType()) && m.getName().startsWith("get");
+        return !Void.TYPE.equals(m.getReturnType()) && isGetter(m.getName());
     }
 
     public static boolean isGetter(String methodName) {
@@ -51,7 +51,7 @@
     }
 
     public static String propertyName(String methodName) {
-        Validate.isTrue(isGetter(methodName), "%s does not represent a property getter");
+        Validate.isTrue(isGetter(methodName), "%s does not represent a property getter", methodName);
         final String suffix = methodName.startsWith("is") ? methodName.substring(2) : methodName.substring(3);
         return Introspector.decapitalize(suffix);
     }