Bug 13938

Summary: Staging MARC records for import fails
Product: Koha Reporter: Kyle M Hall <kyle>
Component: ToolsAssignee: Kyle M Hall <kyle>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: critical    
Priority: P5 - low CC: chris, cmurdock, gwilliams, tomascohen
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 8970    
Bug Blocks:    
Attachments: Bug 13938 - Staging MARC records for import fails
Signed Off patch for bug 13938
[SIGNED-OFF] Bug 13938 - Staging MARC records for import fails
[PASSED QA] Bug 13938 - Staging MARC records for import fails

Description Kyle M Hall 2015-04-01 14:09:30 UTC
It appears that bug 8970 has introduced a bug where a MARC file uploaded for staging fails to import *unless* a matching rule is selected.
Comment 1 Kyle M Hall 2015-04-01 14:12:26 UTC Comment hidden (obsolete)
Comment 2 Cindy Murdock Ames 2015-04-01 15:21:42 UTC Comment hidden (obsolete)
Comment 3 Cindy Murdock Ames 2015-04-01 15:22:37 UTC
Applied patch and was able to upload records without having a matching rule selected.
Comment 4 Kyle M Hall 2015-04-01 15:49:29 UTC
I should note that I'm not seeing this bug on all 3.18.04 servers, but have seen it on couple. I cannot ascertain the exact circumstances that trigger the bug.
Comment 5 Kyle M Hall 2015-04-01 15:51:55 UTC Comment hidden (obsolete)
Comment 6 Katrin Fischer 2015-04-05 20:15:14 UTC
Created attachment 37508 [details] [review]
[PASSED QA] Bug 13938 - Staging MARC records for import fails

It appears that bug 8970 has introduced a bug where a MARC file uploaded
for staging fails to import *unless* a matching rule is selected.

Test Plan:
1) Apply this patch
2) Stage and import a MARC record *without* choosing a matching rule
3) Note the staging works as expected

Signed-off-by: Cindy Ames <cmurdock@ccfls.org>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 7 Tomás Cohen Arazi 2015-04-08 13:58:36 UTC
Patch pushed to master.

Thanks Kyle!
Comment 8 Chris Cormack 2015-04-11 02:43:57 UTC
Pushed to 3.18.x will be in 3.18.6