On Fri, Jun 22, 2007 at 08:18:50AM -0400, Christopher Heschong wrote:
> On Jun 21, 2007, at 1:52 PM, Matt S Trout wrote:
>
> >On Thu, Jun 21, 2007 at 01:16:43PM +1000, brett gardner wrote:
> >>You can no longer do
> >>
> >>__PACKAGE__->mk_classaccessor('foo')
> >>
> >>in the current version of DBIx::Class (08002).
> >>
> >>Is this be design? I have used this feature quite extensively
> >>throughout
> >>my applications.
> >
> >Ah, we replaced Class::Data::Inheritable with Class::Data::Accessor
> >a while
> >back, and made mk_classdata proxy to mk_classaccessor.
> >
> >Then when we moved to Class::Accessor::Grouped the mk_classdata
> >proxy was
> >moved to use that rather than mk_classaccessor in turn becoming a
> >proxy.
> >
> >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 ...
--
Matt S Trout Need help with your Catalyst or DBIx::Class project?
Technical Director Want a managed development or deployment platform?
Shadowcat Systems Ltd. Contact mst (at) shadowcatsystems.co.uk for a quote
http://chainsawblues.vox.com/ http://www.shadowcatsystems.co.uk/
|
