Bug 4326 ; returndate were fixed on the day before
authorHenri-Damien LAURENT <henridamien.laurent@biblibre.com>
Fri, 14 May 2010 21:20:41 +0000 (23:20 +0200)
committerHenri-Damien LAURENT <henridamien.laurent@biblibre.com>
Fri, 14 May 2010 21:20:41 +0000 (23:20 +0200)
dropbox mode was set even though it was not dropbox.
This patch is an adaptation of Ian Walls
http://www.mail-archive.com/koha-patches@lists.koha.org/msg04736.html

C4/Circulation.pm

index 0b0a9b2..5fcd294 100644 (file)
@@ -1431,9 +1431,10 @@ sub AddReturn {
     # case of a return of document (deal with issues and holdingbranch)
             
         if ($doreturn) {
-                       my $circControlBranch = _GetCircControlBranch($iteminformation,$borrower);
+                       my $circControlBranch;
                        if($dropbox) {
                                # don't allow dropbox mode to create an invalid entry in issues (issuedate > returndate) FIXME: actually checks eq, not gt
+                $circControlBranch=_GetCircControlBranch($iteminformation,$borrower) unless ( $iteminformation->{'issuedate'} eq C4::Dates->today('iso') );
                                undef($dropbox) if ( $iteminformation->{'issuedate'} eq C4::Dates->today('iso') );
                        }
             MarkIssueReturned($borrower->{'borrowernumber'}, $iteminformation->{'itemnumber'},$circControlBranch);