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.
--
Matt S Trout Need help with your Catalyst or DBIx::Class project?
Technical Director http://www.shadowcat.co.uk/catalyst/
Shadowcat Systems Ltd. Want a managed development or deployment platform?
http://chainsawblues.vox.com/ http://www.shadowcat.co.uk/servers/
_______________________________________________
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/
