Bug 41036

Summary: Koha::ImportBatch is not logging errors
Product: Koha Reporter: Lari Taskula <lari.taskula>
Component: Architecture, internals, and plumbingAssignee: Lari Taskula <lari.taskula>
Status: Signed Off --- QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low    
Version: unspecified   
Hardware: All   
OS: All   
GIT URL: Initiative type: ---
Sponsorship status: --- Comma delimited list of Sponsors:
Crowdfunding goal: 0 Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Bug 41036: Add tests
Bug 41036: Log errors in Koha::ImportBatch->new_from_file
Bug 41036: Add tests
Bug 41036: Log errors in Koha::ImportBatch->new_from_file

Description Lari Taskula 2025-10-16 17:15:25 UTC
It would be very useful to have Koha::ImportBatch->new_from_file log errors.
Comment 1 Lari Taskula 2025-10-16 17:19:44 UTC
Created attachment 188020 [details] [review]
Bug 41036: Add tests

To test:
1. Apply only this patch, do not yet apply the next one
2. prove t/db_dependent/Koha/ImportBatch.t
3. Observe tests failing
Comment 2 Lari Taskula 2025-10-16 17:19:46 UTC
Created attachment 188021 [details] [review]
Bug 41036: Log errors in Koha::ImportBatch->new_from_file

To test:
1. prove t/db_dependent/Koha/ImportBatch.t
2. Observe success
Comment 3 David Nind 2025-11-28 23:08:07 UTC
Created attachment 190037 [details] [review]
Bug 41036: Add tests

To test:
1. Apply only this patch, do not yet apply the next one
2. prove t/db_dependent/Koha/ImportBatch.t
3. Observe tests failing

Signed-off-by: David Nind <david@davidnind.com>
Comment 4 David Nind 2025-11-28 23:08:10 UTC
Created attachment 190038 [details] [review]
Bug 41036: Log errors in Koha::ImportBatch->new_from_file

To test:
1. prove t/db_dependent/Koha/ImportBatch.t
2. Observe success

Signed-off-by: David Nind <david@davidnind.com>