From 657658c642b21b0497d44fa9a087195b8dc5b06f Mon Sep 17 00:00:00 2001
From: Ben Shum <bshum@biblio.org>
Date: Tue, 21 Jan 2014 13:52:49 -0500
Subject: [PATCH] LP1271218 - clarify how opt-in user notifications are shown
 in my account

As reported in LP127128, end users may not need to be shown the message that
"No notification preferences are configured" when in fact this message is only
displayed when the user opt-in notification settings are not configured.

So, alter this to only show the table and div when there are settings to be
configured and otherwise show nothing.

Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
---
 Open-ILS/src/templates/opac/myopac/prefs_notify.tt2 | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/Open-ILS/src/templates/opac/myopac/prefs_notify.tt2 b/Open-ILS/src/templates/opac/myopac/prefs_notify.tt2
index c0c1fba906..d573b7a4fd 100644
--- a/Open-ILS/src/templates/opac/myopac/prefs_notify.tt2
+++ b/Open-ILS/src/templates/opac/myopac/prefs_notify.tt2
@@ -79,10 +79,8 @@
         </tbody>
     </table>
 
+    [% IF ctx.opt_in_settings.size > 0 %]
     <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>
@@ -101,8 +99,8 @@
             [% END %]
         </tbody>
       </table>
-    [% END %]
     </div>
+    [% END %]
     <input type='submit' value="[% l('Save') %]" alt="[% l('Save') %]" class="opac-button" />
 </form>
 [% END %]
-- 
2.11.0