From 463ac7ecfe29760e6f520e97e7145813c939ead9 Mon Sep 17 00:00:00 2001 From: Kathy Lussier Date: Fri, 4 Mar 2016 20:51:04 -0500 Subject: [PATCH] LP#1576502 Fulfilled holds also should not display as active Once a hold has been fulfilled, the patron doesn't consider it active, so let's change the active status for those holds too. Signed-off-by: Kathy Lussier Signed-off-by: Mike Rylander --- Open-ILS/src/templates/opac/myopac/hold_history.tt2 | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/Open-ILS/src/templates/opac/myopac/hold_history.tt2 b/Open-ILS/src/templates/opac/myopac/hold_history.tt2 index 3b8abb9310..9aae602297 100644 --- a/Open-ILS/src/templates/opac/myopac/hold_history.tt2 +++ b/Open-ILS/src/templates/opac/myopac/hold_history.tt2 @@ -60,7 +60,7 @@ [% l('Author') %] [% l('Format') %] [% l('Pickup Location') %] - [% l('Active On') %] + [% l('Activate') %] [% l('Active') %] [% l('Date Fulfilled') %] [% l('Status') %] @@ -102,11 +102,13 @@ END %] - [% IF hold.hold.status == 6 %] - [% l('Canceled') %] - [% ELSE %] - [% l(ahr.frozen == 'f' ? 'Active' : 'Suspended') %] - [% END %] + [% IF ahr.fulfillment_time; + l('Fulfilled'); + ELSIF hold.hold.status == 6; + l('Canceled'); + ELSE; + l(ahr.frozen == 'f' ? 'Active' : 'Suspended'); + END %] [% IF ahr.fulfillment_time; -- 2.11.0