Skip Menu |

This queue is for tickets about the Text-CSV CPAN distribution.

Report information
The Basics
Id: 82590
Status: resolved
Priority: 0/
Queue: Text-CSV

People
Owner: Nobody in particular
Requestors: barry [...] allegro.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: documentation deficiency for Text::CSV
Date: Wed, 9 Jan 2013 11:26:13 -0800
To: bug-Text-CSV [...] rt.cpan.org
From: Barry Lake <barry [...] allegro.com>
Dear CPAN and Text::CSV maintainers, Thanks very much for an excellent perl module, Text::CSV. As far as I can see, the module is working fine. I'm not writing to report a bug. But I would like to request that you improve the documentation for the is_quoted() method. It is not obvious from reading the documentation that keep_meta_info must be set to 1 in order for is_quoted() to work as expected. The stackoverflow.com web site has a very detailed description of the problem I encountered as a result of this deficiency in the documentation: http://stackoverflow.com/questions/14205022/is-quoted-method-in-textcsv-perl-module (sorry if that wraps!) Best regards, -- Barry Lake mailto://barry@allegro.com Allegro Consultants, Inc. http://www.allegro.com (408)252-2330
Thanks for the report I have amended the documentation of Text::CSV_XS to note the requirement of setting keep_meta_info to a true value in order to have is_quoted () and is_binary () return meaningful values. Those changes may eventually be copied to the documentation of Text::CSV
Applied this patch in https://github.com/makamaka/Text-CSV/pull/22 Hope it helps!
On Sun May 31 00:51:13 2015, GARU wrote: Show quoted text
> Applied this patch in https://github.com/makamaka/Text-CSV/pull/22 > > Hope it helps!
Thanks. Applied.
On Sun Aug 28 01:08:37 2016, ISHIGAKI wrote: Show quoted text
> On Sun May 31 00:51:13 2015, GARU wrote:
> > Applied this patch in https://github.com/makamaka/Text-CSV/pull/22 > > > > Hope it helps!
> > Thanks. Applied.
Closed as 1.91 is out. Thanks.