X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FCatalyst%2FView%2FComponent%2FSubInclude%2FSubRequest.pm;h=7ba7828d9f938d6518aac5defed3b2d435be0ca1;hb=19ff3d049dbf1234af0620b0e71450c983943498;hp=2e97236c5276d848ffcaddc00568b23519eebe88;hpb=6cb4b0016e23c2bbd9759aadfd59d00cd35b8ed9;p=catagits%2FCatalyst-View-Component-SubInclude.git diff --git a/lib/Catalyst/View/Component/SubInclude/SubRequest.pm b/lib/Catalyst/View/Component/SubInclude/SubRequest.pm index 2e97236..7ba7828 100644 --- a/lib/Catalyst/View/Component/SubInclude/SubRequest.pm +++ b/lib/Catalyst/View/Component/SubInclude/SubRequest.pm @@ -11,11 +11,11 @@ Catalyst::View::Component::SubInclude::SubRequest - Sub-requests plugin for C::V =head1 VERSION -Version 0.02 +Version 0.03 =cut -our $VERSION = '0.02'; +our $VERSION = '0.03'; =head1 SYNOPSIS @@ -55,13 +55,17 @@ It requires L. =head2 C -This will translate to the following sub-request call: +This will make a sub-request call to the action specified by C<$path>. Note that +C<$path> should be the private action path - translation to the public path is +handled internally. - $c->sub_request( $path, {}, @args ); +So, after path translation, the call will be (roughly) equivalent to: + + $c->sub_request( $translated_path, {}, @args ); Notice that the stash will always be empty. This behavior could be configurable in the future through an additional switch - for now, this behavior guarantees a -common interface for plugins. +common interface for all plugins. =cut @@ -71,11 +75,13 @@ sub generate_subinclude { croak "subincludes through subrequests require Catalyst::Plugin::SubRequest" unless $c->can('sub_request'); + + my $args = ref $params[0] eq 'ARRAY' ? shift @params : []; my $dispatcher = $c->dispatcher; - my ($action, $args) = $dispatcher->_invoke_as_path( $c, $path, @params ); + my ($action) = $dispatcher->_invoke_as_path( $c, $path, $args ); - my $uri = $c->uri_for( $action, $args ); + my $uri = $c->uri_for( $action, $args, @params ); $c->sub_request( $uri->path, $stash, @params ); }