use Moose;
use Scalar::Util 'looks_like_number';
+use Moose::Util::TypeConstraints ();
with 'MooseX::Emulate::Class::Accessor::Fast';
use namespace::clean -except => 'meta';
default => sub { '/'.shift->reverse },
);
+has number_of_args => (
+ is=>'ro',
+ init_arg=>undef,
+ isa=>'Int|Undef',
+ required=>1,
+ lazy=>1,
+ builder=>'_build_number_of_args');
+
+ sub _build_number_of_args {
+ my $self = shift;
+ return 0 unless exists $self->attributes->{Args};
+ if(!defined($self->attributes->{Args}[0])) {
+ # When its 'Args' that internal cue for 'unlimited'
+ return undef;
+ } elsif(looks_like_number($self->attributes->{Args}[0])) {
+ # 'old school' numberd args (is allowed to be undef as well)
+ return $self->attributes->{Args}[0];
+ } else {
+ # new hotness named arg constraints
+ return $self->number_of_args_constraints;
+ }
+ }
+
+has args_constraints => (
+ is=>'ro',
+ init_arg=>undef,
+ traits=>['Array'],
+ isa=>'ArrayRef',
+ required=>1,
+ lazy=>1,
+ builder=>'_build_args_constraints',
+ handles => {
+ has_args_constraints => 'count',
+ number_of_args_constraints => 'count',
+ });
+
+ sub _build_args_constraints {
+ my $self = shift;
+ my @arg_protos = @{$self->attributes->{Args}||[]};
+
+ return [] unless scalar(@arg_protos);
+ # If there is only one arg and it looks like a number
+ # we assume its 'classic' and the number is the number of
+ # constraints.
+ my @args = ();
+ if(
+ scalar(@arg_protos) == 1 &&
+ looks_like_number($arg_protos[0])
+ ) {
+ return \@args;
+ } else {
+ @args = map { Moose::Util::TypeConstraints::find_or_parse_type_constraint($_) || die "$_ is not a constraint!" } @arg_protos;
+ }
+
+ return \@args;
+ }
+
use overload (
# Stringify to reverse for debug output etc.
);
-
-
no warnings 'recursion';
sub dispatch { # Execute ourselves against a context
$self->code->(@_);
}
-sub match_captures {
- my ( $self, $c, $captures ) = @_;
- ## It would seem that now that we can match captures, we could remove a lot
- ## of the capture_args to args mapping all around. I gave it a go, but was
- ## not trival, contact jnap on irc for what I tried if you want to try.
- ## return $self->_match_has_expected_capture_args($captures) &&
- return $self->_match_has_expected_http_method($c->req->method);
-}
-
sub match {
- my ( $self, $c ) = @_;
- return $self->_match_has_expected_args($c->req->args) &&
- $self->_match_has_expected_http_method($c->req->method);
-}
-
-sub _match_has_expected_args {
- my ($self, $req_args) = @_;
- return 1 unless exists $self->attributes->{Args};
- my $args = $self->attributes->{Args}[0];
- return 1 unless defined($args) && length($args);
- return scalar( @{$req_args} ) == $args;
+ my ( $self, $c ) = @_;
+ #would it be unreasonable to store the number of arguments
+ #the action has as its own attribute?
+ #it would basically eliminate the code below. ehhh. small fish
+ return 1 unless exists $self->attributes->{Args};
+ my $args = $self->attributes->{Args}[0];
+ return 1 unless defined($args) && length($args);
+
+ if($self->has_args_constraints) {
+ for my $i($#{ $c->req->args }) {
+ $self->args_constraints->[$i]->check($c->req->args->[$i]) || return 0;
+ }
+ return 1;
+ } else {
+ return scalar( @{ $c->req->args } ) == $args;
+ }
}
-#sub _match_has_expected_capture_args {
-# my ($self, $req_args) = @_;
-# return 1 unless exists $self->attributes->{CaptureArgs};
-# my $args = $self->attributes->{CaptureArgs}[0];
-# return 1 unless defined($args) && length($args);
-# return scalar( @{$req_args} ) == $args;
-#}
-
-sub _match_has_expected_http_method {
- my ($self, $method) = @_;
- my @methods = @{ $self->attributes->{Method} || [] };
- if(scalar @methods) {
- my $result = scalar(grep { lc($_) eq lc($method) } @methods) ? 1:0;
- return $result;
- } else {
- ## No HTTP Methods to check
- return 1;
- }
-}
+sub match_captures { 1 }
sub compare {
my ($a1, $a2) = @_;
- my ($a1_args) = @{ $a1->attributes->{Args} || [] };
- my ($a2_args) = @{ $a2->attributes->{Args} || [] };
+ # Wen there is no declared Args for Local and Path (and Default??) we
+ # say that means any number of args... If Args exists however we use
+ # the number of args as determined by inspecting the value of it.
+
+ my $a1_args = exists($a1->attributes->{Args}) ? $a1->number_of_args : ~0;
+ my $a2_args = exists($a2->attributes->{Args}) ? $a2->number_of_args : ~0;
- $_ = looks_like_number($_) ? $_ : ~0
+ # If we did have an Args but it was undefined value (:Args() or :Args), that
+ # is the cue for 'as many args as you like also...
+ #
+ $_ = defined($_) ? $_ : ~0
for $a1_args, $a2_args;
return $a1_args <=> $a2_args;
}
-sub number_of_args {
- my ( $self ) = @_;
- return 0 unless exists $self->attributes->{Args};
- return $self->attributes->{Args}[0];
-}
-
sub number_of_captures {
my ( $self ) = @_;
return $self->attributes->{CaptureArgs}[0] || 0;
}
+sub scheme {
+ return exists $_[0]->attributes->{Scheme} ? $_[0]->attributes->{Scheme}[0] : undef;
+}
+
+sub list_extra_info {
+ my $self = shift;
+ return {
+ Args => $self->attributes->{Args}[0],
+ CaptureArgs => $self->number_of_captures,
+ }
+}
+
__PACKAGE__->meta->make_immutable;
1;
=head2 class
Returns the name of the component where this action is defined.
-Derived by calling the L<Catalyst::Component/catalyst_component_name|catalyst_component_name>
+Derived by calling the L<catalyst_component_name|Catalyst::Component/catalyst_component_name>
method on each component.
=head2 code
Returns the number of captures this action expects for L<Chained|Catalyst::DispatchType::Chained> actions.
+=head2 list_extra_info
+
+A HashRef of key-values that an action can provide to a debugging screen
+
+=head2 scheme
+
+Any defined scheme for the action
+
=head2 meta
Provided by Moose.
the same terms as Perl itself.
=cut
+
+
projects / catagits/Catalyst-Runtime.git / blobdiff