[jifty-devel] Re: Improve CRUD->object_type

[Andrew Sterling Hanenkamp]

Done. See r4080. I also added a stub for adding CRUD tests to t/TestApp

On 8/19/07, Jesse Vincent <jesse at bestpractical.com> wrote:
>
> sounds reasonable.
> On Aug 19, 2007, at 1:55 PM, Andrew Sterling Hanenkamp wrote:
>
> I don't have time to make another patch just now, but I wanted to say I
> did find a small problem with the previous one. Rather than using
> $self->{record_class}, I've replaced that with a Class::Data::Inheritable
> field named _record_class since templates are called without a blessed
> invocant.
>
> On 8/19/07, Andrew Sterling Hanenkamp <sterling at hanenkamp.com> wrote:
> >
> > Here's the working patch.
> >
> > On 8/19/07, Andrew Sterling Hanenkamp < sterling at hanenkamp.com> wrote:
> > >
> > > I've attached a patch that should alter the way object_type is used.
> > > This is (should be, I haven't profiled to confirm) a very slight performance
> > > improvement and allow object_type() to be overridden to any class rather
> > > than just a class name prefixed with "MyApp::Module::".
> > >
> > > Cheers,
> > > Andrew
> > >
> > >
> >
> >
> _______________________________________________
> jifty-devel mailing list
> jifty-devel at lists.jifty.org
> http://lists.jifty.org/cgi-bin/mailman/listinfo/jifty-devel
>
>
>
> _______________________________________________
> jifty-devel mailing list
> jifty-devel at lists.jifty.org
> http://lists.jifty.org/cgi-bin/mailman/listinfo/jifty-devel
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.bestpractical.com/pipermail/jifty-devel/attachments/20070911/5696318b/attachment.htm