Use Interpolation Interface

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

@@ -75,7 +75,7 @@ <T> ValidationAdapter<T> buildAnnotation(Class<? extends Annotation> cls, Map<St
         return result;
       }
     }
-    // unknown annotation have noop
+    // unknown annotations have noop
     return NoopAnnotationValidator.INSTANCE;
   }
 }

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

@@ -12,7 +12,6 @@
 import java.util.Locale;
 import java.util.Map;
 import java.util.ServiceLoader;
-import java.util.Set;
 import java.util.concurrent.ConcurrentHashMap;
 
 import io.avaje.lang.Nullable;
@@ -108,22 +107,14 @@ ValidationRequest request(@Nullable Locale locale) {
     return new DRequest(this, locale);
   }
 
-
   String interpolate(Message msg, Locale requestLocale) {
+
     // resolve the locale to use to produce the message
     final Locale locale = localeResolver.resolve(requestLocale);
     // lookup in resource bundles using resolved locale and template
     final String template = templateLookup.lookup(msg.template(), locale);
-    // translate the template using msg attributes
-    final Map<String, Object> attributes = msg.attributes();
-    final Set<Map.Entry<String, Object>> entries = attributes.entrySet();
-    String result = template;
-    for (final Map.Entry<String, Object> entry : entries) {
-      // needs work here to improve functionality, support local specific value formatting eg Duration Max
-      result = result.replace('{' + entry.getKey() + '}', String.valueOf(entry.getValue()));
-    }
-    // return the message
-    return result;
+
+    return interpolator.interpolate(template, msg.attributes());
   }
 
   /** Implementation of Validator.Builder. */
@@ -184,7 +175,7 @@ public DValidator build() {
       final var interpolator =
           ServiceLoader.load(MessageInterpolator.class)
               .findFirst()
-              .orElseGet(NoopMessageInterpolator::new);
+              .orElseGet(BasicMessageInterpolator::new);
       return new DValidator(factories, afactories, interpolator, localeResolver);
     }
 

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

@@ -1,6 +1,8 @@
 package io.avaje.validation.core;
 
+import java.util.Map;
+
 public interface MessageInterpolator {
 
-  String interpolate(String string);
+  String interpolate(String template, Map<String, Object> attributes);
 }

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

@@ -1,9 +1,20 @@
 package io.avaje.validation.core;
 
-final class NoopMessageInterpolator implements MessageInterpolator {
+import java.util.Map;
+
+final class BasicMessageInterpolator implements MessageInterpolator {
 
   @Override
-  public String interpolate(String string) {
-    return string;
+  public String interpolate(String template, Map<String, Object> attributes) {
+
+    String result = template;
+
+    for (final Map.Entry<String, Object> entry : attributes.entrySet()) {
+      // needs work here to improve functionality, support local specific value formatting eg
+      // Duration Max
+      result = result.replace('{' + entry.getKey() + '}', String.valueOf(entry.getValue()));
+    }
+    // return the message
+    return result;
   }
 }