From cf96dd5f53eea3b585ea8a38b916a30b89c24bea Mon Sep 17 00:00:00 2001 From: Owen Leonard 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 html changes only, passes all tests Signed-off-by: Chris Cormack Signed-off-by: Galen Charlton (cherry picked from commit 45168efeb9a2c6e3268e2f002ca31af8e5e6afc8) Signed-off-by: Tomas Cohen Arazi --- .../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 e1037732a8..390a9f061c 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 ) %] - Series: + Series: [% FOREACH MARCSERIE IN MARCSERIES %] [% FOREACH MARCSERIES_SUBFIELDS_LOO IN MARCSERIE.MARCSERIES_SUBFIELDS_LOOP %] [% IF ( MARCSERIES_SUBFIELDS_LOO.value ) %][% MARCSERIES_SUBFIELDS_LOO.value %][% END %][% IF ( MARCSERIES_SUBFIELDS_LOO.volumenum ) %]. [% MARCSERIES_SUBFIELDS_LOO.volumenum %][% END %] @@ -600,17 +600,17 @@ YAHOO.util.Event.onContentReady("furtherm", function () { [% END %] [% IF ( editionstatement ) %] - Edition statement:[% editionstatement %][% IF ( editionresponsibility ) %]/[% editionresponsibility %][% END %] + Edition statement:[% editionstatement %][% IF ( editionresponsibility ) %]/[% editionresponsibility %][% END %] [% END %] [% IF ( publishercode ) %] - Published by : + Published by : [% publishercode |html %] [% IF ( place ) %]([% place %])[% END %] [% IF ( publicationyear ) %], [% publicationyear %][% END %] [% END %] - [% IF ( pages ) %]Physical details: [% pages %] [% illus %] [% size %][% END %] + [% IF ( pages ) %]Physical details: [% pages %] [% illus %] [% size %][% END %] [% IF ( MARCISBNS ) %] ISBN:[% FOREACH MARCISBN IN MARCISBNS %][% MARCISBN.marcisbn %][% IF ( loop.last ) %].[% ELSE %]; [% END %][% END %] @@ -620,10 +620,10 @@ YAHOO.util.Event.onContentReady("furtherm", function () { [% END %] [% END %] [% IF ( issn ) %] - ISSN:[% issn %] + ISSN:[% issn %] [% END %] [% IF ( collectiontitle ) %] - Collection: + Collection: [% collectiontitle %] [% IF ( collectionissn ) %], ISSN [% collectionissn %] [% END %] [% IF ( collectionvolume ) %]; [% collectionvolume %] [% END %] @@ -631,7 +631,7 @@ YAHOO.util.Event.onContentReady("furtherm", function () { [% END %] [% IF ( unititles ) %] - Uniform titles: + Uniform titles: [% FOREACH unititle IN unititles %] [% unititle.unititle %] ; [% END %] @@ -640,7 +640,7 @@ YAHOO.util.Event.onContentReady("furtherm", function () { [% END %] [% IF ( MARCSUBJCTS ) %] - + Subject(s): [% IF OPACPopupAuthorsSearch %]
@@ -694,14 +694,14 @@ YAHOO.util.Event.onContentReady("furtherm", function () { [% END %] [% ELSE %] - [% IF ( subjects ) %]Subject(s): [% FOREACH subject IN subjects %][% subject.subject %] | [% END %][% END %] + [% IF ( subjects ) %]Subject(s): [% FOREACH subject IN subjects %][% subject.subject %] | [% END %][% END %] [% END %] [% IF ( copyrightdate ) %] - Year: [% copyrightdate %] + Year: [% copyrightdate %] [% END %] [% UNLESS ( item_level_itypes ) %][% description %] - Item type: + Item type: [% IF ( description ) %] [% description %] [% ELSE %] @@ -710,7 +710,7 @@ YAHOO.util.Event.onContentReady("furtherm", function () { [% END %] [% IF ( MARCURLS ) %] -
Online resources: +
Online resources:
    [% FOREACH MARCurl IN MARCURLS %]
  • [% IF ( MARCurl.part ) %][% MARCurl.part %]
    [% END %] @@ -718,13 +718,13 @@ YAHOO.util.Event.onContentReady("furtherm", function () { [% IF ( OPACurlOpenInNewWindow ) %] [% IF trackclicks == 'track' || trackclicks == 'anonymous' %] - + [% ELSE %] [% END %] [% ELSE %] [% IF trackclicks == 'track' || trackclicks == 'anonymous' %] - + [% ELSE %] [% END %] @@ -761,7 +761,7 @@ YAHOO.util.Event.onContentReady("furtherm", function () { [% IF ( GetShelves ) %] - List(s) this item appears in: + List(s) this item appears in: [% FOREACH GetShelve IN GetShelves %] [% GetShelve.shelfname %] [% IF ( loop.last ) %][% ELSE %]|[% END %] @@ -770,7 +770,7 @@ YAHOO.util.Event.onContentReady("furtherm", function () { [% END %] [% IF ( TagsShowEnabled ) %] -
    +
    [% IF ( TagLoop ) %] Tags from this library:
      [% FOREACH TagLoo IN TagLoop %] @@ -781,7 +781,7 @@ YAHOO.util.Event.onContentReady("furtherm", function () {
    [% END %] [% IF ( TagsInputEnabled ) %] -
    +
    [% IF ( loggedinusername ) %] Add