Merge pull request #15 from SentryMan/core

removed required

Author: rob-bygrave

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

@@ -1,7 +1,3 @@
 package io.avaje.validation.adapter;
 
-public interface CoreValidation {
-
-    /** Return true to continue validation on this value if needed */
-    boolean required(Object value, ValidationRequest ctx, String propertyName);
-}
+public interface CoreValidation {}

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

@@ -1,26 +1,11 @@
-/*
- * Copyright (C) 2014 Square, Inc.
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *    https://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
 package io.avaje.validation.adapter;
 
-import io.avaje.validation.Validator;
-
 import java.lang.reflect.Type;
 import java.util.Collection;
 import java.util.Objects;
 
+import io.avaje.validation.Validator;
+
 public interface ValidationAdapter<T> {
 
   /** Return true if validation should recurse */
@@ -35,8 +20,9 @@ default boolean validateAll(Collection<T> value, ValidationRequest req, String p
       req.pushPath(propertName);
     }
     int index = -1;
-    for (T element : value) {
-      validate(element, req, String.valueOf(++index));
+    for (final T element : value) {
+      index++;
+  validate(element, req, String.valueOf(index));
     }
     if (propertName != null) {
       req.popPath();
@@ -49,8 +35,9 @@ default boolean validateAll(T[] value, ValidationRequest req, String propertName
       req.pushPath(propertName);
     }
     int index = -1;
-    for (T element : value) {
-      validate(element, req, String.valueOf(++index));
+    for (final T element : value) {
+      index++;
+  validate(element, req, String.valueOf(index));
     }
     if (propertName != null) {
       req.popPath();

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

@@ -1,18 +1,3 @@
-/*
- * Copyright (C) 2014 Square, Inc.
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *    https://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
 package io.avaje.validation.core;
 
 import java.lang.annotation.Annotation;

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

@@ -1,16 +1,8 @@
 package io.avaje.validation.core;
 
 import io.avaje.validation.adapter.CoreValidation;
-import io.avaje.validation.adapter.ValidationRequest;
 
 final class DCoreValidation implements CoreValidation {
 
-    @Override
-    public boolean required(Object value, ValidationRequest ctx, String propertyName) {
-        if (value == null) {
-            ctx.addViolation("Required", propertyName);
-            return false;
-        }
-        return true;
-    }
+
 }

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

@@ -1,18 +1,3 @@
-/*
- * Copyright (C) 2014 Square, Inc.
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *    https://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
 package io.avaje.validation.core;
 
 import java.lang.reflect.Array;

validator/src/test/java/io/avaje/validation/core/CustomerValidationAdapter.java

@@ -53,9 +53,7 @@ public boolean validate(Customer value, ValidationRequest request, String proper
     activeDateAdapter.validate(value.activeDate, request, "activeDate");
 
     final var _billingAddress = value.billingAddress;
-    if (core.required(_billingAddress, request, "billingAddress")) { // required / NotNull
       addressValidator.validate(_billingAddress, request, "billingAddress");
-    }
 
     final var _shippingAddress = value.shippingAddress;
     if (_shippingAddress != null) { // is nullable