From 2e39e18c8ab9a7f00d77dd95bfed9916f240f273 Mon Sep 17 00:00:00 2001
From: Edwin Betancourt <EdwinBetanc0urt@outlook.com>
Date: Tue, 30 Mar 2021 14:17:51 -0400
Subject: [PATCH] fix: Empty mandatory fields. (#699)

Co-authored-by: EdwinBetanc0urt <EdwinBetanco0urt@outlook.com>
---
 src/store/modules/ADempiere/panel/getters.js  |  3 ++-
 .../ADempiere/windowControl/actions.js        | 22 ++++++++++---------
 2 files changed, 14 insertions(+), 11 deletions(-)

diff --git a/src/store/modules/ADempiere/panel/getters.js b/src/store/modules/ADempiere/panel/getters.js
index ef64629b..9825ffd2 100644
--- a/src/store/modules/ADempiere/panel/getters.js
+++ b/src/store/modules/ADempiere/panel/getters.js
@@ -102,8 +102,9 @@ const getters = {
       }
     })
 
-    if (!isEmptyValue(formatReturn)) {
+    if (formatReturn) {
       return fieldsNameEmpty.map(fieldItem => {
+        // fieldItem.name by default
         return fieldItem[formatReturn]
       })
     }
diff --git a/src/store/modules/ADempiere/windowControl/actions.js b/src/store/modules/ADempiere/windowControl/actions.js
index beae5b0a..509ee69f 100644
--- a/src/store/modules/ADempiere/windowControl/actions.js
+++ b/src/store/modules/ADempiere/windowControl/actions.js
@@ -842,20 +842,22 @@ export default {
       }).value
     }
 
-    if (isReference) {
-      if (!isEmptyValue(parsedWhereClause)) {
-        parsedWhereClause += ` AND ${referenceWhereClause}`
-      } else {
-        parsedWhereClause += referenceWhereClause
+    const addWhereClause = (currentWhereClause, newWhereClause) => {
+      if (isEmptyValue(currentWhereClause)) {
+        return newWhereClause
       }
+      if (isEmptyValue(newWhereClause)) {
+        return currentWhereClause
+      }
+      return `${currentWhereClause} AND ${newWhereClause}`
+    }
+
+    if (isReference) {
+      parsedWhereClause = addWhereClause(parsedWhereClause, referenceWhereClause)
     }
 
     if (!isEmptyValue(criteria)) {
-      if (!isEmptyValue(parsedWhereClause)) {
-        parsedWhereClause += ` AND ${criteria.whereClause}`
-      } else {
-        parsedWhereClause += criteria.whereClause
-      }
+      parsedWhereClause = addWhereClause(parsedWhereClause, criteria.whereClause)
     }
 
     const conditionsList = []