Matt S Trout wrote:
> On Fri, Jun 22, 2007 at 03:33:04PM -0400, Christopher Heschong wrote:
>> <snip>
>>
>>>>> Try adding
>>>>>
>>>>> sub mk_classaccessor { shift->mk_classdata(@_); }
>>>>>
>>>>> as a workaround and we'll fix it for 08003.
>>>> I ran into this myself and just added Class::Data::Accessor as a base
>>>> class in my code, since that is where the method really comes from.
>>> Please next time report the bug :)
>>>
>>>> Plus if you haven't upgraded your production systems to 08002 yet it
>>>> should be compatible on either version.
>>> Yes, although the Accessor::Grouped 'inherited type which we're now using
>>> for it is a better approach than Class::Data::Accessor and I'd recommend
>>> if you're going to change your code you change it to ::Grouped ...
>> I thought it was a feature!! But duly noted... still, is it a good idea
>> to rely on methods not provided (or even documented for that matter) by
>> the package?
>
> No, it isn't, but when it doesn't cost us anything to do so we don't tend
> to break compat even with incidental features like that one - and I consider
> "broke compatiblity without gaining anything" to count as a bug :)
>
> C::A::G's 'inherited' type is definitely a better implementation, though
> - and respects the package MRO type automatically which is a real boon.
> On the down side, I programmed it. YMMV. :-P -=Chris
signature.asc
Description: OpenPGP digital signature
_______________________________________________ List: http://lists.rawmode.org/cgi-bin/mailman/listinfo/dbix-class Wiki: http://dbix-class.shadowcatsystems.co.uk/ IRC: irc.perl.org#dbix-class SVN: http://dev.catalyst.perl.org/repos/bast/trunk/DBIx-Class/ Searchable Archive: http://www.mail-archive.com/[email protected]/
