From: Pasi Kallinen <pasi.kallinen@pttk.fi>
Date: Mon, 27 May 2013 09:02:38 +0000 (+0300)
Subject: LP1183414: OPAC patron opt-in settings table does not work correctly
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=97e134a859c2c0cc876f1501a06d9b5ef74fa12d;p=evergreen%2Fpines.git

LP1183414: OPAC patron opt-in settings table does not work correctly

Fix the template logic error, trying to use a variable before
it is defined.

Signed-off-by: Pasi Kallinen <pasi.kallinen@pttk.fi>
Signed-off-by: Bill Erickson <berick@esilibrary.com>
---

diff --git a/Open-ILS/src/templates/opac/myopac/prefs_notify.tt2 b/Open-ILS/src/templates/opac/myopac/prefs_notify.tt2
index fcd9f47823..4f70bb0ef2 100644
--- a/Open-ILS/src/templates/opac/myopac/prefs_notify.tt2
+++ b/Open-ILS/src/templates/opac/myopac/prefs_notify.tt2
@@ -68,16 +68,17 @@
             [% END %]
         </tbody>
     </table>
- 
-    <table>
+
+    <div class='user_opt_in_settings'>
+    [% IF ctx.opt_in_settings.size == 0 %]
+      <strong>[% l('No notification preferences are configured') %]</strong>
+    [% ELSE %]
+      <table>
         <thead><tr>
             <th>[% l('Notification Type') %]</th>
             <th>[% l('Enabled') %]</th>
         </tr></thead>
         <tbody class='data_grid'>
-            [% IF optin.size == 0 %]
-                <strong>[% l('No notification preferences are configured') %]</strong>
-            [% END %]
             [% FOR optin IN ctx.opt_in_settings %]
                 <tr>
                     <td>[% optin.cust.label | html %]</td>
@@ -89,7 +90,9 @@
                 </tr>
             [% END %]
         </tbody>
-    </table>
+      </table>
+    [% END %]
+    </div>
     <input type='submit' value="[% l('Save') %]" alt="[% l('Save') %]" class="opac-button" />
 </form>
 [% END %]