Thanks Jacques.

I've been thinking about this a little, and here is my suggestion:

<adjust-date-time from-field="fromDate" to-field="toDate"
  years="+1" months="+1" ... millis="+1"
  locale="parameters.locale" time-zone="parameters.timeZone"/>

The to-field attribute would be optional, as well as any of the adjusters. An <adjust-date-time> element that has no adjusters produces an error.

Adjustments would be performed using the minilang context's locale and time zone, unless the locale and time-zone attributes are used.

Any thoughts?

-Adrian

Jacques Le Roux wrote:

Thanks Adrian,

I put your comment and Chris's in the issue. Hopefully Fabien will take care of 
it (propose somehting), else we will see later...

Jacques

De : "Adrian Crum" <[EMAIL PROTECTED]>

I agree with Chris. I haven't looked at the issue lately, but when I first 
looked at it, it simply performed millisecond

arithmetic. That might not be the expected behavior.

Date calculation would be a great addition to mini language. Maybe we should 
discuss how it would look in mini lang code, then an

implementation could be submitted.

-Adrian

Jacques Le Roux <[EMAIL PROTECTED]> wrote: Thanks Chris,

Interesting remark, seems that the disussion begins to roll...

Jacques

De : "Chris Howe"

This really needs to go through a Calendar object instead of being cast to a 
long.  Also, I don't think the current

element structure is best suited for the operations that would be most 
beneficial for date/time calculations

----- Original Message ----
From: Jacques Le Roux
To: [email protected]
Sent: Saturday, December 8, 2007 5:37:08 PM
Subject: Calculate on a DATE object in minilang


Hi All,

3 months ago Fabien Carrion has contribued some interesting patches.
One of them is https://issues.apache.org/jira/browse/OFBIZ-1249
David asked <
do a design and requirements brainstorming discussion on the dev list
before putting this in.
The main thing I'm wondering is if there are other similar requirements
that could be satisfied by small changes in the design of this.>>

I would like to commit this patch now except if we begin to exchange
about new requirements per David suggestion

Thanks

Jacques






---------------------------------
Looking for last minute shopping deals?  Find them fast with Yahoo! Search.




Reply via email to