This is an automated email from the ASF dual-hosted git repository.

nmalin pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/ofbiz-framework.git


The following commit(s) were added to refs/heads/trunk by this push:
     new 3fb6b40  Fixed: Clean groovy file FindLookUp No functional change :  * 
remove case syntax  * use groovy syntax (excepted on executeResult.listIt, 
EntityListIterator not support the groovy syntax *if (!listIt)*)  * use Elvis 
operator for default values  * add typage to help ide and compilation
3fb6b40 is described below

commit 3fb6b40aa396561db9a87c9f35a28db3b038a70a
Author: Nicolas Malin <nicolas.ma...@nereide.fr>
AuthorDate: Tue Aug 17 16:24:54 2021 +0200

    Fixed: Clean groovy file FindLookUp
    No functional change :
     * remove case syntax
     * use groovy syntax (excepted on executeResult.listIt, EntityListIterator 
not support the groovy syntax *if (!listIt)*)
     * use Elvis operator for default values
     * add typage to help ide and compilation
---
 .../party/groovyScripts/party/FindLookUp.groovy    | 50 +++++++++-------------
 1 file changed, 21 insertions(+), 29 deletions(-)

diff --git a/applications/party/groovyScripts/party/FindLookUp.groovy 
b/applications/party/groovyScripts/party/FindLookUp.groovy
index c319692..a325f4e 100644
--- a/applications/party/groovyScripts/party/FindLookUp.groovy
+++ b/applications/party/groovyScripts/party/FindLookUp.groovy
@@ -17,46 +17,38 @@
  * under the License.
  */
 
-
-
-
 import org.apache.ofbiz.entity.condition.EntityCondition
 import org.apache.ofbiz.entity.condition.EntityOperator
 import org.apache.ofbiz.entity.util.EntityUtilProperties
 
-if (context.noConditionFind == null) {
-    context.noConditionFind = parameters.noConditionFind
-}
-if (context.noConditionFind == null) {
-    context.noConditionFind = EntityUtilProperties.getPropertyValue("widget", 
"widget.defaultNoConditionFind", delegator)
-}
-if (context.filterByDate == null) {
-    context.filterByDate = parameters.filterByDate
+if (! context.noConditionFind) {
+    context.noConditionFind = parameters.noConditionFind ?:
+            EntityUtilProperties.getPropertyValue("widget", 
"widget.defaultNoConditionFind", delegator)
 }
-prepareResult = runService('prepareFind', [entityName : context.entityName,
-                                           orderBy : context.orderBy,
-                                           inputFields : parameters,
-                                           filterByDate : context.filterByDate,
-                                           filterByDateValue : 
context.filterByDateValue,
-                                           userLogin : context.userLogin])
+Map prepareResult = runService('prepareFind', [entityName: context.entityName,
+                                               orderBy: context.orderBy,
+                                               inputFields: parameters,
+                                               filterByDate: 
context.filterByDate ?: parameters.filterByDate,
+                                               filterByDateValue: 
context.filterByDateValue,
+                                               userLogin: context.userLogin])
 
-exprList = [EntityCondition.makeCondition("statusId", null),
-            EntityCondition.makeCondition("statusId", 
EntityOperator.NOT_EQUAL, "PARTY_DISABLED")]
-statusPartyDisable = EntityCondition.makeCondition(exprList, EntityOperator.OR)
-entityConditionList = null
-if (prepareResult.entityConditionList != null) {
-    ConditionList = [prepareResult.entityConditionList, statusPartyDisable]
-    entityConditionList = EntityCondition.makeCondition(ConditionList)
+EntityCondition statusPartyDisable = EntityCondition.makeCondition(
+        [EntityCondition.makeCondition("statusId", null),
+         EntityCondition.makeCondition("statusId", EntityOperator.NOT_EQUAL, 
"PARTY_DISABLED")],
+        EntityOperator.OR)
+EntityCondition entityConditionList = null
+if (prepareResult.entityConditionList) {
+    entityConditionList = 
EntityCondition.makeCondition([prepareResult.entityConditionList, 
statusPartyDisable])
 } else if ("Y" == context.noConditionFind) {
     entityConditionList = statusPartyDisable
 }
 
-executeResult = runService('executeFind', [entityName : context.entityName,
-                                           orderByList : 
prepareResult.orderByList,
-                                           entityConditionList : 
entityConditionList,
-                                           noConditionFind : 
context.noConditionFind])
+Map executeResult = runService('executeFind', [entityName: context.entityName,
+                                               orderByList: 
prepareResult.orderByList,
+                                               entityConditionList: 
entityConditionList,
+                                               noConditionFind: 
context.noConditionFind])
 if (executeResult.listIt == null) {
-    logWarning("No list found for query string + [" + 
prepareResult.queryString + "]")
+    logWarning("No list found for query string [${prepareResult.queryString}]")
 }
 context.listIt = executeResult.listIt
 context.queryString = prepareResult.queryString

Reply via email to