From: Jason Etheridge <jason@esilibrary.com>
Date: Mon, 25 Feb 2013 20:20:43 +0000 (-0500)
Subject: more tweaks based on feedback from CW/MARS
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=2f61b50b687ac1ce4b6b721261797e182c01a4a9;p=working%2FEvergreen.git

more tweaks based on feedback from CW/MARS

Thanks! This restores the original address scheme to the patron display summary,
where the mailing address is displayed under the patron name, if set and valid,
otherwise the billing address is displayed, if set and valid.  No address is
displayed if there is no valid address set for mailing or billing.

This also adds a potential 'Account has EXPIRED' message to the alerts column.

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
---

diff --git a/Open-ILS/src/templates/opac/PatronSummary.tt2 b/Open-ILS/src/templates/opac/PatronSummary.tt2
index 9832bdd25b..ccae83fc56 100644
--- a/Open-ILS/src/templates/opac/PatronSummary.tt2
+++ b/Open-ILS/src/templates/opac/PatronSummary.tt2
@@ -17,11 +17,29 @@
                                 ctx.user.suffix
                             ) | html %]
                         </span><br/>
+                        [% IF ctx.addr %]
+                        <span>
+                            [% ctx.addr.street1 | html %]<br/>
+                            [% ctx.addr.street2 | html %]<br/>
+                            [% ctx.addr.city | html %],
+                            [% ctx.addr.state | html %]
+                            [% ctx.addr.post_code | html %]
+                        </span><br/>
+                        [% END %]
+                    </div>
+                </div><div class="cell">
+                    <div class="textBackground alerts">
                         [% IF ctx.barred %]
                         <span>
                             [% l('Patron is BARRED') | html %]
                         </span><br/>
                         [% END %]
+                        [% expire_date = ctx.parse_datetime(ctx.user.expire_date); %]
+                        [% IF (date.format(expire_date, '%s') < date.now) %]
+                        <span>
+                            [% l('Account has EXPIRED') | html %]
+                        </span><br/>
+                        [% END %]
                         [% IF ctx.holds_count.ready > 0 %]
                         <span>
                             [% IF ctx.holds_count.ready == 1 %]
@@ -36,14 +54,15 @@
                             [% ctx.user.alert_message | html %]
                         </span><br/>
                         [% END %]
+                        [% IF ctx.user.notes.size > 0 %]
+                        <span>
                         [% IF ctx.user.notes.size == 1 %]
                             [% l('1 note') | html %]
                         [% ELSIF ctx.user.notes.size > 1 %]
                             [% l('[_1] notes', ctx.user.notes.size) | html %]
                         [% END %]
-                    </div>
-                </div><div class="cell">
-                    <div class="textBackground alerts">
+                        </span><br/>
+                        [% END %]
                         [% FOREACH p IN ctx.user.standing_penalties %]
                         <span>
                             [% p.standing_penalty.label | html %]