Bug #23289 for Class-BuildMethods: C::BM does not accept false defaults

Tue Nov 14 21:33:30 2006 collinw [...] gmail.com - Ticket created

Subject:

C::BM does not accept false defaults

As of 0.12, Class::BuildMethods does not allow default values to be false. Attempting to use 0 or "" as a default value results in a default value of undef. This is caused by simple truth tests on $default_for{$method}, rather an exists-based tests. This can be fixed by using exists on lines 164 and 192 of lib/Class/BuildMethods.pm Thanks, Collin Winter

Wed Nov 15 15:41:02 2006 ovid [...] cpan.org - Correspondence added

From:

OVID [...] cpan.org

On Tue Nov 14 21:33:30 2006, collinw wrote: Show quoted text

> As of 0.12, Class::BuildMethods does not allow default values to be > false. Attempting to use 0 or "" as a default value results in a default > value of undef. This is caused by simple truth tests on > $default_for{$method}, rather an exists-based tests. > > This can be fixed by using exists on lines 164 and 192 of > lib/Class/BuildMethods.pm

Hi Collin, Thanks for the bug report. I have fixed this bug and uploaded it as Class::BuildMethods 0.20. Hopefully it should be available soon. Also, this version now adds class data support: package Universe; use Class::BuildMethods pi => { class_data => 1, default => 3.1415927, }; Cheers, Ovidd

Wed Nov 15 15:41:03 2006 The RT System itself - Status changed from 'new' to 'open'

Wed Nov 15 15:41:45 2006 ovid [...] cpan.org - Correspondence added

Fixed in 0.20.

Wed Nov 15 15:41:47 2006 ovid [...] cpan.org - Status changed from 'open' to 'resolved'