[Koha-patches] [PATCH] Bug 4056: Fixes invalid xhtml in opac-messaging.tmpl if SMS is disabled.

Garry Collum gcollum at gmail.com
Fri Jan 22 03:57:44 CET 2010


There is a </td> tag which is outside of the block of code which defines checks to see if SMS is enabled.  If SMS is disabled it causes several extra </td> tags.
---
 .../opac-tmpl/prog/en/modules/opac-messaging.tmpl  |    3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)

diff --git a/koha-tmpl/opac-tmpl/prog/en/modules/opac-messaging.tmpl b/koha-tmpl/opac-tmpl/prog/en/modules/opac-messaging.tmpl
index f1532e2..0e0543c 100644
--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-messaging.tmpl
+++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-messaging.tmpl
@@ -56,8 +56,7 @@
       <!-- TMPL_IF NAME="SMSSendDriver" --><td><input type="checkbox"
                  id="sms<!-- TMPL_VAR NAME="message_attribute_id" -->"
                  name="<!-- TMPL_VAR NAME="message_attribute_id" -->"
-                 value="sms" <!-- TMPL_VAR NAME="transport-sms"   --> onclick = "document.opacmessaging.none<!-- TMPL_VAR NAME="message_attribute_id" -->.checked=false;" /><!-- /TMPL_IF -->
-      </td>
+                 value="sms" <!-- TMPL_VAR NAME="transport-sms"   --> onclick = "document.opacmessaging.none<!-- TMPL_VAR NAME="message_attribute_id" -->.checked=false;" /></td><!-- /TMPL_IF -->
       <!-- TMPL_ELSE -->
       <td>-</td>
       <!-- /TMPL_IF -->
-- 
1.5.6.5




More information about the Koha-patches mailing list