Bugzilla – Attachment 18295 Details for
Bug 10218
Add individual CSS classes for bibliographic information on OPAC results and detail pages (UNIMARC)
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 10218 - In OPAC XSLT search results, add class to 'online access'
Bug-10218---In-OPAC-XSLT-search-results-add-class-.patch (text/plain), 16.13 KB, created by
Galen Charlton
on 2013-05-22 00:18:22 UTC
(
hide
)
Description:
Bug 10218 - In OPAC XSLT search results, add class to 'online access'
Filename:
MIME Type:
Creator:
Galen Charlton
Created:
2013-05-22 00:18:22 UTC
Size:
16.13 KB
patch
obsolete
>From 03b06bf131914b776b4ac9ddb69b9fd95e7bd38f Mon Sep 17 00:00:00 2001 >From: Owen Leonard <oleonard@myacpl.org> >Date: Mon, 13 May 2013 15:07:34 -0400 >Subject: [PATCH] Bug 10218 - In OPAC XSLT search results, add class to 'online access' > >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). > >Signed-off-by: Galen Charlton <gmc@esilibrary.com> >--- > 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 ca6fb76..1effd07 100644 >--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt >+++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt >@@ -605,7 +605,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 %] >@@ -614,17 +614,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> >@@ -634,10 +634,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 %] >@@ -645,7 +645,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 %] >@@ -654,7 +654,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" > >@@ -708,14 +708,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 %] >@@ -724,7 +724,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 %] >@@ -732,13 +732,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 %] >@@ -775,7 +775,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 %] >@@ -784,7 +784,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 %] >@@ -795,7 +795,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;"> >@@ -853,7 +853,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 85a4d45..923effa 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.2.5
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 10218
:
18028
|
18119
|
18295
|
18402
|
18948
|
20678
|
20679
|
20861