From a1f619bd3748c52ea177e52898bdf7b27826e007 Mon Sep 17 00:00:00 2001 From: Dan Scott Date: Fri, 4 Jan 2013 16:26:30 -0500 Subject: [PATCH] TPAC: Alternate graphic fields (880) display In the search results and record details displays, display the corresponding 880 field(s) whenever possible. The contents of the 880 field are generally displayed directly underneath the corresponding primary field, with the exception of the contributor credits on the record details page, in which the 880 contents appear between the name of the contributor and the contributor's credit. Signed-off-by: Dan Scott Signed-off-by: Kathy Lussier Signed-off-by: Ben Shum --- .../src/templates/opac/parts/misc_util.tt2 | 130 ++++++++++++++++-- .../templates/opac/parts/record/authors.tt2 | 18 ++- .../templates/opac/parts/record/contents.tt2 | 11 ++ .../templates/opac/parts/record/subjects.tt2 | 11 ++ .../templates/opac/parts/record/summary.tt2 | 43 ++++++ .../src/templates/opac/parts/result/table.tt2 | 67 ++++++++- 6 files changed, 268 insertions(+), 12 deletions(-) diff --git a/Open-ILS/src/templates/opac/parts/misc_util.tt2 b/Open-ILS/src/templates/opac/parts/misc_util.tt2 index c188e97f57..9e2a054d9d 100644 --- a/Open-ILS/src/templates/opac/parts/misc_util.tt2 +++ b/Open-ILS/src/templates/opac/parts/misc_util.tt2 @@ -1,4 +1,84 @@ [% + # Support multiscript records via alternate graphic 880 fields + # get_graphic_880s(target_field='100') + # See "Model A" in http://www.loc.gov/marc/bibliographic/ecbdmulti.html + # and $6 description in http://www.loc.gov/marc/bibliographic/ecbdcntf.html + MACRO get_graphic_880s BLOCK; + FOR node IN xml.findnodes('//*[@tag="' _ target_field _ '"]'); + raw_vals = []; + core_val = ''; + FOR subnode IN node.findnodes('./*[not(contains("w 0 5 6 8 9", @code))]'); + raw_vals.push(subnode.textContent()); + END; + core_val = raw_vals.join(" "); + raw_vals = []; + + linked_fields = []; + FOR sub IN node.findnodes('./*[@code="6"]'); + linked_fields.push(sub.textContent); + END; + graphics = []; + get_linked_880s; + graphic_880s.push({ + primary => {"occur" => occurrence, "value" => core_val}, + graphic => graphics + }); + END; + END; + + MACRO get_linked_880s BLOCK; + FOR link_field IN linked_fields; + target = target_field _ link_field.substr(3); + # Get the linked 880 value + raw_val = ''; + dir = ''; + occurrence = ''; + script = ''; + FOR node IN xml.findnodes('//*[@tag="880"]'); + # Operate only on the target linked fields + FOR linknode IN node.findnodes('./*[@code="6"]'); + lf = linknode.textContent(); + IF lf.substr(0, target.length) == target; + occurrence = lf.substr(4, 2); + rawscript = lf.substr(7, 2); + SWITCH rawscript; + CASE '(3'; + script = 'Arabic'; + CASE '(B'; + script = 'Latin'; + CASE '$1'; + script = 'CJK'; + CASE '(N'; + script = 'Cyrillic'; + CASE '(S'; + script = 'Greek'; + CASE '(2'; + script = 'Hebrew'; + END; + + rawdir = lf.substr(9, 1); + SWITCH rawdir; + CASE 'r'; + dir = 'rtl'; + END; + + raw_vals = []; + FOR subnode IN node.findnodes('./*[not(contains("w 0 5 6 8 9", @code))]'); + raw_vals.push(subnode.textContent()); + END; + raw_val = raw_vals.join(" "); + END; + END; + END; + graphics.push({ + occur => occurrence, + value => raw_val, + script => script, + dir => dir + }); + END; + END; + # Extract MARC fields from XML # get_marc_attrs( { marc_xml => doc } ) BLOCK get_marc_attrs; @@ -27,9 +107,12 @@ END; args.issn = (args.issns.size) ? args.issn.0 : ''; + graphic_880s = []; + get_graphic_880s(target_field='100'); + args.graphic_authors = graphic_880s; args.authors = []; - FOR sub IN xml.findnodes('//*[@tag="100"]/*[@code="a"]'); - args.authors.push(sub.textContent); + FOR author IN args.graphic_authors; + args.authors.push(author.primary.value); END; args.author = (args.authors.size) ? args.authors.0 : ''; @@ -41,11 +124,14 @@ # Avoid ugly trailing syntax on brief titles args.title = args.title | replace('[:;/]$', ''); - # Provide correct spacing between the subfields - titsubs = xml.findnodes('//*[@tag="245"]/*[@code]'); - titsubs_content = []; - FOR sub IN titsubs; titsubs_content.push(sub.textContent); END; - args.title_extended = titsubs_content.join(" "); + graphic_880s = []; + get_graphic_880s(target_field='245'); + args.graphic_titles = graphic_880s; + args.titles = []; + FOR title IN args.graphic_titles; + args.titles.push(title.primary.value); + END; + args.title_extended = (args.titles.size) ? args.titles.0 : ''; args.pubplaces = []; FOR sub IN xml.findnodes('//*[@tag="260"]/*[@code="a"]'); @@ -68,13 +154,31 @@ # Full publisher info args.pubinfo = "$args.pubplace $args.publisher $args.pubdate"; + graphic_880s = []; + get_graphic_880s(target_field='260'); + args.graphic_pubinfos = graphic_880s; + args.pubinfos = []; + FOR pubinfo IN args.graphic_pubinfos; + args.pubinfos.push(pubinfo.primary.value); + END; + args.pubinfo = (args.pubinfos.size) ? args.pubinfos.0 : ''; + args.summaries = []; FOR sub IN xml.findnodes('//*[@tag="520"]/*[@code="a"]'); args.summaries.push(sub.textContent); END; args.summary = (args.summaries.size) ? args.summaries.0 : ''; + # 250 gets pride of place for edition statement, and is the + # only logical choice for 880 graphic fields + graphic_880s = []; + get_graphic_880s(target_field='250'); + args.graphic_editions = graphic_880s; args.editions = []; + FOR edition IN args.graphic_editions; + args.editions.push(edition.primary.value); + END; + ed_hunt = xml.findnodes('//*[@tag="250"]/*[@code="a"]') || xml.findnodes('//*[@tag="534"]/*[@code="b"]') || xml.findnodes('//*[@tag="775"]/*[@code="b"]'); @@ -91,12 +195,19 @@ END; args.phys_desc = phys_content.join(" "); + graphic_880s = []; + get_graphic_880s(target_field='505'); + args.graphic_contents = graphic_880s; + FOR content IN args.graphic_contents; + args.contents.push(content.primary.value); + END; + args.content = (args.contents.size) ? args.contents.0 : ''; + + # Maintain contents_list in case any custom use was made of it args.contents_list = []; FOR sub IN xml.findnodes('//*[@tag="505"]'); args.contents_list.push(sub.textContent); END; - args.contents = args.contents_list.join(" "); - args.content = (args.contents.size) ? args.contents.0 : ''; # MARC Callnumber args.marc_cns = []; @@ -292,4 +403,5 @@ loc_name = 'locg'; loc_value = CGI.param(loc_name) || CGI.param('loc') || ctx.search_ou; END; + %] diff --git a/Open-ILS/src/templates/opac/parts/record/authors.tt2 b/Open-ILS/src/templates/opac/parts/record/authors.tt2 index 448fdf8290..b52aea3a15 100644 --- a/Open-ILS/src/templates/opac/parts/record/authors.tt2 +++ b/Open-ILS/src/templates/opac/parts/record/authors.tt2 @@ -28,6 +28,7 @@ BLOCK build_author_links; qterm = ''; iprop = ''; tlabel = ''; + graphics = []; FOR subfield IN node.childNodes; NEXT UNLESS subfield.nodeName == "subfield"; code = subfield.getAttribute('code'); @@ -35,6 +36,11 @@ BLOCK build_author_links; relcode = subfield.textContent.substr(0,3); tlabel = relators.$relcode || label; END; + IF code == '6'; + linked_fields = [subfield.textContent()]; + target_field = node.getAttribute('tag'); + get_linked_880s; + END; NEXT UNLESS code.match('[a-z]'); sf = subfield.textContent | html; term = term _ ' ' _ sf; @@ -52,7 +58,17 @@ BLOCK build_author_links; ELSIF type == 'added'; iprop = ' itemprop="contributor"'; END; - '' _ term.replace('^\s+', '') _ ' (' _ author_type _ '). '; + '' _ term.replace('^\s+', '') _ ''; + FOREACH link880 IN graphics; + diratt = ''; + IF link880.dir; + diratt = ' dir="' _ link880.dir _ '"'; + END; + ' '; + link880.value | html; + ''; + END; + ' (' _ author_type _ '). '; END; END; %] diff --git a/Open-ILS/src/templates/opac/parts/record/contents.tt2 b/Open-ILS/src/templates/opac/parts/record/contents.tt2 index 1fb0eb5833..fcf9e54b3d 100644 --- a/Open-ILS/src/templates/opac/parts/record/contents.tt2 +++ b/Open-ILS/src/templates/opac/parts/record/contents.tt2 @@ -157,15 +157,26 @@ BLOCK render_contents; xpath = xpath || '//*[starts-with(@tag,"5")]'; FOR node IN ctx.marc_xml.findnodes(xpath); all_content = []; + graphics = []; FOR subfield IN node.childNodes; NEXT UNLESS subfield.nodeName == "subfield"; code = subfield.getAttribute('code'); + IF code == '6'; + linked_fields = [subfield.textContent()]; + target_field = node.getAttribute('tag'); + get_linked_880s; + END; NEXT UNLESS code.match('[a-z]'); all_content.push(subfield.textContent); END; total_contents = all_content.join(" ").replace('\s+$', ''); %] [% total_contents; IF total_contents.size; "
"; END; + FOREACH link880 IN graphics; + '
'; + link880.value | html; + '
'; + END; END; END %] diff --git a/Open-ILS/src/templates/opac/parts/record/subjects.tt2 b/Open-ILS/src/templates/opac/parts/record/subjects.tt2 index e92644d99b..9a1cf50dc8 100644 --- a/Open-ILS/src/templates/opac/parts/record/subjects.tt2 +++ b/Open-ILS/src/templates/opac/parts/record/subjects.tt2 @@ -31,9 +31,15 @@ xpath = xpath || '//*[starts-with(@tag,"6")]'; FOR node IN ctx.marc_xml.findnodes(xpath); all_terms = []; + graphics = []; FOR subfield IN node.childNodes; NEXT UNLESS subfield.nodeName == "subfield"; code = subfield.getAttribute('code'); + IF code == '6'; + linked_fields = [subfield.textContent()]; + target_field = node.getAttribute('tag'); + get_linked_880s; + END; NEXT UNLESS code.match('[a-z]'); IF code.match('[vxyz]'); " > "; END; # at this point, we actually have a partial term to use. @@ -44,6 +50,11 @@ [% single_term %] [%- END; IF all_terms.size; "
"; END; + FOREACH link880 IN graphics; + '
'; + link880.value | html; + '
'; + END; END; END %] diff --git a/Open-ILS/src/templates/opac/parts/record/summary.tt2 b/Open-ILS/src/templates/opac/parts/record/summary.tt2 index a5767a1f23..b70557424f 100644 --- a/Open-ILS/src/templates/opac/parts/record/summary.tt2 +++ b/Open-ILS/src/templates/opac/parts/record/summary.tt2 @@ -62,6 +62,17 @@ [%- END %]

[% attrs.title_extended | html %]

+ [%- + FOR link880 IN attrs.graphic_titles; + FOR alt IN link880.graphic; + '

'; alt.value | html; '

'; + END; + END; + -%] [%- INCLUDE "opac/parts/record/authors.tt2" %] @@ -146,6 +157,21 @@ IF num_uris > 0;
  • [% l("Edition:") %] [% attrs.edition | html %] + [%- + FOR entry IN attrs.graphic_editions; + FOR alt IN entry.graphic; + diratt = ""; + IF alt.dir; + diratt = ' dir="' _ alt.dir _ '"'; + END; + -%] +
    + [% alt.value | html %] +
    + [%- + END; + END; + -%]
    • [%- END %] [%- IF attrs.publisher %] @@ -160,6 +186,23 @@ IF num_uris > 0; [%- IF attrs.pubdate; %] [%- END; %] + [%- + IF attrs.graphic_pubinfos.size > 0; + FOR entry IN attrs.graphic_pubinfos; + FOR alt IN entry.graphic; + diratt = ""; + IF alt.dir; + diratt = ' dir="' _ alt.dir _ '"'; + END; + -%] +
    + [% alt.value | html %] +
    + [%- + END; + END; + END + -%] [%- END %] diff --git a/Open-ILS/src/templates/opac/parts/result/table.tt2 b/Open-ILS/src/templates/opac/parts/result/table.tt2 index d53c8f25cb..e0f3f2a9d2 100644 --- a/Open-ILS/src/templates/opac/parts/result/table.tt2 +++ b/Open-ILS/src/templates/opac/parts/result/table.tt2 @@ -54,6 +54,21 @@ [% HTML.attributes(title => l('Display record details for "[_1]"', attrs.title)) %] class='search_link'>[% attrs.title | html %] +[%- +FOR entry IN attrs.graphic_titles; + FOR alt IN entry.graphic; + diratt = ""; + IF alt.dir; + diratt = ' dir="' _ alt.dir _ '"'; + END; +-%] +
    + [% alt.value | html %] +
    +[%- + END; +END; +-%]
    +[%- + END; +END; +-%] +
    @@ -87,7 +118,23 @@ - + [% END %] [% IF attrs.isbns.size > 0 %] @@ -128,7 +175,23 @@ - + [% END %] [% IF attrs.phys_desc %] -- 2.43.2
    [% l('Publisher:') %] [% attrs.pubinfo | html %][% attrs.pubinfo | html %] +[%- +FOR entry IN attrs.graphic_pubinfos; + FOR alt IN entry.graphic; + diratt = ""; + IF alt.dir; + diratt = ' dir="' _ alt.dir _ '"'; + END; +-%] +
    + [% alt.value | html %] +
    +[%- + END; +END; +-%] +
    [% l('Edition:') %] [% attrs.edition | html %][% attrs.edition | html %] +[%- +FOR entry IN attrs.graphic_editions; + FOR alt IN entry.graphic; + diratt = ""; + IF alt.dir; + diratt = ' dir="' _ alt.dir _ '"'; + END; +-%] +
    + [% alt.value | html %] +
    +[%- + END; +END; +-%] +