@@ -, +, @@
implement its advertised API
- Run:
prove t/db_dependent/Biblio.t
- catalogue/detail.pl should correctly show ISBNs.
- opac/opac-detail.pl should correctly show ISBNs in both prog and bootstrap.
- opac-opac-sendshelf.pl should correctly show ISBNs in the email.
- Sign off :-D
---
C4/Biblio.pm | 5 ++---
koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt | 2 +-
koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt | 2 +-
koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-sendshelf.tt | 2 +-
koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt | 2 +-
koha-tmpl/opac-tmpl/prog/en/modules/opac-sendshelf.tt | 2 +-
t/db_dependent/Biblio.t | 1 -
7 files changed, 7 insertions(+), 9 deletions(-)
--- a/C4/Biblio.pm
+++ a/C4/Biblio.pm
@@ -1689,13 +1689,12 @@ sub GetMarcISBN {
} else { # assume marc21 if not unimarc
$scope = '020';
}
+
my @marcisbns;
- my $marcisbn;
foreach my $field ( $record->field($scope) ) {
my $isbn = $field->as_string();
if ( $isbn ne "" ) {
- $marcisbn = { marcisbn => $isbn, };
- push @marcisbns, $marcisbn;
+ push @marcisbns, $isbn;
}
}
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt
+++ a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt
@@ -416,7 +416,7 @@ function verify_images() {
[% IF ( MARCISBNS ) %]
- - ISBN:
[% FOREACH MARCISBN IN MARCISBNS %]- [% MARCISBN.marcisbn %]
[% END %]
+ - ISBN:
[% FOREACH MARCISBN IN MARCISBNS %]- [% MARCISBN %]
[% END %]
[% ELSE %]
[% IF ( normalized_isbn ) %]
- ISBN: [% normalized_isbn %]
--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt
+++ a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt
@@ -197,7 +197,7 @@
[% END %]
[% IF ( MARCISBNS ) %]
- ISBN:[% FOREACH MARCISBN IN MARCISBNS %][% MARCISBN.marcisbn %][% IF ( loop.last ) %].[% ELSE %]; [% END %][% END %]
+ ISBN:[% FOREACH MARCISBN IN MARCISBNS %][% MARCISBN %][% IF ( loop.last ) %].[% ELSE %]; [% END %][% END %]
[% ELSE %]
[% IF ( normalized_isbn ) %]
ISBN: [% normalized_isbn %]
--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-sendshelf.tt
+++ a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-sendshelf.tt
@@ -59,7 +59,7 @@ Your list : [% shelfname %]
[% IF ( BIBLIO_RESULT.ISBN && BIBLIO_RESULT.size > 0 ) %]
ISBN: [% FOREACH isbn IN BIBLIO_RESULT.ISBN %]
- [% isbn.marcisbn %]
+ [% isbn %]
[% UNLESS ( loop.last ) %]; [% END %]
[% END %]
--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt
+++ a/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt
@@ -646,7 +646,7 @@ YAHOO.util.Event.onContentReady("furtherm", function () {
[% IF ( pages ) %]Physical details: [% pages %] [% illus %] [% size %][% END %]
[% IF ( MARCISBNS ) %]
- ISBN:[% FOREACH MARCISBN IN MARCISBNS %][% MARCISBN.marcisbn %][% IF ( loop.last ) %].[% ELSE %]; [% END %][% END %]
+ ISBN:[% FOREACH MARCISBN IN MARCISBNS %][% MARCISBN %][% IF ( loop.last ) %].[% ELSE %]; [% END %][% END %]
[% ELSE %]
[% IF ( normalized_isbn ) %]
ISBN: [% normalized_isbn %]
--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-sendshelf.tt
+++ a/koha-tmpl/opac-tmpl/prog/en/modules/opac-sendshelf.tt
@@ -59,7 +59,7 @@ Your list : [% shelfname %]
[% IF ( BIBLIO_RESULT.ISBN && BIBLIO_RESULT.size > 0 ) %]
ISBN: [% FOREACH isbn IN BIBLIO_RESULT.ISBN %]
- [% isbn.marcisbn %]
+ [% isbn %]
[% UNLESS ( loop.last ) %]; [% END %]
[% END %]
--- a/t/db_dependent/Biblio.t
+++ a/t/db_dependent/Biblio.t
@@ -19,7 +19,6 @@ use Modern::Perl;
use Test::More tests => 3;
use Test::MockModule;
-use Data::Dumper;
use MARC::Record;
--