start

Author: SentryMan

validator-generator/src/main/java/io/avaje/validation/generator/ContraintReader.java

@@ -27,8 +27,8 @@ final class ContraintReader implements BeanReader {
     importTypes.add("java.util.Map");
     importTypes.add("io.avaje.validation.adapter.ConstraintAdapter");
     importTypes.add("io.avaje.validation.adapter.ValidationAdapter");
-    importTypes.add("io.avaje.validation.adapter.ValidationContext");
     importTypes.add("io.avaje.validation.adapter.ValidationRequest");
+    importTypes.add("io.avaje.validation.adapter.ValidationContext.AdapterCreateRequest");
     importTypes.add("io.avaje.validation.spi.Generated");
 
     this.annotations =
@@ -121,7 +121,9 @@ public void writeConstructor(Append writer) {
 
     writer.append(
         """
-    final var message = ctx.<Object>message(attributes).template();
+    final var message = req.message().template();
+    final var ctx = req.ctx();
+    final var groups = req.groups();
     this.adapter =
 """);
 

validator-generator/src/main/java/io/avaje/validation/generator/ElementAnnotationContainer.java

@@ -8,6 +8,7 @@
 
 import javax.lang.model.element.Element;
 import javax.lang.model.element.ExecutableElement;
+import javax.lang.model.element.TypeElement;
 import javax.lang.model.element.VariableElement;
 
 // TODO: better name???
@@ -19,7 +20,7 @@ public record ElementAnnotationContainer(
     Map<GenericType, String> typeUse2) {
 
   static ElementAnnotationContainer create(Element element) {
-    final var hasValid = ValidPrism.isPresent(element);
+    final var hasValid = !(element instanceof TypeElement) && ValidPrism.isPresent(element);
     String rawType;
     Map<GenericType, String> typeUse1;
     Map<GenericType, String> typeUse2;

validator-generator/src/main/java/io/avaje/validation/generator/FieldReader.java

@@ -5,6 +5,7 @@
 
 import javax.lang.model.element.Element;
 import javax.lang.model.element.Modifier;
+import javax.lang.model.element.TypeElement;
 
 final class FieldReader {
 
@@ -22,6 +23,7 @@ final class FieldReader {
   private final boolean optionalValidation;
   private final Element element;
   private final ElementAnnotationContainer elementAnnotations;
+  private final boolean classLevel;
 
   FieldReader(Element element, List<String> genericTypeParams) {
     this.genericTypeParams = genericTypeParams;
@@ -34,6 +36,7 @@ final class FieldReader {
     adapterShortType = initAdapterShortType(shortType);
     adapterFieldName = initShortName();
     this.optionalValidation = Util.isNullable(element);
+    this.classLevel = element instanceof TypeElement;
   }
 
   private String initAdapterShortType(String shortType) {
@@ -120,7 +123,9 @@ void writeField(Append writer) {
   }
 
   private void writeGetValue(Append writer, String suffix) {
-    if (getter != null) {
+    if (classLevel) {
+      // don't need a getter
+    } else if (getter != null) {
       writer.append("value.%s()%s", getter.getName(), suffix);
     } else if (publicField) {
       writer.append("value.%s%s", fieldName, suffix);
@@ -131,6 +136,11 @@ private void writeGetValue(Append writer, String suffix) {
   }
 
   void writeValidate(Append writer) {
+    if (classLevel) {
+      writer.append("    %s.validate(value, request, field);", adapterFieldName);
+      writer.eol().eol();
+      return;
+    }
     writer.append("    var _$%s = ", fieldName);
     writeGetValue(writer, ";");
     writer.eol();
@@ -161,4 +171,8 @@ public void writeConstructor(Append writer) {
         writer, elementAnnotations, "        ", PrimitiveUtil.wrap(genericType.shortType()));
     writer.append(";").eol().eol();
   }
+
+  public boolean isClassLvl() {
+    return classLevel;
+  }
 }

validator-generator/src/main/java/io/avaje/validation/generator/SimpleAdapterWriter.java

@@ -50,15 +50,15 @@ void write() throws IOException {
   }
 
   private void writeConstructor() {
-    writer.append("  public %sValidationAdapter(ValidationContext ctx", adapterShortName);
-    for (int i = 0; i < genericParamsCount; i++) {
-      writer.append(", Type param%d", i);
-    }
 
-    if (beanReader instanceof ContraintReader) {
-      writer.append(", Set<Class<?>> groups, Map<String, Object> attributes");
+    if (isContraint) {
+      writer.append("  public %sValidationAdapter(AdapterCreateRequest req", adapterShortName);
+    } else {
+      writer.append("  public %sValidationAdapter(ValidationContext ctx", adapterShortName);
+      for (int i = 0; i < genericParamsCount; i++) {
+        writer.append(", Type param%d", i);
+      }
     }
-
     writer.append(") {", adapterShortName).eol();
     beanReader.writeConstructor(writer);
     writer.append("  }").eol();

validator-generator/src/main/java/io/avaje/validation/generator/TypeReader.java

@@ -32,14 +32,14 @@ final class TypeReader {
   private final Map<String, MethodReader> allGetterMethods = new LinkedHashMap<>();
   private final Map<String, MethodReader> maybeGetterMethods = new LinkedHashMap<>();
   private final TypeElement baseType;
-  private final boolean hasJsonAnnotation;
+  private final boolean hasValidAnnotation;
   private final Set<String> seenFields = new HashSet<>();
   private boolean nonAccessibleField;
   private final List<String> genericTypeParams;
 
   TypeReader(TypeElement baseType) {
     this.baseType = baseType;
-    this.hasJsonAnnotation = Util.isValid(baseType);
+    this.hasValidAnnotation = Util.isValid(baseType);
     this.genericTypeParams = initTypeParams(baseType);
   }
 
@@ -58,6 +58,8 @@ void read(TypeElement type) {
       }
     }
 
+    localFields.add(new FieldReader(type, genericTypeParams));
+
     for (final FieldReader localField : localFields) {
       allFields.add(localField);
       allFieldMap.put(localField.fieldName(), localField);
@@ -93,10 +95,8 @@ private void readMethod(Element element, TypeElement type, List<FieldReader> loc
       final List<? extends VariableElement> parameters = methodElement.getParameters();
       final String methodKey = methodElement.getSimpleName().toString();
       final MethodReader methodReader = new MethodReader(methodElement, type).read();
-      if (parameters.size() == 0) {
-        if (!maybeGetterMethods.containsKey(methodKey)) {
-          maybeGetterMethods.put(methodKey, methodReader);
-        }
+      if (parameters.isEmpty()) {
+        maybeGetterMethods.putIfAbsent(methodKey, methodReader);
         allGetterMethods.put(methodKey.toLowerCase(), methodReader);
       }
       // for reading methods
@@ -112,6 +112,9 @@ private void readMethod(Element element, TypeElement type, List<FieldReader> loc
 
   private void matchFieldsToGetter() {
     for (final FieldReader field : allFields) {
+      if (field.isClassLvl()) {
+        continue;
+      }
       matchFieldToGetter(field);
     }
   }
@@ -121,8 +124,13 @@ private void matchFieldToGetter(FieldReader field) {
         && !matchFieldToGetter2(field, true)
         && !field.isPublicField()) {
       nonAccessibleField = true;
-      if (hasJsonAnnotation) {
-        logError("Non accessible field " + baseType + " " + field.fieldName() + " with no matching getter?");
+      if (hasValidAnnotation) {
+        logError(
+            "Non accessible field "
+                + baseType
+                + " "
+                + field.fieldName()
+                + " with no matching getter?");
       } else {
         logDebug("Non accessible field " + baseType + " " + field.fieldName());
       }

validator-generator/src/test/java/io/avaje/validation/generator/ValidatorProcessorTest.java

@@ -44,7 +44,6 @@ void deleteGeneratedFiles() throws IOException {
     }
   }
 
-  @Disabled
   @Test
   void testGeneration() throws Exception {
     final String source =

validator-generator/src/test/java/io/avaje/validation/generator/models/valid/CustomAnnotationAdapter.java renamed to validator-generator/src/test/java/io/avaje/validation/generator/models/valid/CheckCaseAdapter.java

@@ -2,15 +2,15 @@
 
 import io.avaje.validation.adapter.AbstractConstraintAdapter;
 import io.avaje.validation.adapter.ConstraintAdapter;
-import io.avaje.validation.adapter.ValidationContext;
+import io.avaje.validation.adapter.ValidationContext.AdapterCreateRequest;
 import io.avaje.validation.generator.models.valid.CheckCase.CaseMode;
 
 @ConstraintAdapter(CheckCase.class)
-public final class CustomAnnotationAdapter extends AbstractConstraintAdapter<String> {
+public final class CheckCaseAdapter extends AbstractConstraintAdapter<String> {
 
   private final CaseMode caseMode;
 
-  public CustomAnnotationAdapter(ValidationContext.AdapterCreateRequest request) {
+  public CheckCaseAdapter(AdapterCreateRequest request) {
     super(request);
     final var attributes = request.attributes();
     caseMode = (CaseMode) attributes.get("caseMode");

validator-generator/src/test/java/io/avaje/validation/generator/models/valid/typeconstraint/PassingSkill.java

@@ -0,0 +1,18 @@
+package io.avaje.validation.generator.models.valid.typeconstraint;
+
+import static java.lang.annotation.ElementType.TYPE;
+import static java.lang.annotation.RetentionPolicy.SOURCE;
+
+import java.lang.annotation.Retention;
+import java.lang.annotation.Target;
+
+import io.avaje.validation.constraints.Constraint;
+
+@Target(TYPE)
+@Retention(SOURCE)
+@Constraint
+public @interface PassingSkill {
+  String message() default "put these foolish ambitions to rest"; // default error message
+
+  Class<?>[] groups() default {}; // groups
+}

validator-generator/src/test/java/io/avaje/validation/generator/models/valid/typeconstraint/PassingSkillAdapter.java

@@ -0,0 +1,21 @@
+package io.avaje.validation.generator.models.valid.typeconstraint;
+
+import io.avaje.validation.adapter.AbstractConstraintAdapter;
+import io.avaje.validation.adapter.ConstraintAdapter;
+import io.avaje.validation.adapter.ValidationContext.AdapterCreateRequest;
+
+@ConstraintAdapter(PassingSkill.class)
+public final class PassingSkillAdapter extends AbstractConstraintAdapter<Tarnished> {
+
+  public PassingSkillAdapter(AdapterCreateRequest request) {
+    super(request);
+  }
+
+  @Override
+  public boolean isValid(Tarnished lowlyTarnished) {
+    if (lowlyTarnished == null) {
+      return true;
+    }
+    return lowlyTarnished.vigor() >= 50 && lowlyTarnished.endurance() >= 50;
+  }
+}

validator-generator/src/test/java/io/avaje/validation/generator/models/valid/typeconstraint/Tarnished.java

@@ -0,0 +1,7 @@
+package io.avaje.validation.generator.models.valid.typeconstraint;
+
+import io.avaje.validation.constraints.Valid;
+
+@Valid
+@PassingSkill
+public record Tarnished(int vigor, int endurance) {}

validator/src/main/java/io/avaje/validation/Validator.java

@@ -1,21 +1,21 @@
 package io.avaje.validation;
 
-import io.avaje.lang.Nullable;
-import io.avaje.validation.adapter.*;
-import io.avaje.validation.core.DefaultBootstrap;
-import io.avaje.validation.spi.MessageInterpolator;
-import io.avaje.validation.spi.ValidatorCustomizer;
-
 import java.lang.annotation.Annotation;
 import java.lang.reflect.Type;
 import java.time.Clock;
 import java.time.Duration;
 import java.util.Locale;
-import java.util.Map;
 import java.util.ResourceBundle;
-import java.util.Set;
 import java.util.function.Supplier;
 
+import io.avaje.lang.Nullable;
+import io.avaje.validation.adapter.ValidationAdapter;
+import io.avaje.validation.adapter.ValidationContext;
+import io.avaje.validation.adapter.ValidationContext.AdapterCreateRequest;
+import io.avaje.validation.core.DefaultBootstrap;
+import io.avaje.validation.spi.MessageInterpolator;
+import io.avaje.validation.spi.ValidatorCustomizer;
+
 /**
  * Validate plain Java objects that have been annotated with validation constraints.
  *
@@ -145,8 +145,7 @@ interface AdapterBuilder {
   interface AnnotationAdapterBuilder {
 
     /** Create a ValidationAdapter given the Validator instance. */
-    ValidationAdapter<?> build(
-        ValidationContext ctx, Set<Class<?>> groups, Map<String, Object> attributes);
+    ValidationAdapter<?> build(AdapterCreateRequest request);
   }
 
   /** Components register ValidationAdapters Validator.Builder */

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

@@ -19,12 +19,12 @@
  *
  *   String value;
  *
- *   public CustomAnnotationAdapter(ValidationContext ctx, Set<Class<?>> groups, Map<String, Object> attributes) {
+ *   public CustomAnnotationAdapter(AdapterCreateRequest req) {
  *     //create a message object for error interpolation and set groups
- *      super(ctx.message(attributes), groups);
+ *      super(req);
  *
  *      //use the attributes to extract the annotation values
- *      value = (String) attributes.get("value");
+ *      value = (String) req.attribute("value");
  *   }
  *
  *

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

@@ -41,8 +41,8 @@ public interface ValidationContext {
   <T> ValidationAdapter<T> adapter(Class<? extends Annotation> cls, Map<String, Object> attributes);
 
   /**
-   * Return the constraint adapter for the given annotation with attributes. Used for adapters that combine
-   * multiple annotation adapters.
+   * Return the constraint adapter for the given annotation with attributes. Used for adapters that
+   * combine multiple annotation adapters.
    *
    * @param cls The class representing the annotation type
    * @param groups The validation groups associated with the annotation

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

@@ -89,7 +89,8 @@ private <T> ValidationType<T> type(Class<T> cls) {
 
   @SuppressWarnings("unchecked")
   private <T> ValidationType<T> typeWithCache(Type type) {
-    return (ValidationType<T>) typeCache.computeIfAbsent(type, k -> new ValidationType<>(this, adapter(k)));
+    return (ValidationType<T>)
+        typeCache.computeIfAbsent(type, k -> new ValidationType<>(this, adapter(k)));
   }
 
   @Override
@@ -116,7 +117,8 @@ public <T> ValidationAdapter<T> adapter(Class<T> cls) {
   }
 
   @Override
-  public <T> ValidationAdapter<T> adapter(Class<? extends Annotation> cls, Map<String, Object> attributes) {
+  public <T> ValidationAdapter<T> adapter(
+      Class<? extends Annotation> cls, Map<String, Object> attributes) {
     return builder.annotationAdapter(cls, attributes, null);
   }
 
@@ -302,8 +304,7 @@ private static <T> AnnotationFactory newAnnotationAdapterFactory(
         Type type, ValidationAdapter<T> adapter) {
       requireNonNull(type);
       requireNonNull(adapter);
-      return (request) ->
-          simpleMatch(type, request.annotationType()) ? adapter : null;
+      return (request) -> simpleMatch(type, request.annotationType()) ? adapter : null;
     }
 
     private static <T> AdapterFactory newAdapterFactory(Type type, ValidationAdapter<T> adapter) {
@@ -322,8 +323,7 @@ private static AnnotationFactory newAdapterFactory(
         Class<? extends Annotation> type, AnnotationAdapterBuilder builder) {
       requireNonNull(type);
       requireNonNull(builder);
-      return (req) ->
-          simpleMatch(type, req.annotationType()) ? builder.build(req.ctx(), req.groups(), req.attributes()) : null;
+      return (req) -> simpleMatch(type, req.annotationType()) ? builder.build(req) : null;
     }
 
     private static boolean simpleMatch(Type type, Type targetType) {