Merge 'add_captures_to_visit' into 'trunk'

r9541@dhcp-68 (orig r9539):  t0m | 2009-03-21 15:03:21 +0000
Visit currently does nothing with CaptureArgs, as failing tests in trunk demonstrate.
Branch as fixing this involves multiple commits and changing the visit api, etc..

r9542@dhcp-68 (orig r9540):  t0m | 2009-03-21 15:04:33 +0000
Make CaptureArgs get passed, this makes the test less fail, but not perfect yet. Also, needs docs..
r9543@dhcp-68 (orig r9541):  t0m | 2009-03-21 15:17:38 +0000
Fix the go test
r9544@dhcp-68 (orig r9542):  t0m | 2009-03-21 15:54:33 +0000
Doc changes for passing CaptureArgs
r9545@dhcp-68 (orig r9543):  t0m | 2009-03-21 15:56:08 +0000
Hack _invoke_as_component in a horrible way, so that it gives us back a path Action if it can, which makes >visit work correctly with qw/MyApp::Controller::Chained method_name/. I feel dirty, but it makes the test pass..
r9546@dhcp-68 (orig r9544):  t0m | 2009-03-21 18:04:06 +0000
Much cleaner. The visit class test wasn't actually testing visiting the class, so I fixed that and made the English in the error message more correct (IMO)
r9547@dhcp-68 (orig r9545):  t0m | 2009-03-21 18:11:04 +0000
Unfuck last commit, fix go tests, remove unneeded crud from TestApp, add FIXME for more cleanup
r9548@dhcp-68 (orig r9546):  t0m | 2009-03-21 20:39:52 +0000
Cleanup, changelog

diff --cc Changes
index f32b517,3f9c0a3..4433adf
--- 1/Changes
--- 2/Changes
+++ b/Changes
@@@ -1,8 -1,12 +1,15 @@@
  # This file documents the revision history for Perl extension Catalyst.
  
 +        - Throw an exception rather than loading an app if an action
 +          tries to chain to itself (t0m)
 +          - Tests for this
+         - Change the $c->visit and $c->go methods to optionally take
+           CaptureArgs, making them useful to call ActionChains with (t0m)
+           - Tests for this (radek)
+         - Fix _invoke_as_component method to find the proper action instance
+           for dispatchable actions so that ->visit or ->going to ActionChains
+           with qw/Class::Name method_name/ works correctly (t0m)
+           - Tests for this (radek)
          - Added Catalyst::Test::ctx_request to be able to inspect
            the context object after a request is made (Jos Boumans)
          - debug() POD rewrite (jhannah)