[open-ils-commits] [GIT] Evergreen ILS branch rel_2_4 updated. bb89719aca7cd0fd4ae7b66f36c0e7940578d53d

[Evergreen Git]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Evergreen ILS".

The branch, rel_2_4 has been updated
       via  bb89719aca7cd0fd4ae7b66f36c0e7940578d53d (commit)
      from  bab7ddd4c26739177eb202678b91a09a38d87c20 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit bb89719aca7cd0fd4ae7b66f36c0e7940578d53d
Author: Pasi Kallinen <pasi.kallinen at pttk.fi>
Date:   Mon May 27 12:02:38 2013 +0300

    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 at pttk.fi>
    Signed-off-by: Bill Erickson <berick at 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 43eed64..c0c1fba 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 %]

-----------------------------------------------------------------------

Summary of changes:
 .../src/templates/opac/myopac/prefs_notify.tt2     |   15 +++++++++------
 1 files changed, 9 insertions(+), 6 deletions(-)


hooks/post-receive
-- 
Evergreen ILS