Bug 9063

Summary: When ordering from staged file '# Bibs' is empty
Product: Koha Reporter: Katrin Fischer <katrin.fischer>
Component: AcquisitionsAssignee: Bugs List <koha-bugs>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P5 - low CC: gmcharlt, jonathan.druart, katrin.fischer
Version: Main   
Hardware: All   
OS: All   
See Also: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7180
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 7180    
Bug Blocks:    
Attachments: screenshot
Bug 9063: When ordering from staged file '# Bibs' is empty
[SIGNED-OFF] Bug 9063: When ordering from staged file '# Bibs' is empty
[PASSED QA] Bug 9063: When ordering from staged file '# Bibs' is empty
Bug 9063: When ordering from staged file '# Bibs' is empty

Description Katrin Fischer 2012-11-12 21:33:22 UTC
Created attachment 13409 [details]
screenshot

When ordering from a staged file in acquisitions, the column '# Bibs' in the table is empty for my staged file with 17 records.
Comment 1 Katrin Fischer 2013-09-11 20:40:07 UTC
This is still an issue on current master.
Comment 2 Katrin Fischer 2013-09-11 21:06:58 UTC
I think this problem is caused by renaming the database column from num_biblios to num_records when authority importing/matching was introduced.
Comment 3 Jonathan Druart 2014-04-15 14:51:52 UTC Comment hidden (obsolete)
Comment 4 Jonathan Druart 2014-04-15 14:57:50 UTC
Patch created on top of bug 7180. Feel free to provide a patch based on master.
Comment 5 Owen Leonard 2014-04-17 16:26:16 UTC Comment hidden (obsolete)
Comment 6 Katrin Fischer 2014-04-20 15:42:23 UTC
Created attachment 27326 [details] [review]
[PASSED QA] Bug 9063: When ordering from staged file '# Bibs' is empty

Bug 2060 renames columns num_biblios with num_records in the
import_batches table.

The addorderiso2709 files had not been fixed.

Test plan:
Add an order from a staged file to a basket and verify the "# Bibs"
columns is correctly filled. Before the patch, the column was empty.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works as described, passes all tests and QA script.
Comment 7 Galen Charlton 2014-04-20 17:22:00 UTC
This patch does not apply.  Since the dependency on 7180 is only textual, if Jonathan wants to rebase this one and put it back to passed QA after doing so, that's fine with me.
Comment 8 Galen Charlton 2014-05-04 22:14:18 UTC
I happened to fix this bug already via one of my follow-ups for bug 7180.  Sorry for forgetting about this one.
Comment 9 Jonathan Druart 2014-05-06 07:40:24 UTC
Created attachment 27957 [details] [review]
Bug 9063: When ordering from staged file '# Bibs' is empty

Bug 2060 renames columns num_biblios with num_records in the
import_batches table.

The addorderiso2709 files had not been fixed.

Test plan:
Add an order from a staged file to a basket and verify the "# Bibs"
columns is correctly filled. Before the patch, the column was empty.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works as described, passes all tests and QA script.
Comment 10 Jonathan Druart 2014-05-06 07:40:58 UTC
(In reply to Galen Charlton from comment #8)
> I happened to fix this bug already via one of my follow-ups for bug 7180. 
> Sorry for forgetting about this one.

Maybe this patch is still useful since it changes the variable name in the tt file too.
Comment 11 Galen Charlton 2014-05-06 14:38:42 UTC
Pushed to master.  Thanks, Jonathan!