Bugzilla – Attachment 159724 Details for
Bug 35310
Current renewals 'view' link doesnt work if renewals correspond to an item no longer checked out
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 35310: Only show 'Current renewals' link if item is currently checked out
Bug-35310-Only-show-Current-renewals-link-if-item-.patch (text/plain), 1.87 KB, created by
Martin Renvoize (ashimema)
on 2023-12-12 11:04:55 UTC
(
hide
)
Description:
Bug 35310: Only show 'Current renewals' link if item is currently checked out
Filename:
MIME Type:
Creator:
Martin Renvoize (ashimema)
Created:
2023-12-12 11:04:55 UTC
Size:
1.87 KB
patch
obsolete
>From 5e41526f7c4e7ac8bcc4bdd4f457137590ea7d95 Mon Sep 17 00:00:00 2001 >From: Pedro Amorim <pedro.amorim@ptfs-europe.com> >Date: Fri, 10 Nov 2023 11:01:58 +0000 >Subject: [PATCH] Bug 35310: Only show 'Current renewals' link if item is > currently checked out > >1) Checkout an item to Koha user, barcode 39999000003192 >2) Renew that barcode >3) Visit the items page of the biblio record, click on "View" in front of "Current renewals" >/cgi-bin/koha/catalogue/moredetail.pl?biblionumber=76 >4) Notice it shows the renewals as expected >5) Return the same item 39999000003192 >6) Repeat 3). Notice it never loads renewals and network console shows HTTP error 400 >7) Apply this patch. Repeat the test plan. Notice the 'Current renewals' no longer shows for the item because it is no longer checked out, so whatever past renewals are no longer 'current' > >Signed-off-by: David Nind <david@davidnind.com> >--- > koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/moredetail.tt | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/moredetail.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/moredetail.tt >index f96a1f9e9f8..63682de50b0 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/moredetail.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/moredetail.tt >@@ -193,7 +193,7 @@ > [% END %] > [% END %] > </li> >- [% IF ITEM_DAT.renewals %] >+ [% IF ITEM_DAT.renewals && ITEM_DAT.object.checkout %] > <li> > <span class="label">Current renewals:</span> > [% ITEM_DAT.renewals | html %] >-- >2.43.0
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 35310
:
158799
|
158800
|
158801
|
158822
|
158823
|
158824
|
158976
|
159723
| 159724 |
159725