Bugzilla – Attachment 42220 Details for
Bug 14770
Use $9 in GetMarcSeries to build search link
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 14770: Use $9 in GetMarcSeries to build search link
Bug-14770-Use-9-in-GetMarcSeries-to-build-search-l.patch (text/plain), 4.79 KB, created by
Julian Maurice
on 2015-09-02 12:44:30 UTC
(
hide
)
Description:
Bug 14770: Use $9 in GetMarcSeries to build search link
Filename:
MIME Type:
Creator:
Julian Maurice
Created:
2015-09-02 12:44:30 UTC
Size:
4.79 KB
patch
obsolete
>From c5599b7b399e9d588746dfdfb0a449862ca7f6fe Mon Sep 17 00:00:00 2001 >From: Julian Maurice <julian.maurice@biblibre.com> >Date: Wed, 2 Sep 2015 13:56:39 +0200 >Subject: [PATCH] Bug 14770: Use $9 in GetMarcSeries to build search link > >The behaviour is the same as in GetMarcAuthors. >Also skip $3 subfields if UNIMARC >--- > C4/Biblio.pm | 20 ++++++++++++++++---- > .../prog/en/modules/catalogue/detail.tt | 22 +++++++++++++++++----- > .../opac-tmpl/bootstrap/en/modules/opac-detail.tt | 2 +- > 3 files changed, 34 insertions(+), 10 deletions(-) > >diff --git a/C4/Biblio.pm b/C4/Biblio.pm >index d90094e..8662666 100644 >--- a/C4/Biblio.pm >+++ b/C4/Biblio.pm >@@ -2074,10 +2074,19 @@ sub GetMarcSeries { > my @subfields = $field->subfields(); > my @link_loop; > >+ # if there is an authority link, build the links with an= subfield9 >+ my $subfield9 = $field->subfield('9'); >+ if ($subfield9) { >+ my $linkvalue = $subfield9; >+ $linkvalue =~ s/(\(|\))//g; >+ @link_loop = ( { limit => 'an', 'link' => $linkvalue } ); >+ } >+ > for my $series_subfield (@subfields) { > > # ignore $9, used for authority link > next if ( $series_subfield->[0] eq '9' ); >+ next if ( $series_subfield->[0] eq '3' && $marcflavour eq "UNIMARC" ); > > my $volume_number; > my $code = $series_subfield->[0]; >@@ -2090,10 +2099,13 @@ sub GetMarcSeries { > $volume_number = 1; > } > >- push @link_loop, { >- 'link' => $linkvalue, >- operator => (scalar @link_loop) ? ' and ' : undef >- }; >+ unless ($subfield9) { >+ push @link_loop, { >+ limit => 'se', >+ 'link' => $linkvalue, >+ operator => (scalar @link_loop) ? ' and ' : undef >+ }; >+ } > > if ($volume_number) { > push @subfields_loop, { volumenum => $value }; >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt >index 41730f5..fb29baa 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt >@@ -415,11 +415,23 @@ function verify_images() { > </li> > [% END %] > [% IF ( MARCSERIES ) %] >- <li><strong>Series: </strong><ul>[% FOREACH MARCSERIE IN MARCSERIES %] >- <li>[% FOREACH MARCSERIES_SUBFIELDS_LOO IN MARCSERIE.MARCSERIES_SUBFIELDS_LOOP %] [% IF ( MARCSERIES_SUBFIELDS_LOO.value ) %]<a href="/cgi-bin/koha/catalogue/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 %][% END %]</li> >- [% END %] >- </ul> >- </li> >+ <li> >+ <strong>Series: </strong> >+ <ul> >+ [% FOREACH serie IN MARCSERIES %] >+ <li> >+ [% FOREACH subfield IN serie.MARCSERIES_SUBFIELDS_LOOP %] >+ [% IF ( subfield.value ) %] >+ <a href="/cgi-bin/koha/catalogue/search.pl?q=[% FOREACH link IN subfield.link_loop %][% link.operator |url %][% link.limit %]:[% link.link |url %][% END %]">[% subfield.value %]</a> >+ [% END %] >+ [% IF ( subfield.volumenum ) %] >+ . [% subfield.volumenum %] >+ [% END %] >+ [% END %] >+ </li> >+ [% END %] >+ </ul> >+ </li> > [% END %] > [% IF ( publishercode ) %] > <li><strong>Published by:</strong> >diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt >index 6799e23..9bcb66d 100644 >--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt >+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt >@@ -199,7 +199,7 @@ > [% 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> >+ <a href="/cgi-bin/koha/opac-search.pl?q=[% FOREACH link IN MARCSERIES_SUBFIELDS_LOO.link_loop %][% link.operator |url %][% link.limit %]:[% link.link |url %][% END %]">[% MARCSERIES_SUBFIELDS_LOO.value %]</a> > [% END %] > [% IF ( MARCSERIES_SUBFIELDS_LOO.volumenum ) %] > . [% MARCSERIES_SUBFIELDS_LOO.volumenum %] >-- >1.9.1
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 14770
: 42220