Skip Menu |

This queue is for tickets about the CGI-Application-Plugin-ActionDispatch CPAN distribution.

Report information
The Basics
Id: 30989
Status: resolved
Priority: 0/
Queue: CGI-Application-Plugin-ActionDispatch

People
Owner: Nobody in particular
Requestors: clscott [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 0.90
Fixed in: (no value)



Subject: Basic Path dispatches don't work (patch attached)
When using a basic Path attribute like: Path('/add') it won't match due to a regexp that requires more information than that to match. I've made the regexp a little less greedy in this patch against ActionDispatch.pm version 0.90 Thanks for a great module, Clayton Scott
Subject: CGI-Application-Plugin-ActionDispatch.patch
--- ActionDispatch.pm.orig 2007-11-06 14:57:38.000000000 -0500 +++ ActionDispatch.pm 2007-11-06 14:59:08.000000000 -0500 @@ -22,7 +22,7 @@ $data = "/" . $data; } - my $regex = qr/^$data\/?(\/.*)$/; + my $regex = qr/^$data\/?(\/.*)?$/; push(@{ $_attr_cache{$attr} }, [ $referent, $regex ]); }
From: jaywhy [...] gmail.com
On Mon Nov 26 16:20:34 2007, CLSCOTT wrote: Show quoted text
> When using a basic Path attribute like: > > Path('/add') it won't match due to a regexp that requires more > information than that to match. > > I've made the regexp a little less greedy in this patch against > ActionDispatch.pm version 0.90 > > Thanks for a great module, > Clayton Scott
Thanks for the patch. I have a couple small patches sitting in the queue. I'll probably update it on CPAN within the week.
Ironically enough this was already fixed on my version. Whoops.