From e477dad1c0119285222b7e66f0fc2221600da477 Mon Sep 17 00:00:00 2001
From: Zuhri Utama <zuhriutama@gmail.com>
Date: Thu, 6 Jul 2023 11:14:23 +0700
Subject: [PATCH] IDEMPIERE-2706 bug fix when create record from quick entry
 with default value (#1924)

---
 .../org/adempiere/webui/grid/WQuickEntry.java | 32 +++++++++++--------
 1 file changed, 18 insertions(+), 14 deletions(-)

diff --git a/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/grid/WQuickEntry.java b/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/grid/WQuickEntry.java
index eff277097e..c7d675a0e1 100644
--- a/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/grid/WQuickEntry.java
+++ b/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/grid/WQuickEntry.java
@@ -399,15 +399,7 @@ public class WQuickEntry extends AbstractWQuickEntry implements EventListener<Ev
 		log.config("");
 		boolean anyChange = false;
 		for (int idxf = 0; idxf < quickEditors.size(); idxf++) {
-			WEditor editor = quickEditors.get(idxf);
-			Object value = editor.getValue();
-			Object initialValue = initialValues.get(idxf);
-
-			boolean changed = (value != null && initialValue == null)
-					|| (value == null && initialValue != null)
-					|| (value != null && initialValue != null && ! value.equals(initialValue));
-
-			if (changed) {
+			if (isChanged(idxf)) {
 				anyChange = true;
 			}
 		}
@@ -444,11 +436,7 @@ public class WQuickEntry extends AbstractWQuickEntry implements EventListener<Ev
 
 				WEditor editor = quickEditors.get(idxf);
 				Object value = editor.getValue();
-				Object initialValue = initialValues.get(idxf);
 
-				boolean changed = (value != null && initialValue == null)
-						|| (value == null && initialValue != null)
-						|| (value != null && initialValue != null && !value.equals(initialValue));
 				boolean thisMandatoryError = false;
 				if (field.isMandatory(true)) {
 					if (value == null || value.toString().length() == 0) {
@@ -461,7 +449,7 @@ public class WQuickEntry extends AbstractWQuickEntry implements EventListener<Ev
 				}
 
 				po.set_ValueOfColumn(field.getColumnName(), value);
-				if (changed && ! thisMandatoryError) {
+				if (isChanged(idxf) && ! thisMandatoryError) {
 					savePO = true;
 				}
 			}
@@ -501,6 +489,22 @@ public class WQuickEntry extends AbstractWQuickEntry implements EventListener<Ev
 		}
 		return true;
 	}	//	actionSave
+	
+	/**
+	 * 
+	 * @param index of field
+	 * @return true if changed
+	 */
+	private boolean isChanged(int index)
+	{
+		WEditor editor = quickEditors.get(index);
+		Object value = editor.getValue();
+		Object initialValue = initialValues.get(index);
+		
+		return (value != null && initialValue == null)
+				|| (value == null && initialValue != null)
+				|| (value != null && initialValue != null);
+	}
 
 	/**
 	 *	Returns Record_ID