https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20551
--- Comment #65 from David Gustafsson <[email protected]> --- (In reply to Marcel de Rooy from comment #57) > + my $marc_flavour = C4::Context->preference('marcflavour'); > + my $biblio_metadata = $resultset->find({ > + 'biblionumber' => $biblionumber, > + 'format' => 'marcxml', > + 'marcflavour' => $marc_flavour > + }); > + my $marc_xml = $biblio_metadata->metadata; > + $marc_xml = StripNonXmlChars($marc_xml); > + > + my $record = eval { > + MARC::Record::new_from_xml($marc_xml, 'UTF-8', $marc_flavour) > + }; > + if (!$record) { > > You test $record, but you should have tested $biblio_metadata too ? > Otherwise you already crash on ->metadata.. Hi, thanks for the feedback. There is no definite right or wrong, but personally try to avoid defensive programming where things are very unlikely to fail. Ids should be valid since have been previously fetched from database, in case of the XML-decoding it's highly unlikely to go wrong, but I can imagine cases where it could. -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
