> > Although the IntakeService in Fulcrum contains a DateString class
this
> > doesn't seem to be present in Turbine-2.2. This patch addresses that
problem
> >
> > 1. DateStringField has been copied from Fulcrum and adapted for
Turbine-2.2
> > 2. DateStringValidator has been copied from Fulcrum and adapted for
> > Turbine-2.2
> > 3. FieldFactory has been adapted to work with DateString. Hereby a
diff
> >
> > Could someone check in these three files please?
> 
> done
> 
> thanx!
> 
> martin

Could we rename DateString to just Date? I think it's more
representative of what it actually is...because the DateString
functionality targets Date properties on objects, not String properties
that are in a date format. E.g. if you have an integer property, you use
"int" not "IntegerString".

Also note that Date matches the existing DTD:
http://jakarta.apache.org/turbine/dtd/intake.dtd.

I'd be happy to submit the patches (for both Fulcrum and Turbine-2) if
it'll be committed.

Thanks,
Stephen


--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to