Skip Menu |

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

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

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

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



Text::CSV 1.0 version method is inherited from the worker module, but the documentation says it is the module version. You could add C<sub version { $VERSION }> to Text/CSV.pm but the attached patch fixes the documentation instead. The patch also corrects the documentation of Text::CSV_PP, which refers to the worker module and this module is the worker module. Robin
Subject: Text-CSV-1.patch
diff -ur Text-1.0/CSV.pm Text/CSV.pm --- Text-1.0/CSV.pm 2007-11-28 05:25:47.000000000 +0000 +++ Text/CSV.pm 2008-03-06 18:27:12.000000000 +0000 @@ -382,12 +382,14 @@ =item version -(Class method) Returns the current module version. Not worker module version. -If you want the worker module version, you can use C<module> method. - - print Text::CSV->version; # This module version - print Text::CSV->module->version; # The version of the worker module - +(Class method) Returns the current worker module version. +If you want the module version, you can use the C<VERSION> method, +see L<UNIVERSAL>. + + print Text::CSV->VERSION; # This module version + print Text::CSV->version; # The version of the worker module + # same as Text::CSV->module->version + =item new(\%attr) (Class method) Returns a new instance of Text::CSV. The objects diff -ur Text-1.0/CSV_PP.pm Text/CSV_PP.pm --- Text-1.0/CSV_PP.pm 2007-11-07 07:10:59.000000000 +0000 +++ Text/CSV_PP.pm 2008-03-06 18:21:41.000000000 +0000 @@ -588,8 +588,7 @@ =item version () -(Class method) Returns the current module version. Not worker module version. -If you want the worker module version, you can use C<module> method. +(Class method) Returns the current module version. =item new(\%attr)
Thank you to your patch! I released the updated version on CPAN. Regards,