You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by Nicolas Malin <ni...@nereide.fr> on 2016/06/24 07:14:14 UTC

Re: svn commit: r1750045 - /ofbiz/trunk/applications/party/widget/partymgr/LookupScreens.xml

Le 24/06/2016 09:00, jleroux@apache.org a �crit :
> +                <set field="andCondition" value="${groovy: return org.ofbiz.entity.condition.EntityCondition.makeCondition([
> +                    org.ofbiz.entity.condition.EntityCondition.makeCondition('roleTypeId', 'CONTACT')])}"/>
This commit is fine and when I read it, I ask myself if a dsl groovy 
syntax would be define like as example

${groovy: entity condition make([['roleTypeId' : 'CONTACT']]}

${groovy: entity condition make([andCondition, ['roleTypeId' : 'CONTACT']]}

${groovy: entity condition make([andCondition, make ('roleTypeId', 
NOT_EQUALS,  'CONTACT')]}

It's completly open idea

Nicoals

Re: svn commit: r1750045 - /ofbiz/trunk/applications/party/widget/partymgr/LookupScreens.xml

Posted by Jacques Le Roux <ja...@les7arts.com>.
+1 for 1 option (yes, I'm lazy). Of course people preferring the std groovy snippet could continue to use it :p

Jacques


Le 24/06/2016 � 09:14, Nicolas Malin a �crit :
> Le 24/06/2016 09:00, jleroux@apache.org a �crit :
>> +                <set field="andCondition" value="${groovy: return org.ofbiz.entity.condition.EntityCondition.makeCondition([
>> + org.ofbiz.entity.condition.EntityCondition.makeCondition('roleTypeId', 'CONTACT')])}"/>
> This commit is fine and when I read it, I ask myself if a dsl groovy syntax would be define like as example
>
> ${groovy: entity condition make([['roleTypeId' : 'CONTACT']]}
>
> ${groovy: entity condition make([andCondition, ['roleTypeId' : 'CONTACT']]}
>
> ${groovy: entity condition make([andCondition, make ('roleTypeId', NOT_EQUALS,  'CONTACT')]}
>
> It's completly open idea
>
> Nicoals
>