From 57f074da7137d5f5fb5842e23617aa04c81eed8c Mon Sep 17 00:00:00 2001 From: Lucas Gass Date: Tue, 20 Feb 2024 20:49:29 +0000 Subject: [PATCH] Bug 36138: Add reason to status column on opac-holdshistory, if there is a reason TO TEST: 1. Turn on OPACHoldsHistory 2. Make some holds for a patron and then cancel them with a reason. 3. Log in as that patron and go to the 'Holds history' tab. Notice the text in the 'Status' column is 'Cancelled' but nothing shows up regarding the reason. 4. Apply patch and try again, this time you should see the reason for cancellation. Signed-off-by: Esther Signed-off-by: Pedro Amorim Signed-off-by: Katrin Fischer (cherry picked from commit f4f2fdb65f37780004d6d538c090902c20544167) Signed-off-by: Fridolin Somers --- koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-holdshistory.tt | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-holdshistory.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-holdshistory.tt index 224c5441ea..06d97bb793 100644 --- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-holdshistory.tt +++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-holdshistory.tt @@ -5,6 +5,7 @@ [% USE TablesSettings %] [% USE ItemTypes %] [% USE AdditionalContents %] +[% USE AuthorisedValues %] [% SET OpacNav = AdditionalContents.get( location => "OpacNav", lang => lang, library => logged_in_user.branchcode || default_branch, blocktitle => 0 ) %] [% SET OpacNavBottom = AdditionalContents.get( location => "OpacNavBottom", lang => lang, library => logged_in_user.branchcode || default_branch, blocktitle => 0 ) %] @@ -147,6 +148,9 @@ Fulfilled [% ELSIF hold.cancellationdate %] Cancelled + [% IF hold.cancellation_reason %] + ([% AuthorisedValues.GetByCode('HOLD_CANCELLATION', hold.cancellation_reason, 1) | html %]) + [% END %] [% ELSIF hold.found == 'W' %] [% IF hold.cancellation_requests.count == 0 %] Waiting -- 2.39.5