Re: [Catalyst] Path is "/"

2011-09-06 Thread Nick
On 2011-09-06 22:09, Will Crawford wrote: > On 6 September 2011 15:12, Nick wrote: > ... > >  [debug] "GET" request for "login/124/audit_log" from "127.0.0.1" > ... > > Doesn't match /login/id/*/audit_log (missing the "id" path part). > > Sorry. > > :) No need to be sorry, my mistake. Unfortunat

Re: [Catalyst] Path is "/"

2011-09-06 Thread Will Crawford
On 6 September 2011 15:12, Nick wrote: ... >  [debug] "GET" request for "login/124/audit_log" from "127.0.0.1" ... Doesn't match /login/id/*/audit_log (missing the "id" path part). Sorry. :) ___ List: Catalyst@lists.scsys.co.uk Listinfo: http://lists

Re: [Catalyst] Path is "/"

2011-09-06 Thread Dimitar Petrov
I've just tested both with Catalyst 5.90002 and 5.80032 seems to be just fine? Here is the output and the Login.pm controller. # Login controller package testapp::Controller::Login; use Moose; use namespace::autoclean; BEGIN { extends 'Catalyst::Controller' } sub get_sess :Chained('/') :PathPart

Re: [Catalyst] Path is "/"

2011-09-06 Thread Nick
On 2011-09-06 17:33, Tomas Doran wrote: > >>sub default : Chained('/') PathPart('') Args() { > > > >I made that change, but the default action still runs. > > New debug please? Sure, $ CATALYST_DEBUG=1 script/myapp_server.pl [debug] Debug messages enabled [debug] Statistics enabled [debug] Loaded

Re: [Catalyst] Path is "/"

2011-09-06 Thread Tomas Doran
On 6 Sep 2011, at 15:49, Nick wrote: On 2011-09-06 15:37, Tomas Doran wrote: On 6 Sep 2011, at 15:30, Nick wrote: | /default/...| /default | This is your bug, probably as you copied what I said without stopping to think about it (and I forgot a bit!) :) sub defaul

Re: [Catalyst] Path is "/"

2011-09-06 Thread Nick
On 2011-09-06 15:37, Tomas Doran wrote: > > On 6 Sep 2011, at 15:30, Nick wrote: > > >| /default/...| /default > >| > > This is your bug, probably as you copied what I said without > stopping to think about it (and I forgot a bit!) :) > > sub default : Chained('/') PathPart(

Re: [Catalyst] Path is "/"

2011-09-06 Thread Tomas Doran
On 6 Sep 2011, at 15:30, Nick wrote: | /default/...| / default | This is your bug, probably as you copied what I said without stopping to think about it (and I forgot a bit!) :) sub default : Chained('/') PathPart('') Args() { Cheers t0

Re: [Catalyst] Path is "/"

2011-09-06 Thread Nick
On 2011-09-06 15:17, Tomas Doran wrote: > Fancy sharing the full debug startup for the app (with my suggested > change) and the request in question? Sure, thanks for looking at it. The startup messages and response to the request http://localhost:3000/login/124/audit_log follow below. The source

Re: [Catalyst] Path is "/"

2011-09-06 Thread Tomas Doran
On 6 Sep 2011, at 15:12, Nick wrote: On 2011-09-06 14:47, Tomas Doran wrote: This is due to you mixing path and chained actions. The path actions take higher priority if they're able to match at all.. [debug] Loaded Path actions: .- +-

Re: [Catalyst] Path is "/"

2011-09-06 Thread Nick
On 2011-09-06 14:47, Tomas Doran wrote: > This is due to you mixing path and chained actions. The path actions > take higher priority if they're able to match at all.. > > > [debug] Loaded Path actions: > > .-+--. > > | Path

Re: [Catalyst] Path is "/"

2011-09-06 Thread Tomas Doran
On 6 Sep 2011, at 14:34, Nick wrote: Catalyst isn't dispatching how I expected. A 'Login' controller has This is due to you mixing path and chained actions. The path actions take higher priority if they're able to match at all.. [debug] Loaded Path actions: .--

[Catalyst] Path is "/"

2011-09-06 Thread Nick
Catalyst isn't dispatching how I expected. A 'Login' controller has sub get_sess :Chained('/') :PathPart('login/id') :CaptureArgs(1) {..} sub audit_log :Chained('get_sess') :PathPart('audit_log') :Args(0) {..} On startup, Catalyst reports the actions as expected, | /login/id/*/audit_log