[Koha-patches] [PATCH] (bug #3421) fix the xhtml in serial biblio search

Nahuel ANGELINETTI nahuel.angelinetti at biblibre.com
Wed Jul 15 11:59:06 CEST 2009


This fix differents errors, and escape in bib search.
---
 .../prog/en/modules/serials/result.tmpl            |   18 +++++++++---------
 .../modules/serials/subscription-bib-search.tmpl   |    5 +++--
 2 files changed, 12 insertions(+), 11 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/result.tmpl b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/result.tmpl
index 67a2060..80b2be7 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/result.tmpl
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/result.tmpl
@@ -1,7 +1,7 @@
 <!-- TMPL_INCLUDE NAME="doc-head-open.inc" -->
 <title>Koha &rsaquo; Search Results</title>
 <!-- TMPL_INCLUDE NAME="doc-head-close.inc" -->
-<script language="Javascript">
+<script type="text/javascript">
 //<![CDATA[
 function GetIt(bibno,title)
 {
@@ -34,19 +34,19 @@ function GetIt(bibno,title)
         <tr>
             <td>
                 <a href="#" onclick="GetIt('<!-- TMPL_VAR NAME="biblionumber" -->','<!-- TMPL_VAR escape="URL" NAME="title" -->');" title="Get this one !">
-                    <!-- TMPL_VAR NAME="title" -->
+                    <!-- TMPL_VAR ESCAPE="HTML" NAME="title" -->
                 </a>
                 <br />
-                <small><!-- TMPL_VAR NAME="subtitle" --></small>
+                <small><!-- TMPL_VAR ESCAPE="HTML" NAME="subtitle" --></small>
             </td>
             <td>
-                <!-- TMPL_VAR NAME="author" -->
+                <!-- TMPL_VAR ESCAPE="HTML" NAME="author" -->
             </td>
             <td>
-                <!-- TMPL_VAR NAME="publishercode" -->
+                <!-- TMPL_VAR ESCAPE="HTML" NAME="publishercode" -->
             </td>
             <td>
-                <!-- TMPL_VAR NAME="publicationyear" -->
+                <!-- TMPL_VAR ESCAPE="HTML" NAME="publicationyear" -->
             </td>
         </tr>
     <!-- /TMPL_LOOP -->
@@ -55,14 +55,14 @@ function GetIt(bibno,title)
 <h2>No results found for <b><!-- TMPL_VAR NAME="query" --></b></h2>
 <!-- /TMPL_IF-->
 <!-- TMPL_IF NAME="displayprev" -->
-<a href="/cgi-bin/koha/serials/subscription-bib-search.pl?op=do_search&type=intranet&startfrom=<!-- TMPL_VAR ESCAPE="URL" NAME="startfromprev" -->&q=<!-- TMPL_VAR ESCAPE="URL" NAME="query" -->">&lt;&lt;</a>
+<a href="/cgi-bin/koha/serials/subscription-bib-search.pl?op=do_search&amp;type=intranet&amp;startfrom=<!-- TMPL_VAR ESCAPE="URL" NAME="startfromprev" -->&amp;q=<!-- TMPL_VAR ESCAPE="URL" NAME="query" -->">&lt;&lt;</a>
 <!-- /TMPL_IF -->
 <!-- TMPL_IF NAME="displaynext" -->
-<a href="/cgi-bin/koha/serials/subscription-bib-search.pl?op=do_search&type=intranet&startfrom=<!-- TMPL_VAR ESCAPE="URL" NAME="startfromnext" -->&q=<!-- TMPL_VAR ESCAPE="URL" NAME="query" -->">&gt;&gt;</a>
+<a href="/cgi-bin/koha/serials/subscription-bib-search.pl?op=do_search&amp;type=intranet&amp;startfrom=<!-- TMPL_VAR ESCAPE="URL" NAME="startfromnext" -->&amp;q=<!-- TMPL_VAR ESCAPE="URL" NAME="query" -->">&gt;&gt;</a>
 <!-- /TMPL_IF -->
 <br/>
 <a href="subscription-bib-search.pl">Search for another Biblio</a>
-<a class="button" href="#" onclick="window.close()">  Close</a></p>
+<a class="button" href="#" onclick="window.close()">  Close</a>
 </div>
 
 <!-- TMPL_INCLUDE NAME="intranet-bottom.inc" -->
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-bib-search.tmpl b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-bib-search.tmpl
index a8cf0cf..44a0ca5 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-bib-search.tmpl
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-bib-search.tmpl
@@ -23,8 +23,9 @@
         <select name="itemtypelimit">
 	    <option value="">All</option>
         <!--TMPL_LOOP Name="itemtypeloop"-->
-            <option value="<!--TMPL_VAR Name="code" -->" />
-            <!--TMPL_VAR Name="description" -->
+            <option value="<!--TMPL_VAR ESCAPE="HTML" Name="code" -->">
+            	<!--TMPL_VAR ESCAPE="HTML" Name="description" -->
+            </option>
         <!--/TMPL_LOOP-->
         </select>
         </p>
-- 
1.6.0.4




More information about the Koha-patches mailing list