The SMS section of the template was commented out in the patch
pushed for Bug 5889, 'Allow message names to be translatable'
with the comment 'fixing columns overflow.' That seems to have been
a cosmetic fix for what I can only imagine was a different bug?

Also fixing a T:T variable scope error.

After applying this patch the "SMS" column in the Patron messaging
preferences should appear when the SMSSendDriver preference is not empty.
---
 .../prog/en/includes/messaging-preference-form.inc |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/koha-tmpl/intranet-tmpl/prog/en/includes/messaging-preference-form.inc 
b/koha-tmpl/intranet-tmpl/prog/en/includes/messaging-preference-form.inc
index 90dbf1d..7ece680 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/includes/messaging-preference-form.inc
+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/messaging-preference-form.inc
@@ -51,8 +51,8 @@
       [% ELSE %]
       <td>-</td>
       [% END %]
-<!--      [% IF ( messaging_preference.transport_sms ) %]
-      [% IF ( messaging_preference.SMSSendDriver ) %]<td>
+      [% IF ( messaging_preference.transport_sms ) %]
+      [% IF ( SMSSendDriver ) %]<td>
           [% IF ( messaging_form_inactive ) %]
                  <input type="checkbox"
                  id="sms[% messaging_preference.message_attribute_id %]"
@@ -67,7 +67,7 @@
       </td>[% END %]
       [% ELSE %]
       <td>-</td>
-      [% END %] -->
+      [% END %]
 
       [% IF ( messaging_preference.transport_email ) %]
       <td>
-- 
1.7.3

_______________________________________________
Koha-patches mailing list
Koha-patches@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-patches
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to