X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?p=catagits%2FCatalyst-Runtime.git;a=blobdiff_plain;f=lib%2FCatalyst%2FDispatchType%2FRegex.pm;h=c40be54e5c30433450c3402708470065a7b6115b;hp=94702974a1f4fdb8072f2601a401e3a714c1056e;hb=ea0e58d9d85d93bf94c3ffa6b4d7b71b200bbdf1;hpb=4ab87e274ac0a05f98c10a4cdba467ba4398b0d3 diff --git a/lib/Catalyst/DispatchType/Regex.pm b/lib/Catalyst/DispatchType/Regex.pm index 9470297..c40be54 100644 --- a/lib/Catalyst/DispatchType/Regex.pm +++ b/lib/Catalyst/DispatchType/Regex.pm @@ -3,6 +3,7 @@ package Catalyst::DispatchType::Regex; use strict; use base qw/Catalyst::DispatchType::Path/; use Text::SimpleTable; +use Text::Balanced (); =head1 NAME @@ -24,7 +25,7 @@ Output a table of all regex actions, and their private equivalent. sub list { my ( $self, $c ) = @_; - my $re = Text::SimpleTable->new( [ 36, 'Regex' ], [ 37, 'Private' ] ); + my $re = Text::SimpleTable->new( [ 35, 'Regex' ], [ 36, 'Private' ] ); for my $regex ( @{ $self->{compiled} } ) { my $action = $regex->{action}; $re->row( $regex->{path}, "/$action" ); @@ -48,11 +49,11 @@ sub match { # Check path against plain text first foreach my $compiled ( @{ $self->{compiled} || [] } ) { - if ( my @snippets = ( $path =~ $compiled->{re} ) ) { + if ( my @captures = ( $path =~ $compiled->{re} ) ) { next unless $compiled->{action}->match($c); $c->req->action( $compiled->{path} ); $c->req->match($path); - $c->req->snippets( \@snippets ); + $c->req->captures( \@captures ); $c->action( $compiled->{action} ); $c->namespace( $compiled->{action}->namespace ); return 1; @@ -73,7 +74,7 @@ Returns 1 on if any regexps were registered. sub register { my ( $self, $c, $action ) = @_; - my $attrs = $action->attributes; + my $attrs = $action->attributes; my @register = @{ $attrs->{'Regex'} || [] }; foreach my $r (@register) { @@ -104,6 +105,38 @@ sub register_regex { ); } +=head2 $self->uri_for_action($action, $captures) + +returns a URI for this action if it can find a regex attributes that contains +the correct number of () captures. Note that this may function incorrectly +in the case of nested captures - if your regex does (...(..))..(..) you'll +need to pass the first and third captures only. + +=cut + +sub uri_for_action { + my ( $self, $action, $captures ) = @_; + + if (my $regexes = $action->attributes->{Regex}) { + REGEX: foreach my $orig (@$regexes) { + my $re = "$orig"; + $re =~ s/^\^//; + $re =~ s/\$$//; + my $final = '/'; + my @captures = @$captures; + while (my ($front, $rest) = split(/\(/, $re, 2)) { + ($rest, $re) = + Text::Balanced::extract_bracketed("(${rest}", '('); + next REGEX unless @captures; + $final .= $front.shift(@captures); + } + next REGEX if @captures; + return $final; + } + } + return undef; +} + =head1 AUTHOR Matt S Trout