From aef93fb113fb0257c0f494db69172bb0ec0c5c1a Mon Sep 17 00:00:00 2001
From: Carlos Ruiz <carg67@gmail.com>
Date: Fri, 23 Sep 2022 09:25:51 +0200
Subject: [PATCH] IDEMPIERE-5430 UPDATE generating unnecessary settings for
 null columns (#1496)

---
 org.adempiere.base/src/org/compiere/model/MField.java | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/org.adempiere.base/src/org/compiere/model/MField.java b/org.adempiere.base/src/org/compiere/model/MField.java
index eb965777c4..aa23d93cc0 100644
--- a/org.adempiere.base/src/org/compiere/model/MField.java
+++ b/org.adempiere.base/src/org/compiere/model/MField.java
@@ -210,9 +210,12 @@ public class MField extends X_AD_Field implements ImmutablePOSupport
 				setIsAllowCopy("Y");
 		}
 		if (getAD_Reference_ID() <= 0) {
-			setAD_Reference_Value_ID(0);
-			setAD_Val_Rule_ID(0);
-			setIsToolbarButton(null);
+			if (getAD_Reference_Value_ID()!=0)
+				setAD_Reference_Value_ID(0);
+			if (getAD_Val_Rule_ID()!=0)
+				setAD_Val_Rule_ID(0);
+			if (getIsToolbarButton() != null)
+				setIsToolbarButton(null);
 		}
 
 		//validate logic expression