Bug #74869 for Catalyst-Manual: Catalyst::Manual::Intro#Action-types :Global broken doc

Sat Feb 11 05:33:26 2012 bill+bitcard [...] blunn.org - Ticket created

Subject:

Catalyst::Manual::Intro#Action-types :Global broken doc

(Sorry about raising a new ticket. I forgot to put a subject line in the original ticket, but RT wouldn't let me go back and put one in.) https://metacpan.org/module/Catalyst::Manual::Intro#Action-types "Root-level (:Global) ... Matches http://localhost:3000/foo - that is, the action is mapped directly to the controller namespace, ignoring the function name." This description seems to be wrong. It seems like it should be: "... that is, the action is mapped directly to the method name, ignoring the controller namespace."

Sat Feb 11 22:02:14 2012 ZARQUON [...] cpan.org - Correspondence added

Fixed in git f577e42d6082c6b03 http://git.shadowcat.co.uk/gitweb/gitweb.cgi?p=catagits/Catalyst-Manual.git;a=commitdiff;h=f577e42d6082c6b034295995286d248e57e290e0 Thanks :)

Sat Feb 11 22:02:17 2012 The RT System itself - Status changed from 'new' to 'open'

Sat Feb 11 22:02:17 2012 ZARQUON [...] cpan.org - Status changed from 'open' to 'resolved'

Sun Feb 12 05:42:36 2012 bill+bitcard [...] blunn.org - Correspondence added

From:

bill+bitcard [...] blunn.org

On Sat Feb 11 22:02:14 2012, ZARQUON wrote: Show quoted text

> Fixed in git f577e42d6082c6b03 > > http://git.shadowcat.co.uk/gitweb/gitweb.cgi?p=catagits/Catalyst- > Manual.git;a=commitdiff;h=f577e42d6082c6b034295995286d248e57e290e0 > > Thanks :)

It still says Show quoted text

> Matches http://localhost:3000/foo - that is, the action is mapped > +directly to the controller namespace, ignoring the function name.

Is that not still wrong? It still says (written conceptually): {mapped directly to} : {the controller namespace} {ignores} : {function name} I think it should say {mapped directly to} : {the method name} {ignores} : {the controller namespace} i.e. that's two changes: 1. Swap the two concepts around 2. Change "function name" to "method name" Regards, Bill

Sun Feb 12 05:42:38 2012 The RT System itself - Status changed from 'resolved' to 'open'

Sun Feb 12 06:01:08 2012 rt-cpan [...] trout.me.uk - Correspondence added

On Sun Feb 12 05:42:36 2012, hochgurgler wrote: Show quoted text

> On Sat Feb 11 22:02:14 2012, ZARQUON wrote:

> > Fixed in git f577e42d6082c6b03 > > > > http://git.shadowcat.co.uk/gitweb/gitweb.cgi?p=catagits/Catalyst- > > Manual.git;a=commitdiff;h=f577e42d6082c6b034295995286d248e57e290e0 > > > > Thanks :)

> > It still says >

> > Matches http://localhost:3000/foo - that is, the action is mapped > > +directly to the controller namespace, ignoring the function name.

> > Is that not still wrong?

Bollocks. Yes. Show quoted text

> It still says (written conceptually): > > {mapped directly to} : {the controller namespace} > {ignores} : {function name} > > I think it should say > > {mapped directly to} : {the method name} > {ignores} : {the controller namespace} > > i.e. that's two changes: > > 1. Swap the two concepts around > > 2. Change "function name" to "method name"

rev 7010157: -directly to the controller namespace, ignoring the function name. +directly to the method name, ignoring the controller namespace. Cheers!

Sun Feb 12 06:01:09 2012 rt-cpan [...] trout.me.uk - Status changed from 'open' to 'resolved'