[Bug 8874] New: export_borrowers.pl : add a separator parameter
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8874 Priority: P5 - low Change sponsored?: --- Bug ID: 8874 Assignee: adrien.saurat@biblibre.com Summary: export_borrowers.pl : add a separator parameter Severity: enhancement Classification: Unclassified OS: All Reporter: adrien.saurat@biblibre.com Hardware: All Status: ASSIGNED Version: master Component: Command-line Utilities Product: Koha The patron export is now creating CSV files with a comma separator. It's sometimes useful to be able to use another character as a separator. Adding a --separator for this would help. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8874 --- Comment #1 from Adrien SAURAT <adrien.saurat@biblibre.com> --- Created attachment 12676 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=12676&action=edit proposed patch Adds a --separator parameter. Example of use : ./export_borrowers.pl --separator=';' -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8874 Adrien SAURAT <adrien.saurat@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8874 Frédéric Demians <frederic@tamil.fr> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |frederic@tamil.fr --- Comment #2 from Frédéric Demians <frederic@tamil.fr> --- It doesn't work if you don't specify -separator: no comma as default separator. You should have: my $separator = ','; and maybe add a test to guaranty that the separator is 1 character length. I would also add 'always_quote' parameter to Text::CSV constructor. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8874 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |veron@veron.ch --- Comment #3 from Marc Véron <veron@veron.ch> --- 1) If I indicate just one character, this character is taken as separator (as expected). If I do not indicate a separator, it takes comma (as expected). 2) If I do something wrong like: export_borrowers.pl -separator=':;;' ...it silently takes the first character (:) In this case I would expect to get the usage screen. 3) Maybe I just do not know: How can I make use of a tab character as separator? With '\t' I get \ as separator. 4) It would be nice to have the 'always_quote' parameter from Comment #2 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8874 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA CC| |kyle@bywatersolutions.com Patch complexity|--- |Small patch --- Comment #4 from Kyle M Hall <kyle@bywatersolutions.com> --- Fails QA based on Marc Véron's comments. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8874 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |http://bugs.koha-community. | |org/bugzilla3/show_bug.cgi? | |id=9235 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8874 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also|http://bugs.koha-community. | |org/bugzilla3/show_bug.cgi? | |id=9235 | -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8874 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |http://bugs.koha-community. | |org/bugzilla3/show_bug.cgi? | |id=9045 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8874 --- Comment #5 from Kyle M Hall <kyle@bywatersolutions.com> --- This functionality is also included in Bug 9045 - Adding some parameters to export_borrowers.pl -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8874 --- Comment #6 from Kyle M Hall <kyle@bywatersolutions.com> --- 3) Maybe I just do not know: How can I make use of a tab character as separator? With '\t' I get \ as separator. The answer would be $'\t' -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8874 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |RESOLVED Resolution|--- |DUPLICATE --- Comment #7 from Marc Véron <veron@veron.ch> --- Marked as duplicate based on comment #5 *** This bug has been marked as a duplicate of bug 9045 *** -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org