From: Terran McCanna <tmccanna@georgialibraries.org>
Date: Tue, 5 Jan 2016 22:47:01 +0000 (-0500)
Subject: Temporary modifications to My Account > Holds History until more
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=dbc001ee61c32fee3a79c7e9506a1a6e68af900f;p=evergreen%2Fpines.git

Temporary modifications to My Account > Holds History until more
permanent fixes are developed.

Hid paging because it is broken. See:
https://bugs.launchpad.net/evergreen/+bug/1422927

Showing "Canceled" instead of "Active" for canceled holds. See:
https://bugs.launchpad.net/evergreen/+bug/1275118

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
---

diff --git a/Open-ILS/src/templates/opac/myopac/hold_history.tt2 b/Open-ILS/src/templates/opac/myopac/hold_history.tt2
index 9ccfc45323..930ca519fe 100644
--- a/Open-ILS/src/templates/opac/myopac/hold_history.tt2
+++ b/Open-ILS/src/templates/opac/myopac/hold_history.tt2
@@ -30,6 +30,7 @@
 
     <div class="header_middle">
         <span style="float:left;">[% l("Previously Requested Items") %]</span>
+		<!-- HIDE BECAUSE OF BUG
         <span class='float-left' style='padding-left: 10px;'>
             <a href='[% mkurl('hold_history', {limit => limit,offset => (offset - limit)}) %]'
                 [% IF offset == 0 %] class='invisible' [% END %]><span class="nav_arrow_fix">&#9668;</span>[% l('Previous') %]</a>
@@ -48,6 +49,7 @@
             <a href='[% mkurl('hold_history', {limit => limit, offset => (offset + limit)}) %]'
                [% IF count <= limit + offset %] class='invisible' [% END %] >[% l('Next') %]<span class="nav_arrow_fix">&#9658;</span></a>
         </span>
+		-->
         <span style="float:right;">
             <a class="hide_me" href="#">[% l('Export List') %]</a>
         </span>
@@ -104,7 +106,24 @@
                         [% ctx.get_aou(ahr.pickup_lib).name | html %]
                     </td>
                     <td>
-                        <div name="acct_holds_status">
+                        [% IF ahr.frozen == 't' AND ahr.thaw_date;
+                            date.format(ctx.parse_datetime(ahr.thaw_date), DATE_FORMAT);
+                        END %]
+                    </td>
+                    <td>[% IF hold.hold.status != 6; %]
+					        [% l(ahr.frozen == 'f' ? 'Active' : 'Suspended') %]
+					    [% ELSE %]
+						    Canceled 
+						[% END %]
+                    </td>
+                    <td>
+                        [% IF ahr.fulfillment_time;
+                            date.format(ctx.parse_datetime(ahr.fulfillment_time), DATE_FORMAT);
+                        END %]
+                    </td>
+                    <td>
+                        <div name="acct_holds_status"
+                            style="margin-top:10px;margin-bottom:10px;">
                             [% PROCESS get_hold_status hold=hold; %]
                         </div>
                     </td>