https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41627
--- Comment #1 from Tomás Cohen Arazi (tcohen) <[email protected]> --- Created attachment 203489 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=203489&action=edit Bug 41627: Use Koha::CSV for acquisition CSV exports This patch replaces direct Text::CSV_XS usage with Koha::CSV in: - C4/Acquisition.pm (GetBasketAsCSV with CSV profile) - admin/aqplan.pl (budget planning export) Both use the low-level API (combine/string) since columns are dynamic (determined by CSV profiles or budget structure at runtime). The sep_char from the CSV profile is passed through to Koha::CSV, overriding the system preference as intended. Benefits: - formula => empty enforced (prevents formula injection) - binary => 1 always set (UTF-8 safe) - Consistent with other CSV exports - Removes direct Text::CSV_XS dependency Test plan: 1. Apply patches (depends on bug 41620) 2. Create a basket with orders 3. Export basket as CSV (with and without a CSV profile) => SUCCESS: CSV generated correctly in both cases 4. Go to Administration > Budget planning > Export as CSV => SUCCESS: CSV exported with correct delimiter 5. Sign off :-D -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. _______________________________________________ Koha-bugs mailing list -- [email protected] To unsubscribe send an email to [email protected] website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
