[FIX] dataJson checker is not responsive for null data pointer

This commit is contained in:
Edouard DUPIN 2025-01-25 12:16:13 +01:00
parent 9dad14d200
commit 0d419f651e

View File

@ -5,6 +5,7 @@ import java.sql.Timestamp;
import java.time.LocalDate; import java.time.LocalDate;
import java.time.LocalTime; import java.time.LocalTime;
import java.util.ArrayList; import java.util.ArrayList;
import java.util.Collection;
import java.util.Date; import java.util.Date;
import java.util.HashMap; import java.util.HashMap;
import java.util.List; import java.util.List;
@ -475,6 +476,7 @@ public class CheckJPA<T> implements CheckFunctionInterface {
if (dataJson != null && dataJson.checker() != null) { if (dataJson != null && dataJson.checker() != null) {
final CheckFunctionInterface checkerInstance = dataJson.checker().getDeclaredConstructor() final CheckFunctionInterface checkerInstance = dataJson.checker().getDeclaredConstructor()
.newInstance(); .newInstance();
// check if the type is a list, set, ...
add(fieldName, add(fieldName,
( (
final DBAccess ioDb, final DBAccess ioDb,
@ -484,7 +486,24 @@ public class CheckJPA<T> implements CheckFunctionInterface {
final QueryOptions options) -> { final QueryOptions options) -> {
// get the field of the specific element // get the field of the specific element
final Object tmpData = field.get(data); final Object tmpData = field.get(data);
checkerInstance.check(ioDb, baseName, tmpData, null, options); // 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);
}
}); });
} }
// keep this is last ==> take more time... // keep this is last ==> take more time...