Bug 21641: (follow-up) Address wrong call method
authorAndrew Isherwood <andrew.isherwood@ptfs-europe.com>
Thu, 25 Oct 2018 13:17:59 +0000 (14:17 +0100)
committerNick Clemens <nick@bywatersolutions.com>
Mon, 29 Oct 2018 00:22:17 +0000 (00:22 +0000)
This patch substitutes C4::Accounts->getnextacctno for
C4::Accounts::getnextacctno since getnextacctno is only expecting to be
passed a borrowernumber

Signed-off-by: Alex Buckley <alexbuckley@catalyst.net.nz>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>

C4/Circulation.pm

index efb923d..50d5e6d 100644 (file)
@@ -2851,7 +2851,7 @@ sub AddRenewal {
     # Charge a new rental fee, if applicable?
     my ( $charge, $type ) = GetIssuingCharges( $itemnumber, $borrowernumber );
     if ( $charge > 0 ) {
-        my $accountno = C4::Accounts->getnextacctno( $borrowernumber );
+        my $accountno = C4::Accounts::getnextacctno( $borrowernumber );
         my $manager_id = 0;
         $manager_id = C4::Context->userenv->{'number'} if C4::Context->userenv; 
         $sth = $dbh->prepare(
@@ -3191,7 +3191,7 @@ sub AddIssuingCharge {
 
     # FIXME What if checkout does not exist?
 
-    my $nextaccntno = C4::Accounts->getnextacctno($checkout->borrowernumber);
+    my $nextaccntno = C4::Accounts::getnextacctno( $checkout->borrowernumber );
 
     my $manager_id  = 0;
     $manager_id = C4::Context->userenv->{'number'} if C4::Context->userenv;