fix: Infinity loop on country field of location form. (adempiere#913)

src/store/modules/ADempiere/defaultValueManager.js

@@ -152,10 +152,9 @@ const defaultValueManager = {
             }
 
             // do not use the convertArrayKeyValueToObject method to avoid losing a key with an empty value
-            // TODO: Response from server same name key of value
             if (valueResponse.attributes.length === 1) {
+              // number values (Line for example)
               values.KeyColumn = valueResponse.attributes.at(0).value
-              values.DisplayColumn = undefined
             } else {
               valueResponse.attributes.forEach(attribute => {
                 const { key: column, value: attributeValue } = attribute
@@ -172,7 +171,8 @@ const defaultValueManager = {
               contextAttributesList,
               id,
               displayedValue,
-              value,
+              // set value of server to parsed if is number as string "101" -> 101
+              value: valueOfServer,
               uuid: values.UUID
             })
 

src/store/modules/ADempiere/lookupManager.js

@@ -17,6 +17,7 @@
  */
 
 import Vue from 'vue'
+
 import lang from '@/lang'
 
 // API Request Methods
@@ -335,24 +336,27 @@ const lookupManager = {
       })
 
       // get of stored default value
-      if (isEmptyValue(optionsList)) {
-        const contextAttributesListByDefaultValue = getContextAttributes({
-          parentUuid,
-          containerUuid,
-          contextColumnNames: contextColumnNamesByDefaultValue,
-          isBooleanToString: true
-        })
-        const option = rootGetters.getStoredDefaultValue({
-          parentUuid,
-          containerUuid,
-          contextColumnNames: contextColumnNamesByDefaultValue,
-          contextAttributesList: contextAttributesListByDefaultValue,
-          uuid,
-          value
-        })
+      const contextAttributesListByDefaultValue = getContextAttributes({
+        parentUuid,
+        containerUuid,
+        contextColumnNames: contextColumnNamesByDefaultValue,
+        isBooleanToString: true
+      })
+      const option = rootGetters.getStoredDefaultValue({
+        parentUuid,
+        containerUuid,
+        contextColumnNames: contextColumnNamesByDefaultValue,
+        contextAttributesList: contextAttributesListByDefaultValue,
+        uuid,
+        value
+      })
 
-        // add a item option
-        if (!isEmptyValue(option)) {
+      // add a item option
+      if (!isEmptyValue(option)) {
+        const isExists = optionsList.some(optionItem => {
+          return optionItem.value === option.value
+        })
+        if (!isExists) {
           optionsList.push(option)
         }
       }