Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

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

Report information
The Basics
Id: 32223
Status: resolved
Priority: 0/
Queue: Parse-CSV
People
Owner: Nobody in particular
Requestors: pwomack [...] papermule.co.uk
Cc:
AdminCc:
Bug Information
Severity: Important
Broken in: 1.00
Fixed in: (no value)

History Show all quoted text
  Thu Jan 10 06:43:06 2008 pwomack [...] papermule.co.uk - Ticket created
I think the "binary" flag (passed on to the underlying Text::CSV_XS module) simply does not work. The documentation for Text::CSV_XS points out that you pretty much HAVE to use the getline() function in Text::CSV_XS when you;'re accepting binary fields, especially quoted binary fields with linefeeds in them. Despite this, Parse::CSV uses it's own _getline to get a line, before using parse() from Text::CSV_XS to parse it. This fails in the circumstances outlined above. BugBear
  Wed Mar 26 17:36:32 2008 MARKSTOS [...] cpan.org - Correspondence added
Subject: dupe bug report
On Thu Jan 10 06:43:06 2008, bugbear wrote: Show quoted text
> I think the "binary" flag (passed on to the underlying Text::CSV_XS > module) simply does not work. The documentation for Text::CSV_XS points > out that you pretty much HAVE to use the getline() function in > Text::CSV_XS when you;'re accepting binary fields, especially quoted > binary fields with linefeeds in them. > > Despite this, Parse::CSV uses it's own _getline to get a line, before > using parse() from Text::CSV_XS to parse it. > > This fails in the circumstances outlined above.
BugBear, This bug is a dupe of another bug already filed: #27734 If you have a patch for it, post it here. Mark
  Wed Mar 26 17:36:34 2008 The RT System itself - Status changed from 'new' to 'open'
  Thu Mar 27 05:23:39 2008 pwomack [...] papermule.co.uk - Correspondence added
Subject: Re: [rt.cpan.org #32223] dupe bug report
Date: Thu, 27 Mar 2008 09:22:58 +0000
To: bug-Parse-CSV [...] rt.cpan.org
From: paul womack <pwomack [...] papermule.co.uk>
MARKSTOS via RT wrote: Show quoted text
> <URL: http://rt.cpan.org/Ticket/Display.html?id=32223 > > > On Thu Jan 10 06:43:06 2008, bugbear wrote:
>> I think the "binary" flag (passed on to the underlying Text::CSV_XS >> module) simply does not work. The documentation for Text::CSV_XS points >> out that you pretty much HAVE to use the getline() function in >> Text::CSV_XS when you;'re accepting binary fields, especially quoted >> binary fields with linefeeds in them. >> >> Despite this, Parse::CSV uses it's own _getline to get a line, before >> using parse() from Text::CSV_XS to parse it. >> >> This fails in the circumstances outlined above.
> > BugBear, > > This bug is a dupe of another bug already filed: > > #27734 > > If you have a patch for it, post it here.
I'm afraid I simply bypassed PARSE::CSV and use Text::CSV_XS directly to get my project working. BugBear
  Mon Nov 28 21:04:48 2011 adamk [...] cpan.org - Correspondence added
Closing as a duplicate of the other binary bug report
  Mon Nov 28 21:04:48 2011 adamk [...] cpan.org - Status changed from 'open' to 'resolved'