From 0ebeb8a5a0b7b21667dfb3319b006e21f0a950c5 Mon Sep 17 00:00:00 2001
From: Pasi Kallinen <pasi.kallinen@pttk.fi>
Date: Mon, 27 May 2013 12:02:38 +0300
Subject: [PATCH] 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>
---
 Open-ILS/src/templates/opac/myopac/prefs_notify.tt2 | 15 +++++++++------
 1 file changed, 9 insertions(+), 6 deletions(-)

diff --git a/Open-ILS/src/templates/opac/myopac/prefs_notify.tt2 b/Open-ILS/src/templates/opac/myopac/prefs_notify.tt2
index 43eed643bb..c0c1fba906 100644
--- a/Open-ILS/src/templates/opac/myopac/prefs_notify.tt2
+++ b/Open-ILS/src/templates/opac/myopac/prefs_notify.tt2
@@ -78,16 +78,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>
@@ -99,7 +100,9 @@
                 </tr>
             [% END %]
         </tbody>
-    </table>
+      </table>
+    [% END %]
+    </div>
     <input type='submit' value="[% l('Save') %]" alt="[% l('Save') %]" class="opac-button" />
 </form>
 [% END %]
-- 
2.11.0