From 09bf62e3d369cd95b3420b87429f4c2dad3162e0 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Tue, 12 Nov 2019 18:18:23 +0100 Subject: [PATCH] Bug 23846: Handle exception gracefully at the OPAC I do not think we should have the same trick as the intranet, and display a message. This should be enough. --- koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt | 2 +- opac/opac-detail.pl | 4 +++- 2 files changed, 4 insertions(+), 2 deletions(-) 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 5da7b95c18..571f3b2395 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt @@ -71,7 +71,7 @@ [% IF decoding_error %]
There is an error with this bibliographic record, the view may be degraded. -
Error: [% decoding_error %] +
Error: [% decoding_error | html %]
[% END %] [% IF ( ocoins ) %] diff --git a/opac/opac-detail.pl b/opac/opac-detail.pl index 5e29b3f627..07655b2a4e 100755 --- a/opac/opac-detail.pl +++ b/opac/opac-detail.pl @@ -845,8 +845,10 @@ $template->param( content_identifier_exists => $content_identifier_exists, ); +# Catch the exception as Koha::Biblio::Metadata->record can explode if the MARCXML is invalid # COinS format FIXME: for books Only -$template->param( ocoins => $biblio->get_coins ); +my $coins = eval { $biblio->get_coins }; +$template->param( ocoins => $coins ); my ( $loggedincommenter, $reviews ); if ( C4::Context->preference('reviewson') ) { -- 2.11.0