From ed4f3e77868cbc62a3f39fa620c9636fab699863 Mon Sep 17 00:00:00 2001
From: Carlos Ruiz <carg67@gmail.com>
Date: Fri, 24 Dec 2021 14:32:53 +0100
Subject: [PATCH] IDEMPIERE-5125 Opening/Close right panel cause DB Error
 (#1089)

---
 .../WEB-INF/src/org/adempiere/webui/util/UserPreference.java  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/util/UserPreference.java b/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/util/UserPreference.java
index 523b8ece1b..a7caa18aa4 100644
--- a/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/util/UserPreference.java
+++ b/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/util/UserPreference.java
@@ -23,6 +23,7 @@ import org.compiere.model.Query;
 import org.compiere.util.CLogger;
 import org.compiere.util.Env;
 import org.compiere.util.Language;
+import org.compiere.util.Util;
 
 /**
  *
@@ -110,7 +111,7 @@ public final class UserPreference implements Serializable {
 				String attribute = PROPERTIES[i];
 				String value = props.getProperty(attribute);
 
-				if (value != null) {
+				if (!Util.isEmpty(value)) {
 					MPreference preference = query.setParameters(new Object[]{m_AD_User_ID, attribute}).firstOnly();
 					if (preference == null) {
 						preference = new MUserPreference(Env.getCtx(), 0, null);
@@ -153,7 +154,6 @@ public final class UserPreference implements Serializable {
 				if (preference != null && preference.getValue() != null) {
 					value = preference.getValue();
 				}
-
 				props.setProperty(attribute, value);
 			}
 		}