---- Reported by ian.walls@bywatersolutions.com 2010-03-19 21:09:08 ---- Items returned today as being marked as returned yesterday, even if DropboxMode is not set. This is true even if the item was returned the same day it was issued (making the return date BEFORE the issue date). The problem occurs in Circulation.pm, in the AddReserve subroutine. Code: ------------ if ($doreturn) { $borrower or warn "AddReturn without current borrower"; my $circControlBranch = _GetCircControlBranch($item,$borrower); if ($dropbox) { # don't allow dropbox mode to create an invalid entry in issues (issuedate > returndate) FIXME: actually checks eq, not gt undef($dropbox) if ( $item->{'issuedate'} eq C4::Dates->today('iso') ); } if ($borrowernumber) { MarkIssueReturned($borrowernumber, $item->{'itemnumber'}, $circControlBranch); $messages->{'WasReturned'} = 1; # FIXME is the "= 1" right? This could be the borrower hash. } ModItem({ onloan => undef }, $issue->{'biblionumber'}, $item->{'itemnumber'}); } ------------ $circControlBranch should not be defined unless $dropbox is true, and even then only if issuedate < today. ---- Additional Comments From ian.walls@bywatersolutions.com 2010-03-22 18:19:25 ---- Patch submitted ---- Additional Comments From ian.walls@bywatersolutions.com 2010-03-31 12:52:19 ---- Created an attachment Patch for http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=4326 ---- Additional Comments From gmcharlt@gmail.com 2010-04-30 03:17:38 ---- Patch pushed. Please test and close. --- Bug imported by chris@bigballofwax.co.nz 2010-05-21 01:25 UTC --- This bug was previously known as _bug_ 4326 at http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=4326 Imported an attachment (id=2012) Actual time not defined. Setting to 0.0 Setting qa contact to the default for this product. This bug either had no qa contact or an invalid one. The original submitter of attachment 2012 [details] [review] is unknown. Reassigning to the person who moved it here: chris@bigballofwax.co.nz.
Tested over the last several months, no recurrences found. Closing.