Bug 39215

Summary: EDIFACT quote files should skip orders that cannot be processed rather than failing to complete
Product: Koha Reporter: Martin Renvoize (ashimema) <martin.renvoize>
Component: AcquisitionsAssignee: Martin Renvoize (ashimema) <martin.renvoize>
Status: ASSIGNED --- QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low    
Version: unspecified   
Hardware: All   
OS: All   
GIT URL: Change sponsored?: ---
Patch complexity: --- Documentation contact:
Documentation submission: Text to go in the release notes:
Version(s) released in:
Circulation function:
Bug Depends on: 38689    
Bug Blocks:    

Description Martin Renvoize (ashimema) 2025-02-28 11:21:44 UTC
When loading an EDIFACT quote file, if the process finds problematic data (usually a mismatched LLO - Branchcode or other enumerated coded field) the load process gets stuck and displays as 'Processing' in the EDFACT messages page. This usually means that the remainder of the quotes in the file have to be manually corrected and is it's currently difficult to debug by library staff.

It would be preferable if the process could skip, and report any problem data but complete all other quotes.
Comment 1 Martin Renvoize (ashimema) 2025-02-28 11:44:40 UTC
A first step should be to wrap 'Koha::Item->new($item_hash)->store;' in try/catch blocks and report foreign key failures before moving to the next item creation.