[Dbix-class] Call for testing

Matija Grabnar matija at literal.si
Fri Aug 18 12:42:36 CEST 2006


Brandon Black wrote:
> On 7/16/06, Matija Grabnar <matija at literal.si> wrote:
>   
>> Matt S Trout wrote:
>>     
>>> This is the only functional change since 06999_07 so provided that's ok I
>>> reckon we're ready to ship 07 (you *did* all test 06999_07, right?
>>>       
>> I tested it with an application I wrote which has a fair number of joins
>> and some prefetches, but none really tricky. It seems to work OK.
>>
>> However, when I tried DBIx::Class:Schema::Loader 0.3004, I noticed a
>> bug. Given a field like
>>
>>   `bulk_euro_notax` decimal(11,2) NOT NULL,
>>
>> schema::loader generates
>>
>>   "bulk_euro_notax",
>>   { data_type => "DECIMAL", default_value => "", is_nullable => 0, size
>> => 11 },
>>
>> In other words, it loses the ",2" in the size of the decimal. That
>> decimal is very important when working with monetary values.
>>
>>     
>
> I'll take a look at this in the next few days if nobody else does.
> The Loader gets it from Storage::DBI's columns_info_for, which in turn
> gets it from DBI, the issue could be in either of those, or possibly a
> DBD deficiency.
>
> -- Brandon
>   
Um, has anything been done about this yet?



More information about the Dbix-class mailing list