X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?p=catagits%2FCatalyst-Runtime.git;a=blobdiff_plain;f=lib%2FCatalyst%2FDispatcher.pm;h=cf9825607a1cac636404115b7798c48ec6691f8b;hp=c51510e148fb04022ae249101dec9bc2447dea60;hb=3ac082e46385d8f5a94838e8790538dc1f694565;hpb=d7325e05db00f4eff6b1520368c9c2bda5ab931c diff --git a/lib/Catalyst/Dispatcher.pm b/lib/Catalyst/Dispatcher.pm index c51510e..cf98256 100644 --- a/lib/Catalyst/Dispatcher.pm +++ b/lib/Catalyst/Dispatcher.pm @@ -407,9 +407,14 @@ sub prepare_action { if ( $c->debug && @args ); } -=head2 $self->get_action( $action, $namespace ) +=head2 $self->get_action( $action_name, $namespace ) -returns a named action from a given namespace. +returns a named action from a given namespace. C<$action_name> +may be a relative path on that C<$namespace> such as + + $self->get_action('../bar', 'foo/baz'); + +In which case we look for the action at 'foo/bar'. =cut @@ -419,17 +424,22 @@ sub get_action { $namespace = join( "/", grep { length } split '/', ( defined $namespace ? $namespace : "" ) ); - return $self->_action_hash->{"${namespace}/${name}"}; + return $self->get_action_by_path("${namespace}/${name}"); } =head2 $self->get_action_by_path( $path ); Returns the named action by its full private path. +This method performs some normalization on C<$path> so that if +it includes '..' it will do the right thing (for example if +C<$path> is '/foo/../bar' that is normalized to '/bar'. + =cut sub get_action_by_path { my ( $self, $path ) = @_; + $path =~s/[^\/]+\/\.\.\/// while $path=~m/[^\/]+\/\.\.\//; $path =~ s/^\///; $path = "/$path" unless $path =~ /\//; $self->_action_hash->{$path};