[Koha-patches] [PATCH] Add the feature to create reports with borrower's sex information(Men/Women) (bug #2671).

Nahuel ANGELINETTI nahuel.angelinetti at biblibre.com
Wed Oct 15 09:38:44 CEST 2008


Hi,

This part of the patch was a mistake.
It was a quick fix that allow me to test birthdate reports.
I just resend a new patch without this.

Bests,

Nahuel

Le Tue, 14 Oct 2008 11:43:41 -0400,
"Joe Atzberger" <joe.atzberger at liblime.com> a écrit :

> On Tue, Oct 14, 2008 at 10:23 AM, Nahuel Angelinetti <
> nahuel.angelinetti at biblibre.com> wrote:
> 
> > ---
> > -               DHTMLcalendar_dateformat =>
> > C4::Dates->DHTMLcalendar(),
> > +               DHTMLcalendar_dateformat =>
> > C4::Dates->DHTMLcalendar('iso')
> >
> 
> I'm not sure, but I think this might cause trouble to those whose date
> format is not ISO.  Scripts should expect user-input dates received
> to be in the syspref format, "catching" them by instantiating
> C4::Dates objects (or at least using the Dates regexps).
> 
> The datefomat change is not described in your commit message and seems
> unrelated to it.  Perhaps you can resubmit without it, or explain your
> motivation?
> 
> --Joe



More information about the Koha-patches mailing list