[Dbix-class] mk_classaccessor now longer available in 08002

Christopher H. Laco claco at chrislaco.com
Fri Jun 22 21:45:19 GMT 2007


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 usi=
ng
>>> 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 consi=
der
> "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

-=3DChris

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 187 bytes
Desc: OpenPGP digital signature
Url : http://lists.scsys.co.uk/pipermail/dbix-class/attachments/20070622/09=
d8dc00/signature.pgp


More information about the Dbix-class mailing list