Bug 7631 - Self checkout renewal fails because of reference to non-existent subroutin...
authorOwen Leonard <oleonard@myacpl.org>
Thu, 1 Mar 2012 15:00:14 +0000 (10:00 -0500)
committerPaul Poulain <paul.poulain@biblibre.com>
Thu, 8 Mar 2012 17:10:52 +0000 (18:10 +0100)
Changing "CanBookBeIssuedCheckout" to "CanBookBeIssued"

To test, try to renew an item which has no renewals left. Before the patch
you'll get an error:

Undefined subroutine &main::CanBookBeIssuedCheckout called at /opac/sco/sco-main.pl line 135.

After the patch you'll get the correct message about having no renewals left.
Other tests: checking out a barcode which doesn't exist, checking out an
item which is on hold for another patron.

Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
opac/sco/sco-main.pl

index 7e4f810..f0ca37e 100755 (executable)
@@ -132,7 +132,7 @@ elsif ( $op eq "checkout" ) {
     my $impossible  = {};
     my $needconfirm = {};
     if ( !$confirmed ) {
-        ( $impossible, $needconfirm ) = CanBookBeIssuedCheckout(
+        ( $impossible, $needconfirm ) = CanBookBeIssued(
             $borrower,
             $barcode,
             undef,