Close #232

Author: Open Lowcode SAS

src/org/openlowcode/server/data/properties/Hasmultidimensionalchild.java

@@ -108,7 +108,7 @@ F, E>> void postprocStoredobjectInsert(
 	 */
 	public void addlines(E object, F[] newlines) {
 		MultidimensionchildHelper<F, E> helper = this.casteddefinition.getHelper();
-
+		helper.setContext(object);
 		// get existing children
 		F[] previouschildren = this.casteddefinition.getChildren(object.getId());
 		// check if new line templates collide with existing, delete the ones that exist
@@ -120,13 +120,17 @@ public void addlines(E object, F[] newlines) {
 		}
 		ArrayList<F> allobjectstoinsert = new ArrayList<F>();
 		for (int i = 0; i < newlines.length; i++) {
-			ArrayList<F> missingforthisoptional = helper.getOtherPrimaryelements(newlines[i], childrenbykey);
+			F thisline = newlines[i];
+			boolean invalid = helper.isInvalid(thisline);
+			if (!invalid) {
+ 			ArrayList<F> missingforthisoptional = helper.getOtherPrimaryelements(thisline, childrenbykey);
 			for (int j = 0; j < missingforthisoptional.size(); j++) {
 				F thismissing = missingforthisoptional.get(j);
 				allobjectstoinsert.add(thismissing);
 				childrenbykey.put(helper.generateKeyForObject(thismissing), thismissing);
 
 			}
+			}
 		}
 
 		for (int i = 0; i < allobjectstoinsert.size(); i++)

src/org/openlowcode/server/data/properties/multichild/MultichildValueHelper.java

@@ -592,12 +592,13 @@ public boolean isValid(E optionalorinvalid) {
 		if (this.allowothervalues())
 			return true;
 		F value = getter.apply(optionalorinvalid);
+		if (value==null) return false;
 		F[] mandatoryvalues = this.getMandatoryValues();
 		F[] optionalvalues = this.getOptionalValues();
-		for (int i = 0; i < mandatoryvalues.length; i++)
+		if (mandatoryvalues!=null) for (int i = 0; i < mandatoryvalues.length; i++)
 			if (value.equals(mandatoryvalues[i]))
 				return true;
-		for (int i = 0; i < optionalvalues.length; i++)
+		if (optionalvalues!=null) for (int i = 0; i < optionalvalues.length; i++)
 			if (value.equals(optionalvalues[i]))
 				return true;
 		return false;
@@ -612,10 +613,10 @@ public boolean isTextValid(
 			return true;
 		F[] mandatoryvalues = this.getMandatoryValues();
 		F[] optionalvalues = this.getOptionalValues();
-		for (int i = 0; i < mandatoryvalues.length; i++)
+		if (mandatoryvalues!=null) for (int i = 0; i < mandatoryvalues.length; i++)
 			if (payload.equals(mandatoryvalues[i]))
 				return true;
-		for (int i = 0; i < optionalvalues.length; i++)
+		if (optionalvalues!=null) for (int i = 0; i < optionalvalues.length; i++)
 			if (payload.equals(optionalvalues[i]))
 				return true;
 		return false;