From 6b09dfbc8b23d534cbd48e72325f073fbbb28987 Mon Sep 17 00:00:00 2001
From: Dan Scott <dan@coffeecode.net>
Date: Tue, 3 May 2011 20:42:16 -0400
Subject: [PATCH] HTML compliance in result table

Convert & to &amp; in URLs, and change a SPAN element to a DIV
(because SPAN elements are not allowed to contain block-level
elements like DIV and TABLE).

Signed-off-by: Dan Scott <dan@coffeecode.net>
---
 Open-ILS/web/templates/default/opac/parts/result/table.tt2 | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/Open-ILS/web/templates/default/opac/parts/result/table.tt2 b/Open-ILS/web/templates/default/opac/parts/result/table.tt2
index cb76024a16..83da3d8d8b 100644
--- a/Open-ILS/web/templates/default/opac/parts/result/table.tt2
+++ b/Open-ILS/web/templates/default/opac/parts/result/table.tt2
@@ -25,7 +25,7 @@
                     [%  class = 'search_page_nav_link';
                         href = '#';
                         IF page > 0;
-                            href = propagator _ '&page=' _ (page - 1);
+                            href = propagator _ '&amp;page=' _ (page - 1);
                         ELSE; class = class _ ' invisible'; END;
                     %]
                     <a class='[% class %]' href='[% href %]' 
@@ -37,7 +37,7 @@
                     [%  class = 'search_page_nav_link';
                         href = '#';
                         IF (page + 1) < page_count;
-                            href = propagator _ '&page=' _ (page + 1);
+                            href = propagator _ '&amp;page=' _ (page + 1);
                         ELSE; class = class _ ' invisible'; END;
                     %]
                     <a class='[% class %]' href='[% href %]' 
@@ -91,12 +91,12 @@
                                                         href="[% ctx.opac_root %]/record/[% rec.id _ propagator %]"
                                                         class='search_link'>[% attrs.title %]</a>
                                                 </div>
-                                                <span style="font-size:11px;">
+                                                <div style="font-size:11px;">
                                                     <div>
                                                         <em><a title="[% l("Perform an Author Search") %]"
                                                                 name='item_author'
-                                                                href="[% ctx.opac_root %]/results?qtype=author&query=[% 
-                                                                    attrs.author | replace('[,\.:;]', '') | uri %]&loc=[% CGI.param('loc') | uri %]"
+                                                                href="[% ctx.opac_root %]/results?qtype=author&amp;query=[% 
+                                                                    attrs.author | replace('[,\.:;]', '') | uri %]&amp;loc=[% CGI.param('loc') | uri %]"
                                                                 class='search_link'>[% attrs.author %]</a></em>
                                                         &nbsp;&nbsp;
                                                         [% attrs.pubdate %]
@@ -142,7 +142,7 @@
                                                             #rec.copy_counts.available, rec.copy_counts.visible) 
                                                         %]
                                                     </div>
-                                                </span>
+                                                </div>
                                                 <div class="hide_me">
                                                     <span name='result_table_extra_span' class='hide_me'>
                                                         <span name='result_table_pub_box'
@@ -177,7 +177,7 @@
                                                 <div style="width:250px;text-align:left;">
                                                     <div style="float:right;">
                                                         <div class="results_aux_utils opac-auto-010"><a
-                                                                href="[% ctx.opac_root %]/place_hold[% propagator; propagator.length > 1 ? "&" : ""; %]hold_target=[% rec.id %]&hold_type=T" 
+                                                                href="[% ctx.opac_root %]/place_hold[% propagator; propagator.length > 1 ? "&amp;" : ""; %]hold_target=[% rec.id %]&amp;hold_type=T" 
                                                                     name="place_hold_link" class="no-dec"><img
                                                                 src="[% ctx.media_prefix %]/images/green_check.png"
                                                                 alt="[% l('Place hold') %]"/><span style="position:relative;top:-3px;left:3px;">[% l('Place Hold') %]</span></a>
-- 
2.11.0