Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the DateTime-TimeZone CPAN distribution.

Report information
The Basics
Id: 13676
Status: resolved
Priority: 0/
Queue: DateTime-TimeZone

People
Owner: Nobody in particular
Requestors: pause [...] herlpacker.co.uk
Cc:
AdminCc:

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

Attachments


Subject: names_in_category() doesn't return a list of names
Module: DateTime::TimeZone 0.36 I think that the names_in_category() should use $_[1] instead of $_[0]. Unless I'm not using the method correctly, $_[0] will the the object, not the category name. perl -MData::Dumper -MDateTime::TimeZone -e 'print Dumper [DateTime::TimeZone->names_in_category( "Europe" )];' I'm surprised that there are no tests for this module/function.
Download TimeZoneCatalog-patch
application/octet-stream 598b

Message body not shown because it is not plain text.

[guest - Wed Jul 13 10:53:25 2005]: Show quoted text
> Module: DateTime::TimeZone 0.36 > > I think that the names_in_category() should use $_[1] instead of > $_[0]. > Unless I'm not using the method correctly, $_[0] will the the object, > not the category name.
It's not designed to be used as a method, it's a function. I'll change it in the next release so that it's more flexible. Show quoted text
> > perl -MData::Dumper -MDateTime::TimeZone -e 'print Dumper > [DateTime::TimeZone->names_in_category( "Europe" )];' > > I'm surprised that there are no tests for this module/function.
Well, I'm not ;) Anyway, you're not using it as documented, but adding tests is good.