From 45168efeb9a2c6e3268e2f002ca31af8e5e6afc8 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 --- .../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 ca6fb76a65..1effd07b9f 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 ) %] - 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 %] @@ -614,17 +614,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 %] @@ -634,10 +634,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 %] @@ -645,7 +645,7 @@ YAHOO.util.Event.onContentReady("furtherm", function () { [% END %] [% IF ( unititles ) %] - Uniform titles: + Uniform titles: [% FOREACH unititle IN unititles %] [% unititle.unititle %] ; [% END %] @@ -654,7 +654,7 @@ YAHOO.util.Event.onContentReady("furtherm", function () { [% END %] [% IF ( MARCSUBJCTS ) %] - + Subject(s): [% IF OPACPopupAuthorsSearch %]
@@ -708,14 +708,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 %] @@ -724,7 +724,7 @@ YAHOO.util.Event.onContentReady("furtherm", function () { [% END %] [% IF ( MARCURLS ) %] -
Online resources: +
Online resources:
    [% FOREACH MARCurl IN MARCURLS %]
  • [% IF ( MARCurl.part ) %][% MARCurl.part %]
    [% END %] @@ -732,13 +732,13 @@ YAHOO.util.Event.onContentReady("furtherm", function () { [% IF ( OPACurlOpenInNewWindow ) %] [% IF trackclicks == 'track' || trackclicks == 'anonymous' %] - + [% ELSE %] [% END %] [% ELSE %] [% IF trackclicks == 'track' || trackclicks == 'anonymous' %] - + [% ELSE %] [% END %] @@ -775,7 +775,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 %] @@ -784,7 +784,7 @@ YAHOO.util.Event.onContentReady("furtherm", function () { [% END %] [% IF ( TagsShowEnabled ) %] -
    +
    [% IF ( TagLoop ) %] Tags from this library:
      [% FOREACH TagLoo IN TagLoop %] @@ -795,7 +795,7 @@ YAHOO.util.Event.onContentReady("furtherm", function () {
    [% END %] [% IF ( TagsInputEnabled ) %] -
    +
    [% IF ( loggedinusername ) %] Add