Done. See r4080. I also added a stub for adding CRUD tests to t/TestApp<br><br><div><span class="gmail_quote">On 8/19/07, <b class="gmail_sendername">Jesse Vincent</b> <<a href="mailto:jesse@bestpractical.com">jesse@bestpractical.com
</a>> wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div style="">
sounds reasonable.<div><br><div><div><span class="e" id="q_1148042dd78d7046_1"><div>On Aug 19, 2007, at 1:55 PM, Andrew Sterling Hanenkamp wrote:</div><br></span></div><blockquote type="cite"><div><span class="e" id="q_1148042dd78d7046_3">
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.
<br><br><div><span class="gmail_quote">On 8/19/07, <b class="gmail_sendername">Andrew Sterling Hanenkamp</b> <<a href="mailto:sterling@hanenkamp.com" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">
sterling@hanenkamp.com</a>> wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"> Here's the working patch.<br><br><div><span>
<span class="gmail_quote">On 8/19/07, <b class="gmail_sendername">Andrew Sterling Hanenkamp</b> <<a href="mailto:sterling@hanenkamp.com" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)"> sterling@hanenkamp.com
</a>> wrote:</span> </span><div><span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">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::".
<br><br>Cheers,<br><span>Andrew<br> </span><br clear="all"></blockquote></span></div></div><br> <br clear="all"></blockquote></div><br></span></div><div style="margin: 0px;">_______________________________________________
</div><div style="margin: 0px;">jifty-devel mailing list</div><div style="margin: 0px;"><a href="mailto:jifty-devel@lists.jifty.org" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">jifty-devel@lists.jifty.org
</a></div><div style="margin: 0px;"><a href="http://lists.jifty.org/cgi-bin/mailman/listinfo/jifty-devel" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">http://lists.jifty.org/cgi-bin/mailman/listinfo/jifty-devel
</a></div> </blockquote></div><br></div></div><br>_______________________________________________<br>jifty-devel mailing list<br><a onclick="return top.js.OpenExtLink(window,event,this)" href="mailto:jifty-devel@lists.jifty.org">
jifty-devel@lists.jifty.org</a><br><a onclick="return top.js.OpenExtLink(window,event,this)" href="http://lists.jifty.org/cgi-bin/mailman/listinfo/jifty-devel" target="_blank">http://lists.jifty.org/cgi-bin/mailman/listinfo/jifty-devel
</a><br><br><br clear="all"></blockquote></div><br>