[Dbix-class] mk_classaccessor now longer available in 08002

[Matt S Trout]

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/