https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17502
M. Tompsett <mtomp...@hotmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |mtomp...@hotmail.com
--- Comment #12 from M. Tompsett <mtomp...@hotmail.com> ---
(In reply to Marcel de Rooy from comment #9)
> Main change has been used in 15822.
> So forget the rest.
Well, unless you care about type checking, if you actually right your test, you
will still trigger an error, because dt is a string and not a DateTime.
A quick perusal of the code shows lots of dt_from_string calls, and lots of
$foo's used. Given the number of $foo's, I suspect they probably are dates,
because otherwise we would have hit this problem sooner. So leaving it closed,
because type checking the programmer's code should be part of the QA process.
:)
--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
