Attachment 113494 Details for Bug 24083 – Bug 24083: (follow-up) Make requested changes

[patch] Bug 24083: (follow-up) Make requested changes

Bug-24083-follow-up-Make-requested-changes.patch (text/plain), 3.98 KB, created by Andrew Isherwood on 2020-11-11 13:21:44 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Andrew Isherwood

Created: 2020-11-11 13:21:44 UTC

Size: 3.98 KB

patch

obsolete

>From 60ac37b12d2b47760a240b2c077cc97542d89e4b Mon Sep 17 00:00:00 2001
>From: Andrew Isherwood <andrew.isherwood@ptfs-europe.com>
>Date: Wed, 11 Nov 2020 13:19:18 +0000
>Subject: [PATCH] Bug 24083: (follow-up) Make requested changes
>
>This commit makes the changes requested in comment #120
>---
> koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt | 2 --
> misc/cronjobs/automatic_renewals.pl                      | 1 +
> opac/sco/sco-main.pl                                     | 5 ++---
> 3 files changed, 3 insertions(+), 5 deletions(-)
>
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt
>index f2115e9f7a..907f4bdf81 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt
>@@ -133,7 +133,6 @@
>                                     <input type="hidden" name="patronid" value="[% patronid | html %]" />
>                                     <input type="hidden" name="barcode" value="[% barcode | html %]" />
>                                     <input type="hidden" name="confirmed" value="1" />
>-                                    <input type="hidden" name="seen" value="1" />
>                                     <input type="hidden" name="newissues" value="[% newissues | html %]" />
>                                     <button type="submit" name="confirm" class="btn btn-primary"><i class="fa fa-refresh" aria-hidden="true"></i> Renew item</button>
>                                   </form>
>@@ -143,7 +142,6 @@
>                                     <input type="hidden" name="patronid" value="[% patronid | html %]" />
>                                     <input type="hidden" name="barcode" value="[% barcode | html %]" />
>                                     <input type="hidden" name="confirmed" value="1" />
>-                                    <input type="hidden" name="seen" value="1" />
>                                     <input type="hidden" name="newissues" value="[% newissues | html %]" />
>                                     <button type="submit" class="btn btn-primary"><i class="fa fa-refresh" aria-hidden="true"></i> Renew item</button>
>                                 </form>
>diff --git a/misc/cronjobs/automatic_renewals.pl b/misc/cronjobs/automatic_renewals.pl
>index 7c19e94b45..39f241248c 100755
>--- a/misc/cronjobs/automatic_renewals.pl
>+++ b/misc/cronjobs/automatic_renewals.pl
>@@ -103,6 +103,7 @@ while ( my $auto_renew = $auto_renews->next ) {
>         or $error eq 'on_reserve'
>         or $error eq 'restriction'
>         or $error eq 'overdue'
>+        or $error eq 'too_unseen'
>         or $error eq 'auto_account_expired'
>         or $error eq 'auto_too_late'
>         or $error eq 'auto_too_much_oweing'
>diff --git a/opac/sco/sco-main.pl b/opac/sco/sco-main.pl
>index fa72f8496c..742f7f43ca 100755
>--- a/opac/sco/sco-main.pl
>+++ b/opac/sco/sco-main.pl
>@@ -99,14 +99,13 @@ if (defined C4::Context->preference('SCOAllowCheckin')) {
> }
> 
> my $issuerid = $loggedinuser;
>-my ($op, $patronid, $patronlogin, $patronpw, $barcode, $confirmed, $seen, $newissues) = (
>+my ($op, $patronid, $patronlogin, $patronpw, $barcode, $confirmed, $newissues) = (
>     $query->param("op")         || '',
>     $query->param("patronid")   || '',
>     $query->param("patronlogin")|| '',
>     $query->param("patronpw")   || '',
>     $query->param("barcode")    || '',
>     $query->param("confirmed")  || '',
>-    $query->param("seen")       || 0,
>     $query->param("newissues")  || '',
> );
> 
>@@ -269,7 +268,7 @@ if ( $patron && ( $op eq 'renew' ) ) {
>     my ($status,$renewerror) = CanBookBeRenewed( $borrower->{borrowernumber}, $item->itemnumber );
>     if ($status) {
>         #warn "renewing";
>-        AddRenewal( $borrower->{borrowernumber}, $item->itemnumber, undef, undef, undef, undef, $seen );
>+        AddRenewal( $borrower->{borrowernumber}, $item->itemnumber, undef, undef, undef, undef, 1 );
>         push @newissueslist, $barcode;
>         $template->param( renewed => 1 );
>     }
>-- 
>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