Bug 10587: fix HTML errors on authority detail page in OPAC
authorOwen Leonard <oleonard@myacpl.org>
Fri, 9 Aug 2013 15:41:41 +0000 (11:41 -0400)
committerGalen Charlton <gmc@esilibrary.com>
Wed, 28 Aug 2013 13:47:12 +0000 (13:47 +0000)
A validator check of the OPAC's authority detail returns a few minor
errors. This patch corrects the errors.

To test, apply the patch and run an OPAC authority detail page through
an HTML validator. There should be no errors which are specific to the
authorities detail template (as opposed to header and footer includes).

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
All tests and QA script pass.
Also checked using the W3C validator and testing the
authority detail page in the OPAC is still correctly
displayed.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
koha-tmpl/opac-tmpl/prog/en/modules/opac-auth-detail.tt

index 8367b04..78bc5b9 100644 (file)
@@ -3,9 +3,9 @@
 [% INCLUDE 'doc-head-open.inc' %][% IF ( LibraryNameTitle ) %][% LibraryNameTitle %][% ELSE %]Koha online[% END %] catalog &rsaquo;  Entry
 [% INCLUDE 'doc-head-close.inc' %]
 [% IF ( displayhierarchy ) %]
-<script language="javascript" type="text/javascript" src="/opac-tmpl/lib/jquery/plugins/jquery.jstree.js"></script>
+<script type="text/javascript" src="/opac-tmpl/lib/jquery/plugins/jquery.jstree.js"></script>
 [% END %]
-<script language="javascript" type="text/javascript">
+<script type="text/javascript">
 //<![CDATA[
 $(document).ready(function() {
     $('#authdescriptions').tabs();
@@ -33,7 +33,7 @@ $(document).ready(function() {
     <div id="yui-main">
     <div class="yui-b"><div class="yui-g">
 
-    <div id="views"><span class="view"><span id="Normalview">Normal view</span></span> <span class="view"><a id="MARCview" href="/cgi-bin/koha/opac-authoritiesdetail.pl?authid=[% authid %]&marc=1">MARC view</a></span></div>
+    <div id="views"><span class="view"><span id="Normalview">Normal view</span></span> <span class="view"><a id="MARCview" href="/cgi-bin/koha/opac-authoritiesdetail.pl?authid=[% authid %]&amp;marc=1">MARC view</a></span></div>
 
 <div id="userauthdetails" class="container">
 
@@ -61,6 +61,7 @@ $(document).ready(function() {
         <span class="seefrom">[% PROCESS authheadingdisplay heading=seefro.heading search=seefrosearch authid=seefro.authid %]</span>
         </li>
     [% END %]
+    </ul>
     [% PROCESS otherscript headings=summary.otherscript wantcategory='seefrom' %]
     </div>
 [% END %]
@@ -107,5 +108,5 @@ $(document).ready(function() {
 </div>
 </div>
 </div>
-</div>
+
 [% INCLUDE 'opac-bottom.inc' %]