From: Dan Scott <dan@coffeecode.net>
Date: Wed, 4 May 2011 00:42:16 +0000 (-0400)
Subject: HTML compliance in result table
X-Git-Tag: sprint4-merge-nov22~5128^2~72^2~60^2~1^2~2
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=6b09dfbc8b23d534cbd48e72325f073fbbb28987;p=working%2FEvergreen.git

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>
---

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>