https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43020
--- Comment #1 from Martin Renvoize (ashimema) <[email protected]> --- Created attachment 201689 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201689&action=edit Bug 43020: Do not let one bad OAI-harvested record kill the whole harvest Koha::OAI::Client::Harvester::processRecord called MARC::Record->new_from_xml on harvested metadata with no error handling, and AddBiblio/ModBiblio/AddAuthority/ModAuthority with no handling for Koha::Exceptions::Metadata::Invalid either. Since init()'s harvest loop has no try/catch around processRecord, either failure died uncaught, aborting the whole harvest_oai.pl cron run partway through - no completion log entry, no report email, and every remaining record in that batch left unprocessed. processRecord now catches both cases the same way it already handles an XSLT transform failure a few lines above: log the record identifier and error, report status 'in_error' for that one record, and let the harvest continue with the rest. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. _______________________________________________ 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/
