[Koha-patches] [PATCH] [SIGNED-OFF] Fix for Bug 6726 - When SMS is enabled the messaging table is misaligned

Nicole C. Engard nengard at bywatersolutions.com
Mon Aug 15 17:55:09 CEST 2011


From: Owen Leonard <oleonard at myacpl.org>

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 at bywatersolutions.com>
---
 .../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.2.3



More information about the Koha-patches mailing list