Joe Atzberger a écrit :
>
>
> On Tue, Oct 14, 2008 at 10:23 AM, Nahuel Angelinetti
> <[EMAIL PROTECTED]
> <mailto:[EMAIL PROTECTED]>> 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
Hi joe,
the problem here is that SQL expects ISO formatted dates and was
provided non iso.
So he solved it quick way.
And would not format input dates into iso from those provided by the page.
I agree with you.
Will resubmit a patch.
Cheers.
-- 
Henri-Damien LAURENT
_______________________________________________
Koha-patches mailing list
[email protected]
http://lists.koha.org/mailman/listinfo/koha-patches

Reply via email to