Discussion: Adding the Unified Expression Language to the framework

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
10 messages Options
Reply | Threaded
Open this post in threaded view
|

Discussion: Adding the Unified Expression Language to the framework

Adrian Crum
I'm considering integrating JUEL (http://juel.sourceforge.net) into the
FlexibleStringExpander class and maybe the FlexibleMapAccessor class.
Jacopo had expressed a desire to integrate Apache Commons EL in this
manner
(http://docs.ofbiz.org/display/OFBADMIN/New+Features+Roadmap+-+Living+Document),
but the Commons EL project is based on an older EL specification.

Making this change would allow expressions like:

"Order balance: ${order.amount - payment.amount}"

anywhere the FlexibleStringExpander is used (simple methods, screen
widgets).

What do you think?

-Adrian
Reply | Threaded
Open this post in threaded view
|

Re: Discussion: Adding the Unified Expression Language to the framework

Jacques Le Roux
Administrator
Sounds fine

Jacques

From: "Adrian Crum" <[hidden email]>

> I'm considering integrating JUEL (http://juel.sourceforge.net) into the
> FlexibleStringExpander class and maybe the FlexibleMapAccessor class.
> Jacopo had expressed a desire to integrate Apache Commons EL in this
> manner
> (http://docs.ofbiz.org/display/OFBADMIN/New+Features+Roadmap+-+Living+Document),
> but the Commons EL project is based on an older EL specification.
>
> Making this change would allow expressions like:
>
> "Order balance: ${order.amount - payment.amount}"
>
> anywhere the FlexibleStringExpander is used (simple methods, screen
> widgets).
>
> What do you think?
>
> -Adrian
>
Reply | Threaded
Open this post in threaded view
|

Re: Discussion: Adding the Unified Expression Language to the framework

Adrian Crum-2
In reply to this post by Adrian Crum
Okay, I have this working but I haven't committed it yet because I'm not clear on the licensing issues. The JUEL library is Apache 2 license, but it depends on the javax.el.* API sources which is Sun CDDL 1.0.

Could someone well-versed in licenses help me out here?

-Adrian


--- On Thu, 12/4/08, Adrian Crum <[hidden email]> wrote:

> From: Adrian Crum <[hidden email]>
> Subject: Discussion: Adding the Unified Expression Language to the framework
> To: [hidden email]
> Date: Thursday, December 4, 2008, 11:08 AM
> I'm considering integrating JUEL
> (http://juel.sourceforge.net) into the
> FlexibleStringExpander class and maybe the
> FlexibleMapAccessor class. Jacopo had expressed a desire to
> integrate Apache Commons EL in this manner
> (http://docs.ofbiz.org/display/OFBADMIN/New+Features+Roadmap+-+Living+Document),
> but the Commons EL project is based on an older EL
> specification.
>
> Making this change would allow expressions like:
>
> "Order balance: ${order.amount - payment.amount}"
>
> anywhere the FlexibleStringExpander is used (simple
> methods, screen widgets).
>
> What do you think?
>
> -Adrian


     
Reply | Threaded
Open this post in threaded view
|

Re: Discussion: Adding the Unified Expression Language to the framework

David E Jones-3

Is there an AL2 version of the interfaces for javax.el around?  
Geronimo has a lot of these implemented, but for something like this  
another project may be better, perhaps even the Commons EL project. I  
know you mentioned Commons EL is based on an older version, but  
perhaps they have some partial effort on a newer one? Like at least an  
API for it?

-David


On Dec 5, 2008, at 4:38 PM, Adrian Crum wrote:

> Okay, I have this working but I haven't committed it yet because I'm  
> not clear on the licensing issues. The JUEL library is Apache 2  
> license, but it depends on the javax.el.* API sources which is Sun  
> CDDL 1.0.
>
> Could someone well-versed in licenses help me out here?
>
> -Adrian
>
>
> --- On Thu, 12/4/08, Adrian Crum <[hidden email]> wrote:
>
>> From: Adrian Crum <[hidden email]>
>> Subject: Discussion: Adding the Unified Expression Language to the  
>> framework
>> To: [hidden email]
>> Date: Thursday, December 4, 2008, 11:08 AM
>> I'm considering integrating JUEL
>> (http://juel.sourceforge.net) into the
>> FlexibleStringExpander class and maybe the
>> FlexibleMapAccessor class. Jacopo had expressed a desire to
>> integrate Apache Commons EL in this manner
>> (http://docs.ofbiz.org/display/OFBADMIN/New+Features+Roadmap+-+Living+Document 
>> ),
>> but the Commons EL project is based on an older EL
>> specification.
>>
>> Making this change would allow expressions like:
>>
>> "Order balance: ${order.amount - payment.amount}"
>>
>> anywhere the FlexibleStringExpander is used (simple
>> methods, screen widgets).
>>
>> What do you think?
>>
>> -Adrian
>
>
>

Reply | Threaded
Open this post in threaded view
|

Re: Discussion: Adding the Unified Expression Language to the framework

David E Jones-3

I just took a peek and it appears that jar files that are CDDL 1.0  
licensed are allowed:

http://www.apache.org/legal/3party.html

-David


On Dec 7, 2008, at 12:22 AM, David E Jones wrote:

>
> Is there an AL2 version of the interfaces for javax.el around?  
> Geronimo has a lot of these implemented, but for something like this  
> another project may be better, perhaps even the Commons EL project.  
> I know you mentioned Commons EL is based on an older version, but  
> perhaps they have some partial effort on a newer one? Like at least  
> an API for it?
>
> -David
>
>
> On Dec 5, 2008, at 4:38 PM, Adrian Crum wrote:
>
>> Okay, I have this working but I haven't committed it yet because  
>> I'm not clear on the licensing issues. The JUEL library is Apache 2  
>> license, but it depends on the javax.el.* API sources which is Sun  
>> CDDL 1.0.
>>
>> Could someone well-versed in licenses help me out here?
>>
>> -Adrian
>>
>>
>> --- On Thu, 12/4/08, Adrian Crum <[hidden email]> wrote:
>>
>>> From: Adrian Crum <[hidden email]>
>>> Subject: Discussion: Adding the Unified Expression Language to the  
>>> framework
>>> To: [hidden email]
>>> Date: Thursday, December 4, 2008, 11:08 AM
>>> I'm considering integrating JUEL
>>> (http://juel.sourceforge.net) into the
>>> FlexibleStringExpander class and maybe the
>>> FlexibleMapAccessor class. Jacopo had expressed a desire to
>>> integrate Apache Commons EL in this manner
>>> (http://docs.ofbiz.org/display/OFBADMIN/New+Features+Roadmap+-+Living+Document 
>>> ),
>>> but the Commons EL project is based on an older EL
>>> specification.
>>>
>>> Making this change would allow expressions like:
>>>
>>> "Order balance: ${order.amount - payment.amount}"
>>>
>>> anywhere the FlexibleStringExpander is used (simple
>>> methods, screen widgets).
>>>
>>> What do you think?
>>>
>>> -Adrian
>>
>>
>>
>

Reply | Threaded
Open this post in threaded view
|

Re: Discussion: Adding the Unified Expression Language to the framework

David E Jones-3
In reply to this post by Adrian Crum

This looks pretty cool. The "${bsh:" stuff may need to be pre-filtered.

Also, I noticed that a pre-parse or cache plugin feature is supported,  
and for something like this it would make a big difference.

-David


On Dec 4, 2008, at 12:08 PM, Adrian Crum wrote:

> I'm considering integrating JUEL (http://juel.sourceforge.net) into  
> the FlexibleStringExpander class and maybe the FlexibleMapAccessor  
> class. Jacopo had expressed a desire to integrate Apache Commons EL  
> in this manner (http://docs.ofbiz.org/display/OFBADMIN/New+Features+Roadmap+-+Living+Document 
> ), but the Commons EL project is based on an older EL specification.
>
> Making this change would allow expressions like:
>
> "Order balance: ${order.amount - payment.amount}"
>
> anywhere the FlexibleStringExpander is used (simple methods, screen  
> widgets).
>
> What do you think?
>
> -Adrian

Reply | Threaded
Open this post in threaded view
|

Re: Discussion: Adding the Unified Expression Language to the framework

Adrian Crum-2
In reply to this post by David E Jones-3
I spent some time checking into it, and it's nothing more than the Sun 1.0 license - which we have in the LICENSE file already.

I'm putting the finishing touches on it and will have it committed soon.

-Adrian


--- On Sat, 12/6/08, David E Jones <[hidden email]> wrote:

> From: David E Jones <[hidden email]>
> Subject: Re: Discussion: Adding the Unified Expression Language to the framework
> To: [hidden email]
> Date: Saturday, December 6, 2008, 11:22 PM
> Is there an AL2 version of the interfaces for javax.el
> around? Geronimo has a lot of these implemented, but for
> something like this another project may be better, perhaps
> even the Commons EL project. I know you mentioned Commons EL
> is based on an older version, but perhaps they have some
> partial effort on a newer one? Like at least an API for it?
>
> -David
>
>
> On Dec 5, 2008, at 4:38 PM, Adrian Crum wrote:
>
> > Okay, I have this working but I haven't committed
> it yet because I'm not clear on the licensing issues.
> The JUEL library is Apache 2 license, but it depends on the
> javax.el.* API sources which is Sun CDDL 1.0.
> >
> > Could someone well-versed in licenses help me out
> here?
> >
> > -Adrian
> >
> >
> > --- On Thu, 12/4/08, Adrian Crum
> <[hidden email]> wrote:
> >
> >> From: Adrian Crum <[hidden email]>
> >> Subject: Discussion: Adding the Unified Expression
> Language to the framework
> >> To: [hidden email]
> >> Date: Thursday, December 4, 2008, 11:08 AM
> >> I'm considering integrating JUEL
> >> (http://juel.sourceforge.net) into the
> >> FlexibleStringExpander class and maybe the
> >> FlexibleMapAccessor class. Jacopo had expressed a
> desire to
> >> integrate Apache Commons EL in this manner
> >>
> (http://docs.ofbiz.org/display/OFBADMIN/New+Features+Roadmap+-+Living+Document),
> >> but the Commons EL project is based on an older EL
> >> specification.
> >>
> >> Making this change would allow expressions like:
> >>
> >> "Order balance: ${order.amount -
> payment.amount}"
> >>
> >> anywhere the FlexibleStringExpander is used
> (simple
> >> methods, screen widgets).
> >>
> >> What do you think?
> >>
> >> -Adrian
> >
> >
> >


     
Reply | Threaded
Open this post in threaded view
|

Re: Discussion: Adding the Unified Expression Language to the framework

Adrian Crum-2
Committed, rev 724146.

-Adrian


--- On Sun, 12/7/08, Adrian Crum <[hidden email]> wrote:

> From: Adrian Crum <[hidden email]>
> Subject: Re: Discussion: Adding the Unified Expression Language to the framework
> To: [hidden email]
> Date: Sunday, December 7, 2008, 6:42 AM
> I spent some time checking into it, and it's nothing
> more than the Sun 1.0 license - which we have in the LICENSE
> file already.
>
> I'm putting the finishing touches on it and will have
> it committed soon.
>
> -Adrian
>
>
> --- On Sat, 12/6/08, David E Jones
> <[hidden email]> wrote:
>
> > From: David E Jones
> <[hidden email]>
> > Subject: Re: Discussion: Adding the Unified Expression
> Language to the framework
> > To: [hidden email]
> > Date: Saturday, December 6, 2008, 11:22 PM
> > Is there an AL2 version of the interfaces for javax.el
> > around? Geronimo has a lot of these implemented, but
> for
> > something like this another project may be better,
> perhaps
> > even the Commons EL project. I know you mentioned
> Commons EL
> > is based on an older version, but perhaps they have
> some
> > partial effort on a newer one? Like at least an API
> for it?
> >
> > -David
> >
> >
> > On Dec 5, 2008, at 4:38 PM, Adrian Crum wrote:
> >
> > > Okay, I have this working but I haven't
> committed
> > it yet because I'm not clear on the licensing
> issues.
> > The JUEL library is Apache 2 license, but it depends
> on the
> > javax.el.* API sources which is Sun CDDL 1.0.
> > >
> > > Could someone well-versed in licenses help me out
> > here?
> > >
> > > -Adrian
> > >
> > >
> > > --- On Thu, 12/4/08, Adrian Crum
> > <[hidden email]> wrote:
> > >
> > >> From: Adrian Crum <[hidden email]>
> > >> Subject: Discussion: Adding the Unified
> Expression
> > Language to the framework
> > >> To: [hidden email]
> > >> Date: Thursday, December 4, 2008, 11:08 AM
> > >> I'm considering integrating JUEL
> > >> (http://juel.sourceforge.net) into the
> > >> FlexibleStringExpander class and maybe the
> > >> FlexibleMapAccessor class. Jacopo had
> expressed a
> > desire to
> > >> integrate Apache Commons EL in this manner
> > >>
> >
> (http://docs.ofbiz.org/display/OFBADMIN/New+Features+Roadmap+-+Living+Document),
> > >> but the Commons EL project is based on an
> older EL
> > >> specification.
> > >>
> > >> Making this change would allow expressions
> like:
> > >>
> > >> "Order balance: ${order.amount -
> > payment.amount}"
> > >>
> > >> anywhere the FlexibleStringExpander is used
> > (simple
> > >> methods, screen widgets).
> > >>
> > >> What do you think?
> > >>
> > >> -Adrian
> > >
> > >
> > >


     
Reply | Threaded
Open this post in threaded view
|

Re: Discussion: Adding the Unified Expression Language to the framework

Adrian Crum-2
This is finished with rev 726260. I added the UEL to FlexibleMapAccessor so that l-values can have expressions.

Now we can do things like (in screen widget):

<actions>
  <set field="someList[]" value="Red"/>
  <set field="someList[]" value="Blue"/>
  <set field="someList[]" value="Green"/>
  <set field="blueIndex" value="1"/>
  <set field="someList[blueIndex]" value="Indigo"/>
</actions>
<widgets>
  <label text="${someList}"/>
  <!-- Displays [Red, Indigo, Green] -->
</widgets>

-Adrian


--- On Sun, 12/7/08, Adrian Crum <[hidden email]> wrote:

> From: Adrian Crum <[hidden email]>
> Subject: Re: Discussion: Adding the Unified Expression Language to the framework
> To: [hidden email]
> Date: Sunday, December 7, 2008, 8:12 AM
> Committed, rev 724146.
>
> -Adrian
>
>
> --- On Sun, 12/7/08, Adrian Crum
> <[hidden email]> wrote:
>
> > From: Adrian Crum <[hidden email]>
> > Subject: Re: Discussion: Adding the Unified Expression
> Language to the framework
> > To: [hidden email]
> > Date: Sunday, December 7, 2008, 6:42 AM
> > I spent some time checking into it, and it's
> nothing
> > more than the Sun 1.0 license - which we have in the
> LICENSE
> > file already.
> >
> > I'm putting the finishing touches on it and will
> have
> > it committed soon.
> >
> > -Adrian
> >
> >
> > --- On Sat, 12/6/08, David E Jones
> > <[hidden email]> wrote:
> >
> > > From: David E Jones
> > <[hidden email]>
> > > Subject: Re: Discussion: Adding the Unified
> Expression
> > Language to the framework
> > > To: [hidden email]
> > > Date: Saturday, December 6, 2008, 11:22 PM
> > > Is there an AL2 version of the interfaces for
> javax.el
> > > around? Geronimo has a lot of these implemented,
> but
> > for
> > > something like this another project may be
> better,
> > perhaps
> > > even the Commons EL project. I know you mentioned
> > Commons EL
> > > is based on an older version, but perhaps they
> have
> > some
> > > partial effort on a newer one? Like at least an
> API
> > for it?
> > >
> > > -David
> > >
> > >
> > > On Dec 5, 2008, at 4:38 PM, Adrian Crum wrote:
> > >
> > > > Okay, I have this working but I haven't
> > committed
> > > it yet because I'm not clear on the licensing
> > issues.
> > > The JUEL library is Apache 2 license, but it
> depends
> > on the
> > > javax.el.* API sources which is Sun CDDL 1.0.
> > > >
> > > > Could someone well-versed in licenses help
> me out
> > > here?
> > > >
> > > > -Adrian
> > > >
> > > >
> > > > --- On Thu, 12/4/08, Adrian Crum
> > > <[hidden email]> wrote:
> > > >
> > > >> From: Adrian Crum
> <[hidden email]>
> > > >> Subject: Discussion: Adding the Unified
> > Expression
> > > Language to the framework
> > > >> To: [hidden email]
> > > >> Date: Thursday, December 4, 2008, 11:08
> AM
> > > >> I'm considering integrating JUEL
> > > >> (http://juel.sourceforge.net) into the
> > > >> FlexibleStringExpander class and maybe
> the
> > > >> FlexibleMapAccessor class. Jacopo had
> > expressed a
> > > desire to
> > > >> integrate Apache Commons EL in this
> manner
> > > >>
> > >
> >
> (http://docs.ofbiz.org/display/OFBADMIN/New+Features+Roadmap+-+Living+Document),
> > > >> but the Commons EL project is based on
> an
> > older EL
> > > >> specification.
> > > >>
> > > >> Making this change would allow
> expressions
> > like:
> > > >>
> > > >> "Order balance: ${order.amount -
> > > payment.amount}"
> > > >>
> > > >> anywhere the FlexibleStringExpander is
> used
> > > (simple
> > > >> methods, screen widgets).
> > > >>
> > > >> What do you think?
> > > >>
> > > >> -Adrian
> > > >
> > > >
> > > >


     
Reply | Threaded
Open this post in threaded view
|

Re: Discussion: Adding the Unified Expression Language to the framework

Jacques Le Roux
Administrator
Thanks Adrian !

Jacques

From: "Adrian Crum" <[hidden email]>

> This is finished with rev 726260. I added the UEL to FlexibleMapAccessor so that l-values can have expressions.
>
> Now we can do things like (in screen widget):
>
> <actions>
>  <set field="someList[]" value="Red"/>
>  <set field="someList[]" value="Blue"/>
>  <set field="someList[]" value="Green"/>
>  <set field="blueIndex" value="1"/>
>  <set field="someList[blueIndex]" value="Indigo"/>
> </actions>
> <widgets>
>  <label text="${someList}"/>
>  <!-- Displays [Red, Indigo, Green] -->
> </widgets>
>
> -Adrian
>
>
> --- On Sun, 12/7/08, Adrian Crum <[hidden email]> wrote:
>
>> From: Adrian Crum <[hidden email]>
>> Subject: Re: Discussion: Adding the Unified Expression Language to the framework
>> To: [hidden email]
>> Date: Sunday, December 7, 2008, 8:12 AM
>> Committed, rev 724146.
>>
>> -Adrian
>>
>>
>> --- On Sun, 12/7/08, Adrian Crum
>> <[hidden email]> wrote:
>>
>> > From: Adrian Crum <[hidden email]>
>> > Subject: Re: Discussion: Adding the Unified Expression
>> Language to the framework
>> > To: [hidden email]
>> > Date: Sunday, December 7, 2008, 6:42 AM
>> > I spent some time checking into it, and it's
>> nothing
>> > more than the Sun 1.0 license - which we have in the
>> LICENSE
>> > file already.
>> >
>> > I'm putting the finishing touches on it and will
>> have
>> > it committed soon.
>> >
>> > -Adrian
>> >
>> >
>> > --- On Sat, 12/6/08, David E Jones
>> > <[hidden email]> wrote:
>> >
>> > > From: David E Jones
>> > <[hidden email]>
>> > > Subject: Re: Discussion: Adding the Unified
>> Expression
>> > Language to the framework
>> > > To: [hidden email]
>> > > Date: Saturday, December 6, 2008, 11:22 PM
>> > > Is there an AL2 version of the interfaces for
>> javax.el
>> > > around? Geronimo has a lot of these implemented,
>> but
>> > for
>> > > something like this another project may be
>> better,
>> > perhaps
>> > > even the Commons EL project. I know you mentioned
>> > Commons EL
>> > > is based on an older version, but perhaps they
>> have
>> > some
>> > > partial effort on a newer one? Like at least an
>> API
>> > for it?
>> > >
>> > > -David
>> > >
>> > >
>> > > On Dec 5, 2008, at 4:38 PM, Adrian Crum wrote:
>> > >
>> > > > Okay, I have this working but I haven't
>> > committed
>> > > it yet because I'm not clear on the licensing
>> > issues.
>> > > The JUEL library is Apache 2 license, but it
>> depends
>> > on the
>> > > javax.el.* API sources which is Sun CDDL 1.0.
>> > > >
>> > > > Could someone well-versed in licenses help
>> me out
>> > > here?
>> > > >
>> > > > -Adrian
>> > > >
>> > > >
>> > > > --- On Thu, 12/4/08, Adrian Crum
>> > > <[hidden email]> wrote:
>> > > >
>> > > >> From: Adrian Crum
>> <[hidden email]>
>> > > >> Subject: Discussion: Adding the Unified
>> > Expression
>> > > Language to the framework
>> > > >> To: [hidden email]
>> > > >> Date: Thursday, December 4, 2008, 11:08
>> AM
>> > > >> I'm considering integrating JUEL
>> > > >> (http://juel.sourceforge.net) into the
>> > > >> FlexibleStringExpander class and maybe
>> the
>> > > >> FlexibleMapAccessor class. Jacopo had
>> > expressed a
>> > > desire to
>> > > >> integrate Apache Commons EL in this
>> manner
>> > > >>
>> > >
>> >
>> (http://docs.ofbiz.org/display/OFBADMIN/New+Features+Roadmap+-+Living+Document),
>> > > >> but the Commons EL project is based on
>> an
>> > older EL
>> > > >> specification.
>> > > >>
>> > > >> Making this change would allow
>> expressions
>> > like:
>> > > >>
>> > > >> "Order balance: ${order.amount -
>> > > payment.amount}"
>> > > >>
>> > > >> anywhere the FlexibleStringExpander is
>> used
>> > > (simple
>> > > >> methods, screen widgets).
>> > > >>
>> > > >> What do you think?
>> > > >>
>> > > >> -Adrian
>> > > >
>> > > >
>> > > >
>
>
>      
>