From 0657257f828615893173f0dd131e1bd87b5eee18 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 (cherry picked from commit cf96dd5f53eea3b585ea8a38b916a30b89c24bea) Solved conflicts in: koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt Signed-off-by: Bernardo Gonzalez Kriegel (cherry picked from commit cf96dd5f53eea3b585ea8a38b916a30b89c24bea) --- .../opac-tmpl/prog/en/modules/opac-detail.tt | 32 +++++++++---------- .../opac-tmpl/prog/en/modules/opac-results.tt | 10 +++--- .../prog/en/xslt/MARC21slim2OPACResults.xsl | 18 +++++------ 3 files changed, 30 insertions(+), 30 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 94db17df67..6f1134ebd0 100644 --- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt +++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt @@ -392,7 +392,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 %] @@ -401,17 +401,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 %] @@ -421,10 +421,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 %] @@ -432,7 +432,7 @@ YAHOO.util.Event.onContentReady("furtherm", function () { [% END %] [% IF ( unititles ) %] - Uniform titles: + Uniform titles: [% FOREACH unititle IN unititles %] [% unititle.unititle %] ; [% END %] @@ -441,7 +441,7 @@ YAHOO.util.Event.onContentReady("furtherm", function () { [% END %] [% IF ( MARCSUBJCTS ) %] - + Subject(s): [% FOREACH MARCSUBJCT IN MARCSUBJCTS %] [% subjectsloop = loop %] @@ -456,14 +456,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 %] @@ -472,7 +472,7 @@ YAHOO.util.Event.onContentReady("furtherm", function () { [% END %] [% IF ( MARCURLS ) %] -
Online resources: +
Online resources:
    [% FOREACH MARCurl IN MARCURLS %]
  • [% IF ( MARCurl.part ) %][% MARCurl.part %]
    [% END %] @@ -511,7 +511,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 %] @@ -520,7 +520,7 @@ YAHOO.util.Event.onContentReady("furtherm", function () { [% END %] [% IF ( TagsShowEnabled ) %] -
    +
    [% IF ( TagLoop ) %] Tags from this library:
      [% FOREACH TagLoo IN TagLoop %] @@ -531,7 +531,7 @@ YAHOO.util.Event.onContentReady("furtherm", function () {
    [% END %] [% IF ( TagsInputEnabled ) %] -
    +
    [% IF ( loggedinusername ) %] Add