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.

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt

@@ -71,7 +71,7 @@
     [% IF decoding_error %]
         <div>
             There is an error with this bibliographic record, the view may be degraded.
-            <br/> Error: [% decoding_error %]
+            <br/> Error: [% decoding_error | html %]
         </div>
     [% END %]
     [% IF ( ocoins ) %]

misc/maintenance/search_for_data_inconsistencies.pl

@@ -154,4 +154,5 @@ Catch data inconsistencies in Koha database
     then items.itype must be set else biblioitems.itemtype must be set
   * Item types defined in items or biblioitems must be defined in the itemtypes table
 * Invalid MARCXML in bibliographic records
+
 =cut

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') ) {