[Koha-bugs] [Bug 6430] DB changes needed for Hourly Circulation

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Fri May 25 12:28:20 CEST 2012


http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6430

Jonathan Druart <jonathan.druart at biblibre.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jonathan.druart at biblibre.co
                   |                            |m,
                   |                            |paul.poulain at biblibre.com

--- Comment #3 from Jonathan Druart <jonathan.druart at biblibre.com> ---
Hummm. I don't understand how this patch have been pushed into master:

commit e56c4b249d9ac66e05054f916560a0f7bea9a194
Author:     Chris Cormack <chrisc at catalyst.net.nz>
AuthorDate: Tue May 31 13:35:14 2011 +1200
Commit:     Chris Cormack <chrisc at catalyst.net.nz>
CommitDate: Tue Mar 20 13:28:52 2012 +1300

    Bug 6430 - DB changes needed to support hourly circ


So, it is in master since Mar 20 2012.
It breaks my patch Bug 6751: I compare the issue.returndate and
C4::Dates->new();
Before this patch, it was consistent, comparing date and date. But now,
issue.returndate is never equal to C4::Dates->new(); (compare datetime and
date).

I think my patch is not the only one encountering this problem.

With a quick search, I think at least these lines are impacted:

C4/Members.pm:        if ($it->{'issuedate'} eq $today) {
circ/circulation.pl:        if ( $todaysdate eq $it->{'issuedate'} or
$todaysdate eq $it->{'lastreneweddate'} ) {

Did I misunderstand something (and then I need my week-end break now!) or does
this patch introduce side effects ?

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.


More information about the Koha-bugs mailing list