LP#1710972 - 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>

Signed-off-by: Chris Sharp <csharp@georgialibraries.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 (file)
new mode 100755 (executable)
index 1f66d15..6e66654
--- 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="true"' : '' %]>
+        [% carrier.name | html %] ([% carrier.region | html %])
+    </option>
     [% END -%]
 </select>
 [% IF NOT sms_carrier_hide_warning; l('Note: carrier charges may apply'); END; %]