From: Cesar Velez <cesar.velez@equinoxinitiative.org>
Date: Mon, 14 Aug 2017 16:19:39 +0000 (-0400)
Subject: Fix issue related to LP#1098685 in myOPAC notification prefs
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=refs%2Fheads%2Fuser%2Fcesardv%2Fopac_sms_carriers_dropdown;p=working%2FEvergreen.git

Fix issue related to LP#1098685 in myOPAC notification prefs

Bug 1098685, addressed issues with sms carrier defaults
and selection/validation. Same bahavior should apply in
MyOPAC user prefs.

Signed-off by: Cesar Velez <cesar.velez@equinoxinitiative.org>
---

diff --git a/Open-ILS/src/templates/opac/parts/sms_carrier_selector.tt2 b/Open-ILS/src/templates/opac/parts/sms_carrier_selector.tt2
old mode 100644
new mode 100755
index 1f66d15541..55a518d892
--- a/Open-ILS/src/templates/opac/parts/sms_carrier_selector.tt2
+++ b/Open-ILS/src/templates/opac/parts/sms_carrier_selector.tt2
@@ -21,9 +21,9 @@
 <select name="sms_carrier">
     <option value="">[% l('') %]</option>
     [% FOR carrier IN carriers.sort('name','region') -%]
-    <option value='[% carrier.id | html %]'[%
-        default_carrier == carrier.id ? ' selected="selected"' : ''
-    %]>[% carrier.name | html %] ([% carrier.region | html %])</option>
+    <option value='[% carrier.id | html %]' [% default_carrier == carrier.id ? ' selected' : '' %]>
+        [% carrier.name | html %] ([% carrier.region | html %])
+    </option>
     [% END -%]
 </select>
 [% IF NOT sms_carrier_hide_warning; l('Note: carrier charges may apply'); END; %]