Attachment 111857 Details for Bug 24083 – Bug 24083: (follow-up) Fix GetRenewCount return

[patch] Bug 24083: (follow-up) Fix GetRenewCount return

Bug-24083-follow-up-Fix-GetRenewCount-return.patch (text/plain), 1.01 KB, created by Andrew Isherwood on 2020-10-16 11:20:36 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Andrew Isherwood

Created: 2020-10-16 11:20:36 UTC

Size: 1.01 KB

patch

obsolete

>From 625dcb639b24ba9740264ddad8dd02a05cb4336a Mon Sep 17 00:00:00 2001
>From: Andrew Isherwood <andrew.isherwood@ptfs-europe.com>
>Date: Tue, 6 Oct 2020 11:30:31 +0100
>Subject: [PATCH] Bug 24083: (follow-up) Fix GetRenewCount return
>
>GetRenewCount now returns 6 values when a call to it succeeds, a
>failed call should also return the same number of values. This commit
>adds these additional values.
>---
> C4/Circulation.pm | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/C4/Circulation.pm b/C4/Circulation.pm
>index 7a872ed3b1..9058c4d7ee 100644
>--- a/C4/Circulation.pm
>+++ b/C4/Circulation.pm
>@@ -3204,7 +3204,7 @@ sub GetRenewCount {
>     my $patron = Koha::Patrons->find( $bornum );
>     my $item   = Koha::Items->find($itemno);
> 
>-    return (0, 0, 0) unless $patron or $item; # Wrong call, no renewal allowed
>+    return (0, 0, 0, 0, 0, 0) unless $patron or $item; # Wrong call, no renewal allowed
> 
>     # Look in the issues table for this item, lent to this borrower,
>     # and not yet returned.
>-- 
>2.20.1

Actions: View | Diff | Splinter Review

Attachments on bug 24083: 95861 | 95862 | 95863 | 95864 | 95865 | 95866 | 95867 | 95868 | 98749 | 98750 | 98751 | 98752 | 98765 | 98766 | 98767 | 98768 | 100244 | 100245 | 100246 | 100247 | 100380 | 100381 | 100382 | 100383 | 100392 | 110607 | 110608 | 110609 | 110610 | 110611 | 111147 | 111148 | 111149 | 111150 | 111151 | 111152 | 111283 | 111284 | 111285 | 111286 | 111287 | 111288 | 111289 | 111290 | 111291 | 111292 | 111293 | 111849 | 111850 | 111851 | 111852 | 111853 | 111854 | 111855 | 111856 | 111857 | 111858 | 111859 | 111860 | 112230 | 112231 | 112232 | 112233 | 112234 | 112235 | 112236 | 112237 | 112238 | 112239 | 112240 | 112241 | 112242 | 112243 | 112410 | 112411 | 112412 | 112413 | 112414 | 112415 | 112416 | 112417 | 112418 | 112419 | 112420 | 112421 | 112422 | 112423 | 113073 | 113074 | 113075 | 113076 | 113077 | 113078 | 113079 | 113080 | 113314 | 113315 | 113484 | 113485 | 113486 | 113487 | 113488 | 113489 | 113490 | 113491 | 113492 | 113493 | 113494 | 113546 | 113556 | 113557 | 113814