From: Steven Callender <stevecallender@esilibrary.com>
Date: Thu, 12 Sep 2013 17:30:16 +0000 (-0400)
Subject: Added in warning message when placing hold with no email address.
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=6707d536768f6cb72da4d974d6f742abf0a4681d;p=evergreen%2Fmasslnc.git

Added in warning message when placing hold with no email address.

Previously when an account didn't have an email address associated with it
when placing a hold, the input box would be greyed out and the display
would show the Email Address text with no address to go with it.

This change will instead show text warning the user that they have
no email address configured and they need to go to My Account in order
to add one to their account.

Signed-off-by: Steven Callender <stevecallender@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
---

diff --git a/Open-ILS/src/templates/opac/parts/place_hold.tt2 b/Open-ILS/src/templates/opac/parts/place_hold.tt2
index 9b410fba2b..cb6abfffd5 100644
--- a/Open-ILS/src/templates/opac/parts/place_hold.tt2
+++ b/Open-ILS/src/templates/opac/parts/place_hold.tt2
@@ -83,7 +83,8 @@
                     [% IF !ctx.user.email %]disabled="true"[% ELSIF ctx.default_email_notify %]checked="checked"[% END %]/>
                     <label for="email_notify">[% l('Yes, by Email') %]</label><br/>
                 <blockquote>
-                    [% l('Email Address:') %] <span name="email_address">[% ctx.user.email %]</span>
+                    [% IF !ctx.user.email; l('No configured Email address. See "My Account" for setting your Email address.');
+                     ELSE; l('Email Address:') %] <span name="email_address">[% ctx.user.email %]</span>[% END %]
                 </blockquote>
                 [%- IF allow_phone_notifications == 'true' %]
                 <input type="checkbox" id="phone_notify_checkbox" name="phone_notify_checkbox"