Skip Menu |

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

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

People
Owner: Nobody in particular
Requestors: RSAVAGE [...] cpan.org
Cc:
AdminCc:

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



Subject: Docs unclear about values for status()
Hi If I call $csv -> status(), does a true value or a false value mean success? The docs don't say. I suggest the docs for status() should read (if true means success): This object function returns true for success and false for failure, for combine () or parse (), whichever was called more recently. Or, if true means failure: This object function returns false for success and true for failure, for combine () or parse (), whichever was called more recently. Cheers Ron
from what I understood of the docs, true means success. So I have applied your patch in https://github.com/makamaka/Text-CSV/pull/20 Hope to see this merged and released soon \o/
Subject: Re: [rt.cpan.org #78580] Docs unclear about values for status()
Date: Sun, 31 May 2015 10:04:17 +1000
To: bug-Text-CSV [...] rt.cpan.org
From: Ron Savage <ron [...] savage.net.au>
Hi Breno Good news at last! $many x $thanx; On 31/05/15 01:31, Breno G. de Oliveira via RT wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=78580 > > > from what I understood of the docs, true means success. So I have applied your patch in https://github.com/makamaka/Text-CSV/pull/20 > > Hope to see this merged and released soon \o/ >
-- Ron Savage - savage.net.au
On Sun May 31 00:31:11 2015, GARU wrote: Show quoted text
> from what I understood of the docs, true means success. So I have > applied your patch in https://github.com/makamaka/Text-CSV/pull/20 > > Hope to see this merged and released soon \o/
Thanks. Applied.
On Sun Aug 28 01:21:48 2016, ISHIGAKI wrote: Show quoted text
> On Sun May 31 00:31:11 2015, GARU wrote:
> > from what I understood of the docs, true means success. So I have > > applied your patch in https://github.com/makamaka/Text-CSV/pull/20 > > > > Hope to see this merged and released soon \o/
> > > Thanks. Applied.
Closed as 1.91 is out. Thanks.