Extra stability fix for #169

Author: Open Lowcode SAS

src/org/openlowcode/client/graphic/widget/CGrid.java

@@ -1111,7 +1111,7 @@ public DataElt getDataElt(DataEltType type, String eltname, String objectdataloc
 
 	@Override
 	public void forceUpdateData(DataElt dataelt) {
-		reviewDataWarningForGrid();
+		
 
 		if (!(dataelt instanceof ArrayDataElt))
 			throw new RuntimeException(
@@ -1124,27 +1124,33 @@ public void forceUpdateData(DataElt dataelt) {
 			updatedrowsbyid.put(thiselementarray.getObjectAtIndex(i).getUID(), thiselementarray.getObjectAtIndex(i));
 		}
 		int updated = 0;
-		for (int i = 0; i < dataingrid.getSize(); i++) {
-			CObjectGridLine<String> currentline = dataingrid.get(i);
-			for (int j = 0; j < currentline.getObjectinlineNumber(); j++) {
-				ObjectInGrid object = currentline.getObjectinline(j);
-				String uid = object.getObject().getUID();
-				ObjectDataElt relevantobject = updatedrowsbyid.get(uid);
-				logger.finest("adding new object through force update data - Adding to Grid Line "
-						+ currentline.getLineLabel() + " (" + currentline.hashCode() + ") " + relevantobject.hashCode()
-						+ " - " + relevantobject.lookupEltByName("ID") + " - "
-						+ relevantobject.lookupEltByName("YEARALLOCATED"));
-				if (relevantobject != null) {
-					object.forceUpdatedObject(relevantobject);
-					updated++;
-				}
+		if (!this.reversetree) {
+			for (int i = 0; i < dataingrid.getSize(); i++) {
+				CObjectGridLine<String> currentline = dataingrid.get(i);
+				for (int j = 0; j < currentline.getObjectinlineNumber(); j++) {
+					ObjectInGrid object = currentline.getObjectinline(j);
+					String uid = object.getObject().getUID();
+					ObjectDataElt relevantobject = updatedrowsbyid.get(uid);
+					logger.finest("adding new object through force update data - Adding to Grid Line "
+							+ currentline.getLineLabel() + " (" + currentline.hashCode() + ") "
+							+ relevantobject.hashCode() + " - " + relevantobject.lookupEltByName("ID") + " - "
+							+ relevantobject.lookupEltByName("YEARALLOCATED"));
+					if (relevantobject != null) {
+						object.forceUpdatedObject(relevantobject);
+						updated++;
+					}
 
+				}
 			}
+			if (updated != thiselementarray.getObjectNumber())
+				actionmanager.getClientSession().getActiveClientDisplay()
+						.updateStatusBar("Received " + thiselementarray.getObjectNumber() + " elements, but only "
+								+ updated + " could be updated on the page", true);
+		} else {
+			logger.severe("Tree Table Resetunmarked items");
+			treetable.unMarkUpdatedItems();
 		}
-		if (updated != thiselementarray.getObjectNumber())
-			actionmanager.getClientSession().getActiveClientDisplay()
-					.updateStatusBar("Received " + thiselementarray.getObjectNumber() + " elements, but only " + updated
-							+ " could be updated on the page", true);
+		reviewDataWarningForGrid();
 	}
 
 	@Override
@@ -1246,7 +1252,9 @@ public void reviewDataWarningForGrid() {
 						updatedrow++;
 				}
 			} else {
+				
 				updatedrow = this.treetable.getUpdatedItems().size();
+				logger.severe("Get all updated rows, result = "+updatedrow);
 			}
 			if (updatedrow == 0) {
 				// remove any present unsaved data warning for this component

src/org/openlowcode/client/graphic/widget/table/EditableTreeTable.java

@@ -166,6 +166,13 @@ public List<E> getUpdatedItems() {
 		}
 		return updateditems;
 	}
+	
+	public void unMarkUpdatedItems() {
+		for (int i = 0; i < this.wrappedpayload.size(); i++) {
+			Wrapper<E> item = this.wrappedpayload.get(i);
+			item.setNotUpdated();
+		}
+	}
 
 	/**
 	 * defines a line grouping based on the value of a given field of the payload
@@ -1298,6 +1305,11 @@ public Wrapper(E payload) {
 			this.payload = payload;
 		}
 
+		public void setNotUpdated() {
+			this.updated=false;
+			
+		}
+
 		public E getPayload() {
 			return payload;
 		}