[Koha-patches] [PATCH] Bug 10218 - In OPAC XSLT search results, add class to 'online access'
Owen Leonard
oleonard at myacpl.org
Mon May 13 21:15:27 CEST 2013
In some cases in OPAC search results and the detail page there is a
class specific to each line of data being output (publisher, date,
subjects, etc). In other cases there is no additional class. This patch
makes modifications to a few different OPAC files in an attempt to make
them consistent with each other.
To test, apply the patch and view:
- an OPAC detail page with OPACXSLTDetailsDisplay off
- the OPAC search results page with OPACXSLTDetailsDisplay off
- the OPAC search results page with OPACXSLTDetailsDisplay on
Labels should have classes relevant to their data. To test the specific
case requested by this bug, view details/results for a record which
includes online resources (856u in MARC21).
---
koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt | 36 ++++++++++----------
.../opac-tmpl/prog/en/modules/opac-results.tt | 10 +++---
.../prog/en/xslt/MARC21slim2OPACResults.xsl | 18 +++++-----
3 files changed, 32 insertions(+), 32 deletions(-)
diff --git a/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt b/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt
index a8dabf3..0d1a04d 100644
--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt
+++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt
@@ -591,7 +591,7 @@ YAHOO.util.Event.onContentReady("furtherm", function () {
[% END %]
[% IF ( MARCSERIES ) %]
- <span class="results_summary"><span class="label">Series:</span>
+ <span class="results_summary series"><span class="label">Series:</span>
[% FOREACH MARCSERIE IN MARCSERIES %]
[% FOREACH MARCSERIES_SUBFIELDS_LOO IN MARCSERIE.MARCSERIES_SUBFIELDS_LOOP %]
[% IF ( MARCSERIES_SUBFIELDS_LOO.value ) %]<a href="/cgi-bin/koha/opac-search.pl?q=se:[% MARCSERIES_SUBFIELDS_LOO.value |url %]">[% MARCSERIES_SUBFIELDS_LOO.value %]</a>[% END %][% IF ( MARCSERIES_SUBFIELDS_LOO.volumenum ) %]. [% MARCSERIES_SUBFIELDS_LOO.volumenum %][% END %]
@@ -600,17 +600,17 @@ YAHOO.util.Event.onContentReady("furtherm", function () {
</span>
[% END %]
[% IF ( editionstatement ) %]
- <span class="results_summary"><span class="label">Edition statement:</span>[% editionstatement %][% IF ( editionresponsibility ) %]/[% editionresponsibility %][% END %]</span>
+ <span class="results_summary edition"><span class="label">Edition statement:</span>[% editionstatement %][% IF ( editionresponsibility ) %]/[% editionresponsibility %][% END %]</span>
[% END %]
[% IF ( publishercode ) %]
- <span class="results_summary"><span class="label">Published by :</span>
+ <span class="results_summary publisher"><span class="label">Published by :</span>
<a href="/cgi-bin/koha/opac-search.pl?q=pb:[% publishercode |url %] ">
[% publishercode |html %]
</a> [% IF ( place ) %]([% place %])[% END %] [% IF ( publicationyear ) %], [% publicationyear %][% END %]
</span>
[% END %]
- [% IF ( pages ) %]<span class="results_summary"><span class="label">Physical details:</span> [% pages %] [% illus %] [% size %]</span>[% END %]
+ [% IF ( pages ) %]<span class="results_summary physical"><span class="label">Physical details:</span> [% pages %] [% illus %] [% size %]</span>[% END %]
[% IF ( MARCISBNS ) %]
<span class="results_summary isbn"><span class="label">ISBN:</span>[% FOREACH MARCISBN IN MARCISBNS %][% MARCISBN.marcisbn %][% IF ( loop.last ) %].[% ELSE %]; [% END %][% END %]</span>
@@ -620,10 +620,10 @@ YAHOO.util.Event.onContentReady("furtherm", function () {
[% END %]
[% END %]
[% IF ( issn ) %]
- <span class="results_summary"><span class="label">ISSN:</span>[% issn %]</span>
+ <span class="results_summary issn"><span class="label">ISSN:</span>[% issn %]</span>
[% END %]
[% IF ( collectiontitle ) %]
- <span class="results_summary"><span class="label">Collection: </span>
+ <span class="results_summary collection"><span class="label">Collection: </span>
[% collectiontitle %]
[% IF ( collectionissn ) %], ISSN [% collectionissn %] [% END %]
[% IF ( collectionvolume ) %]; [% collectionvolume %] [% END %]
@@ -631,7 +631,7 @@ YAHOO.util.Event.onContentReady("furtherm", function () {
</span>
[% END %]
[% IF ( unititles ) %]
- <span class="results_summary"><span class="label">Uniform titles:</span>
+ <span class="results_summary unititle"><span class="label">Uniform titles:</span>
[% FOREACH unititle IN unititles %]
[% unititle.unititle %] ;
[% END %]
@@ -640,7 +640,7 @@ YAHOO.util.Event.onContentReady("furtherm", function () {
[% END %]
[% IF ( MARCSUBJCTS ) %]
- <span class="results_summary">
+ <span class="results_summary subject">
<span class="label">Subject(s):</span>
[% IF OPACPopupAuthorsSearch %]
<div class="subjectSearch" >
@@ -694,14 +694,14 @@ YAHOO.util.Event.onContentReady("furtherm", function () {
[% END %]
</span>
[% ELSE %]
- [% IF ( subjects ) %]<span class="results_summary"><span class="label">Subject(s):</span> [% FOREACH subject IN subjects %]<a href="/cgi-bin/koha/opac-search.pl?q=su:[% subject.subject |url %]">[% subject.subject %]</a> | [% END %]</span>[% END %]
+ [% IF ( subjects ) %]<span class="results_summary subject"><span class="label">Subject(s):</span> [% FOREACH subject IN subjects %]<a href="/cgi-bin/koha/opac-search.pl?q=su:[% subject.subject |url %]">[% subject.subject %]</a> | [% END %]</span>[% END %]
[% END %]
[% IF ( copyrightdate ) %]
- <span class="results_summary"><span class="label">Year: </span>[% copyrightdate %]</span>
+ <span class="results_summary year"><span class="label">Year: </span>[% copyrightdate %]</span>
[% END %]
[% UNLESS ( item_level_itypes ) %]<img src="[% imageurl %]" alt="[% description %]" title="[% description %]" />
- <span class="results_summary"><span class="label">Item type: </span>
+ <span class="results_summary itemtype"><span class="label">Item type: </span>
[% IF ( description ) %]
[% description %]
[% ELSE %]
@@ -710,7 +710,7 @@ YAHOO.util.Event.onContentReady("furtherm", function () {
</span>[% END %]
[% IF ( MARCURLS ) %]
- <div class="results_summary"><span class="label">Online resources:</span>
+ <div class="results_summary online_resources"><span class="label">Online resources:</span>
<ul> [% FOREACH MARCurl IN MARCURLS %]
<li>[% IF ( MARCurl.part ) %][% MARCurl.part %]
<br />[% END %]
@@ -718,13 +718,13 @@ YAHOO.util.Event.onContentReady("furtherm", function () {
[% IF ( OPACurlOpenInNewWindow ) %]
[% IF trackclicks == 'track' || trackclicks == 'anonymous' %]
- <a href="/cgi-bin/koha/tracklinks.pl?uri=[% MARCurl.MARCURL |url%]&biblionumber=[% biblionumber | url %]" title="[% MARCurl.MARCURL %]" target="_blank">
+ <a href="/cgi-bin/koha/tracklinks.pl?uri=[% MARCurl.MARCURL |url%]&biblionumber=[% biblionumber | url %]" title="[% MARCurl.MARCURL %]" target="_blank">
[% ELSE %]
<a href="[% MARCurl.MARCURL %]" title="[% MARCurl.MARCURL %]" target="_blank">
[% END %]
[% ELSE %]
[% IF trackclicks == 'track' || trackclicks == 'anonymous' %]
- <a href="/cgi-bin/koha/tracklinks.pl?uri=[% MARCurl.MARCURL |url%]&biblionumber=[% biblionumber | url %]" title="[% MARCurl.MARCURL %]">
+ <a href="/cgi-bin/koha/tracklinks.pl?uri=[% MARCurl.MARCURL |url%]&biblionumber=[% biblionumber | url %]" title="[% MARCurl.MARCURL %]">
[% ELSE %]
<a href="[% MARCurl.MARCURL %]" title="[% MARCurl.MARCURL %]">
[% END %]
@@ -761,7 +761,7 @@ YAHOO.util.Event.onContentReady("furtherm", function () {
<!--This grabs all of the lists a bib record appears in -->
[% IF ( GetShelves ) %]
- <span class="results_summary"><span class="label">List(s) this item appears in: </span>
+ <span class="results_summary lists"><span class="label">List(s) this item appears in: </span>
[% FOREACH GetShelve IN GetShelves %]
<a href="/cgi-bin/koha/opac-shelves.pl?viewshelf=[% GetShelve.shelfnumber %]">[% GetShelve.shelfname %]</a>
[% IF ( loop.last ) %][% ELSE %]|[% END %]
@@ -770,7 +770,7 @@ YAHOO.util.Event.onContentReady("furtherm", function () {
[% END %]
[% IF ( TagsShowEnabled ) %]
- <div class="results_summary">
+ <div class="results_summary tags">
[% IF ( TagLoop ) %]
<span class="label">Tags from this library:</span>
<ul id="tagslist">[% FOREACH TagLoo IN TagLoop %]
@@ -781,7 +781,7 @@ YAHOO.util.Event.onContentReady("furtherm", function () {
</div>
[% END %]
[% IF ( TagsInputEnabled ) %]
- <div class="results_summary">
+ <div class="results_summary tagsinput">
[% IF ( loggedinusername ) %]
<a class="tag_add" id="tag_add[% biblionumber %]" href="#">Add</a>
<form id="tagform[% biblionumber %]" method="post" action="/cgi-bin/koha/opac-tags.pl" style="display:none;">
@@ -839,7 +839,7 @@ YAHOO.util.Event.onContentReady("furtherm", function () {
[% IF ( OpacStarRatings != 'disable' ) %]
<form method="post" action="/cgi-bin/koha/opac-ratings.pl">
- <div class="results_summary">
+ <div class="results_summary ratings">
[% FOREACH i IN [ 1 2 3 4 5 ] %]
[% IF rating_avg == i && borrowernumber %]
diff --git a/koha-tmpl/opac-tmpl/prog/en/modules/opac-results.tt b/koha-tmpl/opac-tmpl/prog/en/modules/opac-results.tt
index 0704f06..6e431cd 100644
--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-results.tt
+++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-results.tt
@@ -514,7 +514,7 @@ $(document).ready(function(){
[% IF ( SEARCH_RESULT.author ) %]by <a href="/cgi-bin/koha/opac-search.pl?q=au:[% SEARCH_RESULT.author |url %]" title="Search for works by this author" class="author">[% SEARCH_RESULT.author %]</a>
[% ELSE %]
[% END %]
- <span class="results_summary"><span class="label">Publication:</span>
+ <span class="results_summary publisher"><span class="label">Publication:</span>
[% IF ( SEARCH_RESULT.place ) %][% SEARCH_RESULT.place %] [% END %][% IF ( SEARCH_RESULT.publishercode ) %][% SEARCH_RESULT.publishercode|html %][% END %][% IF ( SEARCH_RESULT.publicationyear ) %] [% SEARCH_RESULT.publicationyear %]
[% ELSE %][% IF ( SEARCH_RESULT.copyrightdate ) %] [% SEARCH_RESULT.copyrightdate %][% END %][% END %]
[% IF ( SEARCH_RESULT.pages ) %]. [% SEARCH_RESULT.pages %][% END %]
@@ -527,9 +527,9 @@ $(document).ready(function(){
[% SEARCH_RESULT.summary %]
</span>
[% END %]
- [% IF ( SEARCH_RESULT.copyrightdate ) %]<span class="results_summary"><span class="label">Date:</span>[% SEARCH_RESULT.copyrightdate %]</span>[% END %]
+ [% IF ( SEARCH_RESULT.copyrightdate ) %]<span class="results_summary date"><span class="label">Date:</span>[% SEARCH_RESULT.copyrightdate %]</span>[% END %]
- <span class="results_summary">
+ <span class="results_summary availability">
<span class="label">Availability:</span>
[% IF ( SEARCH_RESULT.available_items_loop.size() ) %]
<span class="available"><strong>Copies available:</strong>
@@ -591,7 +591,7 @@ $(document).ready(function(){
[% IF ( LibraryThingForLibrariesID ) %]<div class="ltfl_reviews"></div>[% END %]
[% IF ( TagsShowEnabled ) %]
[% IF ( SEARCH_RESULT.TagLoop.size ) %]
- <div class="results_summary"><span class="label">Tags:</span>
+ <div class="results_summary tags"><span class="label">Tags:</span>
<ul style="display: inline; list-style: none;">[% FOREACH TagLoo IN SEARCH_RESULT.TagLoop %]<li style="display: inline; list-style: none;"><a href="/cgi-bin/koha/opac-search.pl?tag=[% TagLoo.term |url %]&q=[% TagLoo.term |url %]">[% TagLoo.term %]</a> <span class="weight">([% TagLoo.weight_total %])</span></li>
[% END %]
</ul>
@@ -604,7 +604,7 @@ $(document).ready(function(){
[% IF ( OpacStarRatings == 'all' ) %]
- <div class="results_summary">
+ <div class="results_summary ratings">
[% FOREACH i IN [ 1 2 3 4 5 ] %]
diff --git a/koha-tmpl/opac-tmpl/prog/en/xslt/MARC21slim2OPACResults.xsl b/koha-tmpl/opac-tmpl/prog/en/xslt/MARC21slim2OPACResults.xsl
index cdeb672..20fdabf 100644
--- a/koha-tmpl/opac-tmpl/prog/en/xslt/MARC21slim2OPACResults.xsl
+++ b/koha-tmpl/opac-tmpl/prog/en/xslt/MARC21slim2OPACResults.xsl
@@ -509,7 +509,7 @@
</p>
<xsl:if test="marc:datafield[@tag=250]">
- <span class="results_summary">
+ <span class="results_summary edition">
<span class="label">Edition: </span>
<xsl:for-each select="marc:datafield[@tag=250]">
<xsl:call-template name="subfieldSelect">
@@ -522,7 +522,7 @@
<xsl:if test="marc:datafield[@tag=773]">
<xsl:for-each select="marc:datafield[@tag=773]">
<xsl:if test="marc:subfield[@code='t']">
- <span class="results_summary">
+ <span class="results_summary source">
<span class="label">Source: </span>
<xsl:value-of select="marc:subfield[@code='t']"/>
</span>
@@ -531,7 +531,7 @@
</xsl:if>
<xsl:if test="$DisplayOPACiconsXSLT!='0'">
- <span class="results_summary">
+ <span class="results_summary type">
<xsl:if test="$typeOf008!=''">
<span class="label">Type: </span>
<xsl:choose>
@@ -881,7 +881,7 @@
<xsl:call-template name="m880Select">
<xsl:with-param name="basetags">260</xsl:with-param>
<xsl:with-param name="codes">abcg</xsl:with-param>
- <xsl:with-param name="class">results_summary</xsl:with-param>
+ <xsl:with-param name="class">results_summary publisher</xsl:with-param>
<xsl:with-param name="label">Publisher: </xsl:with-param>
</xsl:call-template>
</xsl:if>
@@ -918,13 +918,13 @@
<xsl:call-template name="m880Select">
<xsl:with-param name="basetags">246</xsl:with-param>
<xsl:with-param name="codes">ab</xsl:with-param>
- <xsl:with-param name="class">results_summary</xsl:with-param>
- <xsl:with-param name="label">Other Title: </xsl:with-param>
+ <xsl:with-param name="class">results_summary other_title</xsl:with-param>
+ <xsl:with-param name="label">Other title: </xsl:with-param>
</xsl:call-template>
</xsl:if>
<xsl:if test="marc:datafield[@tag=246]">
- <span class="results_summary">
+ <span class="results_summary other_title">
<span class="label">Other title: </span>
<xsl:for-each select="marc:datafield[@tag=246]">
<xsl:call-template name="subfieldSelect">
@@ -935,7 +935,7 @@
</span>
</xsl:if>
<xsl:if test="marc:datafield[@tag=242]">
- <span class="results_summary">
+ <span class="results_summary translated_title">
<span class="label">Title translated: </span>
<xsl:for-each select="marc:datafield[@tag=242]">
<xsl:call-template name="subfieldSelect">
@@ -946,7 +946,7 @@
</span>
</xsl:if>
<xsl:if test="marc:datafield[@tag=856]">
- <span class="results_summary">
+ <span class="results_summary online_resources">
<span class="label">Online Access: </span>
<xsl:for-each select="marc:datafield[@tag=856]">
<xsl:variable name="SubqText"><xsl:value-of select="marc:subfield[@code='q']"/></xsl:variable>
--
1.7.9.5
More information about the Koha-patches
mailing list