On Dec 18, 2007 2:00 AM, Matt S Trout <[EMAIL PROTECTED]> wrote: > On Mon, Dec 17, 2007 at 08:39:29PM +0000, Zbigniew Lukasiak wrote: > > Yeah - some time ago I proposed to add an EndPoint attribute and get > > rid of the CaptureArgs one that is not very intuitive (and use Args in > > both cases). > > I don't remember seeing the code - if you update your patch to work against > 5.80 trunk we can have a look ...
Ok - so I attach a patch (it is against http://dev.catalyst.perl.org/repos/Catalyst/branches/current/Catalyst-Runtime - I hope that is the 5.80 trunk - if not tell me and I'll change it). It passes the tests. In two cases when the test application uses the original TestApp::Controller::Action::Chained::* instead of TestApp::Controller::Action::ChainedEndPoint::* - I think this is due to how the test application is build. I've marked those cases in the test file and modified the tests so that they pass. For the tests I've copied the tests of live_component_controller_action_chained.t and adopted it to test TestApp::Controller::Action::ChainedEndPoint instead of TestApp::Controller::Action::Chained. So - what do you think? Zbigniew http://perlalchemy.blogspot.com/
ChainedEndPoint.diff.gz
Description: GNU Zip compressed data
_______________________________________________ List: [email protected] Listinfo: http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst Searchable archive: http://www.mail-archive.com/[EMAIL PROTECTED]/ Dev site: http://dev.catalyst.perl.org/
