X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?p=catagits%2FCatalyst-Runtime.git;a=blobdiff_plain;f=TODO;h=7d3486343290ad54d89c890a0ac50fa2f9662203;hp=fd103b1274f7bd8c5173f515a86b3414445ecb6b;hb=79f5d5718d885ee1c3f288159a31562f9cf5b02f;hpb=36d5d607a1917d0c44d07571b8628d0b29ccd4c0 diff --git a/TODO b/TODO index fd103b1..7d34863 100644 --- a/TODO +++ b/TODO @@ -1,74 +1,36 @@ -Back-compat investigation: +Known Bugs: - - Get engines tested: - - Catalyst-Engine-Zeus - - Catalyst-Engine-XMPP2 - - Catalyst-Engine-HTTPEngine - - Catalyst-Engine-HTTP-Prefork - - Catalyst-Engine-Wx + - Bug ->go or ->visit causes actions which have Args or CaptureArgs called + twice when called via ->go or ->visit. -Known issues: - - Catalyst-Log-Log4perl Deep recursion on subroutine "MockApp::setup" - (rafl) + Test app: http://github.com/bobtfish/catalyst-app-bug-go_chain/tree/master - - CatalystX-CRUD and CatalystX-CRUD-ModelAdapter-DBIC - fail tests against 5.80 (karpet) +Compatibility warnings to add: - - Catalyst-Plugin-Authorization-ACL fails as - Catalyst::Dispatcher::_do_forward does not fix arguments if you throw - an exception. Needs a test case (Caelum) - - - Catalyst::Plugin::Authentication new release. - - - Catalyst::Action::RenderView - - Catalyst::Plugin::DebugCookie - - Conflict with each other: $c->config->{debug}->{foo} = 'bar' - now breaks, unsure why.. + - $self->config should warn as config should only ever be called as a + class method. -Cleanups: +Proposed functionality / feature additions: -18:01 <@mst> hmm. I think that warning should show what it found -18:01 <@mst> and say "you probably called ->model("Sessions") when you meant ->model("DB::Sessions")" or whatever -18:05 <@mst> unsafe_component_use_ok -18:05 <@mst> like we did for BindLex + - Log setup needs to be less lame, so Catalyst::Plugin::Log::* can die + in a fire. Having $c->log_class would be a good start. kane volunteered + to do some of this. + Simple example: Catalyst::Plugin::Log::Colorful should just be a + subclass of Catalyst::Log, no ::Plugin:: needed. + See also: Catalyst::Plugin::Log::Dispatch and + http://github.com/willert/catalyst-plugin-log4perl-simple/tree - - Update Test suite to not assume MyApp ISA Controller - - After that set up attr handlers that will output helpful error messages - when you do it as well as how to fix it. +TODO for brach namespace_handling_refactor: - - Eliminate all instances of $instance->{$key}, I think the only thing - left is lib/Catalyst/Engine/HTTP.pm: $self->{inputbuf}, which I haven't - touched as it is used as an lvalue in a lot of places (t0m) +- refactor code in + * Catalyst::Dispatcher::get_containers # No Idea + * Catalyst::Dispatcher::dispatch_type # DONE - - Find whatever in the test suite craps a file called GLOB(0xfoo) onto my - disk and fix it. (Believed to be in the optional TEST_HTTP tests?) (t0m) - -Documentation: - - - Manual / Tutorial updates - - - extends in components with attributes must be inside a BEGIN block. - - - How to write Moosified Catalyst components. - - - Fix the Roadmap to be less full of lies. - - - Catalyst/Upgrading.pod - -Reported bugs: - - - Issues with TWMC not being loaded when it used to be in 5.70 - (Bill Moseley) - -Profiling: - - - vs 5.70 and optimisation as needed. - -Tests: - - - Moosified test application? - - - Test & document warnings from plugin method on Catalyst + * Catalyst::Controller::_parse_ActionClass_attr # DONE + * Catalyst::Dispatcher::_load_dispatch_types # DONE + * Catalyst::setup_plugins # DONE + to use the same namespacing method +- Ok, so can you add tests for ->config(actions => { foo => { ActionClass => '+Bar' }});