Fix for Bug 6726 - When SMS is enabled the messaging table is misaligned

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.

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>

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 (file)
--- 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>