Skip Menu |

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

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

People
Owner: Nobody in particular
Requestors: Philip [...] kime.org.uk
Cc:
AdminCc:

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



Subject: is_quoted always returns false
Date: Wed, 7 Aug 2013 10:20:51 +0200
To: bug-Text-CSV [...] rt.cpan.org
From: Philip Kime <philkime [...] kime.org.uk>
The is_quoted() method seems to be broken (XS and PP) ... #!/usr/local/bin/perl use Text::CSV; my $csv = Text::CSV->new({sep_char => ' '}); $csv->parse('"a"'); print $csv->is_quoted(0) ? 'true' : 'false' . "\n"; -- Dr Philip Kime
Thanks for the report. It is not broken. The docs can be clearer on the fact the is_quoted () and is_binary only return usable values when passing the keep_meta_info as a true value: use Text::CSV; my $csv = Text::CSV->new ({ sep_char => ' ', keep_meta_info => 1 }); $csv->parse (q{"a"}); say $csv->is_quoted (0) ? "true" : "false"; I have changed that in the docs for Text::CSV_XS, which eventually might end up in the docs for Text::CSV.
On Wed Oct 08 02:49:02 2014, HMBRAND wrote: Show quoted text
> Thanks for the report. > > It is not broken.
(snip) Show quoted text
> I have changed that in the docs for Text::CSV_XS, which eventually > might end up in the docs for Text::CSV.
A pull request with this doc update is available in https://github.com/makamaka/Text-CSV/pull/22, and referenced in https://rt.cpan.org/Ticket/Display.html?id=82590 Hope it helps!
On Sun May 31 02:08:28 2015, GARU wrote: Show quoted text
> On Wed Oct 08 02:49:02 2014, HMBRAND wrote:
> > Thanks for the report. > > > > It is not broken.
> (snip)
> > I have changed that in the docs for Text::CSV_XS, which eventually > > might end up in the docs for Text::CSV.
> > A pull request with this doc update is available in > https://github.com/makamaka/Text-CSV/pull/22, and referenced in > https://rt.cpan.org/Ticket/Display.html?id=82590 > > Hope it helps!
Thanks. Applied.
On Sun Aug 28 01:15:50 2016, ISHIGAKI wrote: Show quoted text
> On Sun May 31 02:08:28 2015, GARU wrote:
> > On Wed Oct 08 02:49:02 2014, HMBRAND wrote:
> > > Thanks for the report. > > > > > > It is not broken.
> > (snip)
> > > I have changed that in the docs for Text::CSV_XS, which eventually > > > might end up in the docs for Text::CSV.
> > > > A pull request with this doc update is available in > > https://github.com/makamaka/Text-CSV/pull/22, and referenced in > > https://rt.cpan.org/Ticket/Display.html?id=82590 > > > > Hope it helps!
> > > Thanks. Applied.
Closed as 1.91 is out. Thanks.