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 <klussier@masslnc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

diff --git a/Open-ILS/src/templates/opac/myopac/hold_history.tt2 b/Open-ILS/src/templates/opac/myopac/hold_history.tt2
index 3b8abb9..9aae602 100644 (file)
--- a/Open-ILS/src/templates/opac/myopac/hold_history.tt2
+++ b/Open-ILS/src/templates/opac/myopac/hold_history.tt2
@@ -60,7 +60,7 @@
                     <td><span>[% l('Author') %]</span></td>
                     <td> <span>[% l('Format') %]</span></td>
                     <td><span>[% l('Pickup Location') %]</span> </td>
-                    <td>[% l('Active On') %]</td>
+                    <td>[% l('Activate') %]</td>
                     <td>[% l('Active') %]</td>
                     <td>[% l('Date Fulfilled') %]</td>
                     <td><span>[% l('Status') %]</span></td>
@@ -102,11 +102,13 @@
                         END %]
                     </td>
                     <td>
-                        [% 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 %]
                     </td>
                     <td>
                         [% IF ahr.fulfillment_time;