src/org/kar/archidata/dataAccess/DBAccessSQL.java

@@ -815,9 +815,6 @@ public class DBAccessSQL extends DBAccess {
 	@Override
 	@SuppressFBWarnings("SQL_PREPARED_STATEMENT_GENERATED_FROM_NONCONSTANT_STRING")
 	public <T> T insert(final T data, final QueryOption... option) throws Exception {
-		if (data == null) {
-			throw new DataAccessException("Try to check a null data ==> wrong API");
-		}
 		final Class<?> clazz = data.getClass();
 		final QueryOptions options = new QueryOptions(option);
 

src/org/kar/archidata/dataAccess/addOnSQL/AddOnManyToMany.java

@@ -355,12 +355,12 @@ public class AddOnManyToMany implements DataAccessAddOn {
 		final FieldName columnName = AnnotationTools.getFieldName(field, options);
 		final String linkTableName = generateLinkTableName(tableName, columnName.inTable());
 		@SuppressWarnings("unchecked")
-		final List<Object> dataCasted = (List<Object>) data;
+		final List<Long> dataCasted = (List<Long>) data;
 		if (dataCasted.size() == 0) {
 			return;
 		}
 		final List<LinkTableGeneric> insertElements = new ArrayList<>();
-		for (final Object remoteKey : dataCasted) {
+		for (final Long remoteKey : dataCasted) {
 			if (remoteKey == null) {
 				throw new DataAccessException("Try to insert remote key with null value");
 			}

src/org/kar/archidata/dataAccess/options/CheckJPA.java

@@ -5,7 +5,6 @@ import java.sql.Timestamp;
 import java.time.LocalDate;
 import java.time.LocalTime;
 import java.util.ArrayList;
-import java.util.Collection;
 import java.util.Date;
 import java.util.HashMap;
 import java.util.List;
@@ -476,7 +475,6 @@ public class CheckJPA<T> implements CheckFunctionInterface {
 				if (dataJson != null && dataJson.checker() != null) {
 					final CheckFunctionInterface checkerInstance = dataJson.checker().getDeclaredConstructor()
 							.newInstance();
-					// check if the type is a list, set, ...
 					add(fieldName,
 							(
 									final DBAccess ioDb,
@@ -486,24 +484,7 @@ public class CheckJPA<T> implements CheckFunctionInterface {
 									final QueryOptions options) -> {
 								// get the field of the specific element
 								final Object tmpData = field.get(data);
-								// It is not the objective of this element to check if it is authorize to set NULL
-								if (tmpData == null) {
-									return;
-								}
-								if (tmpData instanceof Collection) {
-									final Collection<?> tmpCollection = (Collection<?>) tmpData;
-									final Object[] elements = tmpCollection.toArray();
-									for (int iii = 0; iii < elements.length; iii++) {
-										if (elements[iii] != null) {
-											checkerInstance.check(ioDb, baseName + '.' + fieldName + '[' + iii + ']',
-													elements[iii], null, options);
-										}
-									}
-
-								} else {
-									checkerInstance.check(ioDb, baseName + '.' + fieldName, tmpData, null, options);
-								}
-
+								checkerInstance.check(ioDb, baseName, tmpData, null, options);
 							});
 				}
 				// keep this is last ==> take more time...