Refactor rename message2() -> message()

Author: rbygrave

validator/src/main/java/io/avaje/validation/adapter/ValidationContext.java

@@ -24,7 +24,7 @@ public interface ValidationContext {
     /**
      * Return the message object held by a validation adapter
      */
-    Message message2(Map<String, Object> attributes);
+    Message message(Map<String, Object> attributes);
 
     interface Message {
 

validator/src/main/java/io/avaje/validation/core/DValidator.java

@@ -70,7 +70,7 @@ private <T> ValidationType<T> typeWithCache(Type type) {
   }
 
   @Override
-  public Message message2(Map<String, Object> attributes) {
+  public Message message(Map<String, Object> attributes) {
     final String keyOrTemplate = (String)attributes.get("message");
 
     // if configured to support only 1 Locale then we can do the lookup and message translation once and early

validator/src/main/java/io/avaje/validation/core/adapters/BasicAdapters.java

@@ -4,7 +4,6 @@
 import java.util.Collection;
 import java.util.List;
 import java.util.Map;
-import java.util.Optional;
 import java.util.function.Predicate;
 import java.util.regex.Pattern;
 
@@ -18,19 +17,19 @@ private BasicAdapters() {}
 
   public static final ValidationContext.AnnotationFactory FACTORY = (annotationType, context, attributes) ->
     switch (annotationType.getSimpleName()) {
-        case "Email" -> new EmailAdapter(context.message2(attributes), attributes);
-        case "Null" -> new NullAdapter(context.message2(attributes));
-        case "NotNull", "NonNull" -> new NotNullAdapter(context.message2(attributes));
-        case "AssertTrue" -> new AssertBooleanAdapter(context.message2(attributes), false);
-        case "AssertFalse" -> new AssertBooleanAdapter(context.message2(attributes), true);
-        case "NotBlank" -> new NotBlankAdapter(context.message2(attributes));
-        case "NotEmpty" -> new NotEmptyAdapter(context.message2(attributes));
-        case "Past" -> new FuturePastAdapter(context.message2(attributes), true, false);
-        case "PastOrPresent" -> new FuturePastAdapter(context.message2(attributes), true, true);
-        case "Future" -> new FuturePastAdapter(context.message2(attributes), false, false);
-        case "FutureOrPresent" -> new FuturePastAdapter(context.message2(attributes), false, true);
-        case "Pattern" -> new PatternAdapter(context.message2(attributes), attributes);
-        case "Size" -> new SizeAdapter(context.message2(attributes), attributes);
+        case "Email" -> new EmailAdapter(context.message(attributes), attributes);
+        case "Null" -> new NullAdapter(context.message(attributes));
+        case "NotNull", "NonNull" -> new NotNullAdapter(context.message(attributes));
+        case "AssertTrue" -> new AssertBooleanAdapter(context.message(attributes), false);
+        case "AssertFalse" -> new AssertBooleanAdapter(context.message(attributes), true);
+        case "NotBlank" -> new NotBlankAdapter(context.message(attributes));
+        case "NotEmpty" -> new NotEmptyAdapter(context.message(attributes));
+        case "Past" -> new FuturePastAdapter(context.message(attributes), true, false);
+        case "PastOrPresent" -> new FuturePastAdapter(context.message(attributes), true, true);
+        case "Future" -> new FuturePastAdapter(context.message(attributes), false, false);
+        case "FutureOrPresent" -> new FuturePastAdapter(context.message(attributes), false, true);
+        case "Pattern" -> new PatternAdapter(context.message(attributes), attributes);
+        case "Size" -> new SizeAdapter(context.message(attributes), attributes);
         default -> null;
       };
 

validator/src/main/java/io/avaje/validation/core/adapters/NumberAdapters.java

@@ -17,19 +17,19 @@ private NumberAdapters() {}
   public static final ValidationContext.AnnotationFactory FACTORY =
       (annotationType, context, attributes) ->
           switch (annotationType.getSimpleName()) {
-            case "Digits" -> new DigitsAdapter(context.message2(attributes), attributes);
-            case "Positive" -> new PositiveAdapter(context.message2(attributes));
+            case "Digits" -> new DigitsAdapter(context.message(attributes), attributes);
+            case "Positive" -> new PositiveAdapter(context.message(attributes));
             case "PositiveOrZero" -> new PositiveAdapter(
-                context.message2(attributes), true);
-            case "Negative" -> new NegativeAdapter(context.message2(attributes));
+                context.message(attributes), true);
+            case "Negative" -> new NegativeAdapter(context.message(attributes));
             case "NegativeOrZero" -> new NegativeAdapter(
-                context.message2(attributes), true);
-            case "Max" -> new MaxAdapter(context.message2(attributes), attributes);
-            case "Min" -> new MinAdapter(context.message2(attributes), attributes);
+                context.message(attributes), true);
+            case "Max" -> new MaxAdapter(context.message(attributes), attributes);
+            case "Min" -> new MinAdapter(context.message(attributes), attributes);
             case "DecimalMax" -> new DecimalMaxAdapter(
-                context.message2(attributes), attributes);
+                context.message(attributes), attributes);
             case "DecimalMin" -> new DecimalMinAdapter(
-                context.message2(attributes), attributes);
+                context.message(attributes), attributes);
 
             default -> null;
           };