Skip Menu |

This queue is for tickets about the Catalyst-Runtime CPAN distribution.

Report information
The Basics
Id: 57780
Status: rejected
Priority: 0/
Queue: Catalyst-Runtime
People
Owner: bobtfish [...] bobtfish.net
Requestors: AJGB [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Important
Broken in: 5.80024
Fixed in: (no value)

History Show all quoted text
  Sun May 23 09:23:44 2010 AJGB [...] cpan.org - Ticket created
Subject: Global atrributes set via config( action => { '*' => ...} ) [with patch]
Hi, When setting attributes with: __PACKAGE__->config( action => { '*' => { Chained => 'base', Args => 0 }, base => { Chained => '/', PathPart => '', CaptureArgs => 0 }, }, ); the attributes defined for all methods are applied to Private methods as well. Fix with test available at: http://github.com/ajgb/catalyst-runtime/commit/616f605745b045d3bddebaf71264035ccfe6345e Thanks, Alex
  Wed Jul 28 18:40:40 2010 bobtfish [...] bobtfish.net - Correspondence added
Heya. Sorry it's taken aaaages, but I've read your patch and I'm don't think that this is a bug fix, it's a new feature.. I'm against overlaying yet more magic onto : Private (which already has magical meanings that can trip you up). Also, I'm not convinced that the tests included in any way test the functionality changed here. Can you grab me on irc for a chat about this when you have a sec?
  Wed Jul 28 18:40:42 2010 The RT System itself - Status changed from 'new' to 'open'
  Wed Jul 28 18:57:19 2010 bobtfish [...] bobtfish.net - Taken
  Wed Jul 28 18:57:29 2010 bobtfish [...] bobtfish.net - Status changed from 'open' to 'rejected'