[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