[Bug 41629] New: Add Koha::CSV::Inventory for inventory exports
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41629 Bug ID: 41629 Summary: Add Koha::CSV::Inventory for inventory exports Initiative type: --- Sponsorship --- status: Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: koha-bugs@lists.koha-community.org Reporter: tomascohen@gmail.com QA Contact: testopia@bugs.koha-community.org Depends on: 41619 Blocks: 41621 This bug adds Koha::CSV::Inventory to standardize CSV generation for inventory exports. Files to update: - tools/inventory.pl Benefits: - Consistent CSV formatting - Proper quote handling - Testable code Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41619 [Bug 41619] Add `Koha::CSV` https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41621 [Bug 41621] Use Koha::CSV everywhere [UMBRELLA] -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41629 Bug 41629 depends on bug 41619, which changed state. Bug 41619 Summary: Add `Koha::CSV` https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41619 What |Removed |Added ---------------------------------------------------------------------------- Status|Needs documenting |RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41629 Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff Patch complexity|--- |Trivial patch -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41629 --- Comment #1 from Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> --- Created attachment 203491 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=203491&action=edit Bug 41629: Use Koha::CSV for inventory exports This patch replaces the eval/Text::CSV usage with Koha::CSV in tools/inventory.pl using the low-level API. Benefits: - formula => empty enforced (prevents formula injection) - binary => 1 always set (UTF-8 safe) - CSVDelimiter system preference used - Removes runtime eval and direct Text::CSV dependency - Removes explicit \n handling (eol handled by Koha::CSV) Test plan: 1. Apply patches (depends on bug 41620) 2. Go to Tools > Inventory 3. Run an inventory and export as CSV => SUCCESS: CSV generated with correct columns and data 4. Sign off :-D -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41629 Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Add Koha::CSV::Inventory |Use Koha::CSV for inventory |for inventory exports |exports CC| |tomascohen@gmail.com -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org