Merge pull request #4793 from Tyoneb/fix-issue-4723

Fix row data update workflow to properly trigger cells update events of uninitialized cells (#4723)

Author: azmy60

src/js/core/row/Row.js

@@ -267,15 +267,12 @@ export default class Row extends CoreFeature{
 			}
 
 			newRowData = this.chain("row-data-changing", [this, tempData, updatedData], null, updatedData);
-			
-			//set data
-			for (let attrname in newRowData) {
-				this.data[attrname] = newRowData[attrname];
-			}
-			
-			this.dispatch("row-data-save-after", this);
-			
-			//update affected cells only
+
+			// compute cells to update
+			// This must be done prior to updating the row data otherwise uninitialized cells get
+			// generated directly with the updated data, which prevents the run of callbacks
+			// registered on cells updates (e.g. mutators)
+			const cellsToUpdate = [];
 			for (let attrname in updatedData) {
 
 				let columns = this.table.columnManager.getColumnsByFieldRoot(attrname);
@@ -286,16 +283,28 @@ export default class Row extends CoreFeature{
 					if(cell){
 						let value = column.getFieldValue(newRowData);
 						if(cell.getValue() !== value){
-							cell.setValueProcessData(value);
-							
-							if(visible){
-								cell.cellRendered();
-							}
+							cellsToUpdate.push([cell, value]);
 						}
 					}
 				});
 			}
 
+			//set data
+			for (let attrname in newRowData) {
+				this.data[attrname] = newRowData[attrname];
+			}
+			
+			this.dispatch("row-data-save-after", this);
+			
+			//update affected cells only
+			cellsToUpdate.forEach(([cell, value]) => {
+				cell.setValueProcessData(value);
+							
+				if(visible){
+					cell.cellRendered();
+				}
+			});
+			
 			//Partial reinitialization if visible
 			if(visible){
 				this.normalizeHeight(true);