On Sun, Aug 24, 2008 at 8:01 PM, Matt S Trout <[EMAIL PROTECTED]> wrote: > > On Tue, Aug 12, 2008 at 12:27:26PM -0400, Charles Bailey wrote: > > On Mon, Jul 21, 2008 at 5:32 PM, Matt S Trout <[EMAIL PROTECTED]> wrote: > > > On Mon, Jul 21, 2008 at 07:13:37AM -0400, Charles Bailey wrote: > > >> Agreed. I've had it on TODO for the past two weeks, but getting > > >> hammered at dayjob. Will try to do this week. > > > > > > We're one more dev release short of shipping. Please try because otherwise > > > it'll probably have to wait until 5.80 > > > > Patch attached, following a severe tuit drought. I'm sorry I was > > unable to turn it around faster, but I hope it's useful for the > > future. > > > > The dispatcher change is trivial; most of the size is tests. These > > are modelled closely on existing tests, with the thought that > > consistency in the test suite is better than streamlining each case. > > The dispatcher change is sadly insufficient. > > Because visit() returns, you really need to reset action and namespace. > > local is one option, simple code to do so another. > >
Hi, Here's another *go* on "go" and "visit": http://flodhest.net/files/perl/catalyst-go/ All tests succeed. I've also added some tests that check if you're trying to ->go() to a view/model. -- Best regards, Jan Henning Thorsen http://flodhest.net _______________________________________________ List: [email protected] Listinfo: http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst Searchable archive: http://www.mail-archive.com/[email protected]/ Dev site: http://dev.catalyst.perl.org/
