https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43028

--- Comment #5 from Martin Renvoize (ashimema) 
<[email protected]> ---
Created attachment 201731
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201731&action=edit
Bug 43028: Isolate one undecodable record from aborting a batch import or
revert

BatchCommitRecords already ran each record in its own transaction;
extend that to catch Koha::Exceptions::Metadata::Invalid (bug 35104)
per-record instead of letting it propagate, flagging the record with
an 'error' status and import_error message and counting it via the new
$num_errored return value, rather than aborting records after it in
the same batch. _batchCommitItems gets the same per-item treatment for
undecodable item MARCXML.

BatchRevertRecords runs as a single whole-batch transaction, so the
same care is needed in its 'restore' branch too: both the marcxml_old
decode step and the ModBiblio/ModAuthority call that follows it are
isolated, so one record that can't be restored doesn't roll back every
other record already reverted in the same batch.

Signed-off-by: Baptiste Bayche <[email protected]>
Signed-off-by: Baptiste Wojtkowski <[email protected]>
Sponsored-by: OpenFifth

-- 
You are receiving this mail because:
You are the assignee for the bug.
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/

Reply via email to