From df495eff86056f135bf9babff809f3e2180e35f7 Mon Sep 17 00:00:00 2001
From: senator <lebbeous@esilibrary.com>
Date: Tue, 15 Feb 2011 10:31:44 -0500
Subject: [PATCH] show grocery fines; define date format in a single location

---
 .../web/templates/default/opac/myopac/circs.tt2    |  2 +-
 .../web/templates/default/opac/myopac/holds.tt2    |  4 +-
 .../web/templates/default/opac/myopac/main.tt2     | 55 +++++++++++++---------
 .../web/templates/default/opac/parts/header.tt2    |  4 +-
 4 files changed, 39 insertions(+), 26 deletions(-)

diff --git a/Open-ILS/web/templates/default/opac/myopac/circs.tt2 b/Open-ILS/web/templates/default/opac/myopac/circs.tt2
index c56b23ecfb..77ba056fb4 100644
--- a/Open-ILS/web/templates/default/opac/myopac/circs.tt2
+++ b/Open-ILS/web/templates/default/opac/myopac/circs.tt2
@@ -106,7 +106,7 @@
                         </td>
                         <td width="13%" style="padding-left:5px;"
                             name="due_date">
-                            [% date.format(ctx.parse_datetime(circ.circ.due_date),'%m/%d/%Y') %]
+                            [% date.format(ctx.parse_datetime(circ.circ.due_date),DATE_FORMAT) %]
                         </td>
                         <td width="16%" name="barcode">
                             [% circ.circ.target_copy.barcode %]
diff --git a/Open-ILS/web/templates/default/opac/myopac/holds.tt2 b/Open-ILS/web/templates/default/opac/myopac/holds.tt2
index 1dd741fd9b..6c6cafe62a 100644
--- a/Open-ILS/web/templates/default/opac/myopac/holds.tt2
+++ b/Open-ILS/web/templates/default/opac/myopac/holds.tt2
@@ -150,7 +150,7 @@
                             name="activate_box" type="text" />
                         [% IF hold.hold.hold.frozen == 't' AND
                                 hold.hold.hold.thaw_date;
-                            date.format(ctx.parse_datetime(hold.hold.hold.thaw_date), '%m/%d/%Y');
+                            date.format(ctx.parse_datetime(hold.hold.hold.thaw_date), DATE_FORMAT);
                         END %]
                     </td>
                     <td width="106">
@@ -158,7 +158,7 @@
                             class="hide_me" style="width:91px;"
                             name="hold_expires_box" type="text" />
                         [% IF hold.hold.hold.expire_time;
-                            date.format(ctx.parse_datetime(hold.hold.hold.expire_time), '%m/%d/%Y');
+                            date.format(ctx.parse_datetime(hold.hold.hold.expire_time), DATE_FORMAT);
                         END %]
                     </td>
                     <td width="95">
diff --git a/Open-ILS/web/templates/default/opac/myopac/main.tt2 b/Open-ILS/web/templates/default/opac/myopac/main.tt2
index cea25a8b6a..b6a53df51c 100644
--- a/Open-ILS/web/templates/default/opac/myopac/main.tt2
+++ b/Open-ILS/web/templates/default/opac/myopac/main.tt2
@@ -177,14 +177,14 @@
                                 [% date.format(
                                     ctx.parse_datetime(
                                         f.xact.circulation.xact_start
-                                    ), "%m/%d/%Y"
+                                    ), DATE_FORMAT
                                 ) %]
                             </td>
                             <td name='myopac_circ_trans_due'>
                                 [% date.format(
                                     ctx.parse_datetime(
                                         f.xact.circulation.due_date
-                                    ), "%m/%d/%Y"
+                                    ), DATE_FORMAT
                                 ) %]
                             </td>
                             <td name='myopac_circ_trans_finished'>
@@ -192,7 +192,7 @@
                                         date.format(
                                             ctx.parse_datetime(
                                                 f.xact.circulation.checkin_time
-                                            ), "%m/%d/%Y"
+                                            ), DATE_FORMAT
                                         );
                                     ELSE %]
                                     <!-- XXX TODO fines aren't really accruing
@@ -208,7 +208,8 @@
                             </td>
                             <td align="center">
                                 <input type="checkbox" checked="checked"
-                                    name="selector" title="pay this fine" />
+                                    name="selector"
+                                    title="[% l('Pay this fine') %]" />
                             </td>
                         </tr>
                         [% END %]
@@ -217,8 +218,9 @@
             </div>
         [% END %]
 
+        [% IF ctx.fines.grocery.size > 0 %]
             <!-- Table for all non-circulation transactions -->
-            <div id='myopac_trans_div' class='hide_me'>
+            <div id='myopac_trans_div'>
                 <br/>
                 <hr style="border-bottom:none;*height:0px;" color="#dcdbdb" />
                 <br/>
@@ -239,40 +241,49 @@
                             <td width='16%'>[% l("Billing Type") %]</td>
                             <td width='4%' align="center" nowrap="nowrap"
                                 style="white-space:nowrap;">
-                                <label for="pay_fines_box2">Pay Fines</label>
+                                <label for="pay_fines_box2">[% l("Pay Fines") %]</label>
                                 <br />
                                 <input id="pay_fines_box2" checked="checked"
                                     type="checkbox"
-                                    title="Click to (un)select all fines" />
+                                    title="[% l('Click to (un)select all fines') %]" />
                             </td>
                         </tr>
                     </thead>
                     <tbody id='myopac_trans_tbody'>
+                        [% FOR f IN ctx.fines.grocery %]
                         <tr id='myopac_trans_row'>
-                            <td name='myopac_trans_start'> </td>
-                            <td name='myopac_trans_last_payment'> </td>
-                            <td name='myopac_trans_init_amount'>
-                                [% l("\$") %]
-                            </td>
-                            <td name='myopac_trans_total_paid'>
-                                [% l("\$") %]
+                            <td>[% date.format(
+                                    ctx.parse_datetime(f.xact.xact_start),
+                                    DATE_FORMAT
+                            ) %]</td>
+                            <td>
+                                [%  IF f.xact.last_payment_ts;
+                                        date.format(
+                                            ctx.parse_datetime(
+                                                f.xact.last_payment_ts
+                                            ), DATE_FORMAT
+                                        );
+                                    END %]
                             </td>
-                            <td style='color:red; font-weight: bold;'>
-                                <span name='myopac_trans_balance_recur'
-                                    class='hide_me'> * </span>
-                                <span name='myopac_trans_balance'>
-                                    [% l("\$") %]
-                                </span>
+                            <td>[% money(f.xact.total_owed) %]</td>
+                            <td>[% money(f.xact.total_paid) %]</td>
+                            <td class="red">
+                                <strong>
+                                    [% money(f.xact.balance_owed) %]
+                                </strong>
                             </td>
-                            <td name='myopac_trans_bill_type'></td>
+                            <td>[% f.xact.last_billing_type %]</td>
                             <td align="center">
                                 <input type="checkbox" name='selector'
-                                    title='pay this fine' checked="checked" />
+                                    title='[% l("Pay this fine") %]'
+                                    checked="checked" />
                             </td>
                         </tr>
+                        [% END %]
                     </tbody>
                 </table>
             </div>
+            [% END %]
             <a href="#"><img alt="Pay Fines"
                 onmouseover="this.src='[% ctx.media_prefix %]/images/pay-fines-btn-hover.png';"
                 src="[% ctx.media_prefix %]/images/pay-fines-btn.png"
diff --git a/Open-ILS/web/templates/default/opac/parts/header.tt2 b/Open-ILS/web/templates/default/opac/parts/header.tt2
index 08f0854a94..4f72639434 100644
--- a/Open-ILS/web/templates/default/opac/parts/header.tt2
+++ b/Open-ILS/web/templates/default/opac/parts/header.tt2
@@ -1,4 +1,6 @@
 [%- USE money = format('$%.2f');
     USE date;
     USE CGI; 
-    USE EGI18N; -%]
+    USE EGI18N;
+    SET DATE_FORMAT = '%m/%d/%Y';
+-%]
-- 
2.11.0