Bug 15493

Summary: Export records using a CSV profile does not work
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: ToolsAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: major    
Priority: P5 - low CC: brendan, hector.hecaxmmx, julian.maurice
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:
Circulation function:
Bug Depends on: 14722    
Bug Blocks: 15498    
Attachments: Bug 15493: Use the correct CSV profile when exporting records from the export tool
[SIGNED-OFF]Bug 15493: Use the correct CSV profile when exporting records from the export tool
[PASSED QA] Bug 15493: Use the correct CSV profile when exporting records from the export tool

Description Jonathan Druart 2016-01-06 16:18:57 UTC
The CSV profile used is not the one the user has chosen, but the one defined in the pref ExportWithCsvProfile
Comment 1 Jonathan Druart 2016-01-06 16:22:41 UTC Comment hidden (obsolete)
Comment 2 Jonathan Druart 2016-01-06 16:22:58 UTC
Affect 3.22
Comment 3 Héctor Eduardo Castro Avalos 2016-01-06 21:33:12 UTC Comment hidden (obsolete)
Comment 4 Katrin Fischer 2016-01-06 22:24:24 UTC
Created attachment 46372 [details] [review]
[PASSED QA] Bug 15493: Use the correct CSV profile when exporting records from the export tool

The CSV profile used is not the one the user has chosen, but the one
defined in the pref ExportWithCsvProfile.

It's caused by a wrong variable name csv_profile vs csv_profile_id

Test plan:
Define 1+ csv profiles
Set ExportWithCsvProfile with 1 of these profile
Use the export tool to export some records using a csv profile.
Without this patch, the csv profile used, will always be the one defined
in ExportWithCsvProfile.

Signed-off-by: Hector Castro <hector.hecaxmmx@gmail.com>
Works as descrived

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 5 Brendan Gallagher 2016-01-07 18:47:32 UTC
Pushed to Master - Thanks.
Comment 6 Julian Maurice 2016-01-12 13:47:01 UTC
Patch pushed to 3.22.x, will be in 3.22.2