Bug 17453: Take into account items that are lost or damaged
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Fri, 10 Feb 2017 07:54:39 +0000 (08:54 +0100)
committerKyle M Hall <kyle@bywatersolutions.com>
Fri, 17 Feb 2017 17:15:07 +0000 (17:15 +0000)
If all the items are either lost, damaged or checked out, then pickup
should be allowed.

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
https://bugs.koha-community.org/show_bug.cgi?id=14753

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
opac/opac-reserve.pl

index f5e4130..d42cc15 100755 (executable)
@@ -289,7 +289,8 @@ if ( $query->param('place_reserve') ) {
         unless ( $can_place_hold_if_available_at_pickup ) {
             my $items_in_this_library = Koha::Items->search({ biblionumber => $biblioNum, holdingbranch => $branch });
             my $nb_of_items_issued = $items_in_this_library->search({ 'issue.itemnumber' => { not => undef }}, { join => 'issue' })->count;
-            if ( $items_in_this_library->count > $nb_of_items_issued ) {
+            my $nb_of_items_unavailable = $items_in_this_library->search({ -or => { lost => { '!=' => 0 }, damaged => { '!=' => 0 }, } });
+            if ( $items_in_this_library->count > $nb_of_items_issued + $nb_of_items_unavailable ) {
                 $canreserve = 0
             }
         }