X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=TODO;h=694dc0e0bbd7ebea307b3ee83a31eb13b7d7e86e;hb=d91474f509dab7dce4c3e4328e674ab9fea29284;hp=318654aa5eb1908cfed77830b156dc8968be9e95;hpb=dae2b0faaf2982bde81606ea0392c13fd9309cb1;p=catagits%2FCatalyst-Runtime.git diff --git a/TODO b/TODO index 318654a..694dc0e 100644 --- a/TODO +++ b/TODO @@ -1,69 +1,96 @@ -TBD pre 5.8000_04 dev release: - - - Go through everything which used to use CAF, and ensure that it now uses MX::E::CAF + tests, - extending t/caf_backcompat.t and replacing t/custom_request.t (t0m) - - - Simple test for NEXT compat in core (t0m). - - - Looks like stash is not available during prepare_path when running under - Catalyst::Test.. Run MojoMojo's t/01app.t and it will explode with - "Can't use an undefined value as a HASH reference" - test case for core - + fix maybe (marcus) - ---- +Pending patches: + - meta test for MX::Emulate::CAF needed by Catalyst::Plugin::Cache::Curried + + - Class::Accessor::Chained::Fast test for MX::Emulate::CAF, to fix + HTML::Widget + + - Re-opening packages with MX::Emulate::CAF (for + Catalyst::Plugin::HashedCookies) + +Back-compat investigation / known issues: - - Make the skipped test at the bottom of t/aggregate/live_engine_request_uri.t - pass / not be skipped. (From what C::P::SmartURI used to do) + - FIXME notes in Catalyst::Dispatcher - This can be fixed by the following patch: + - Common engine test failures, look into and get tests into core. -Index: lib/Catalyst/Request.pm -=================================================================== ---- lib/Catalyst/Request.pm (revision 8709) -+++ lib/Catalyst/Request.pm (working copy) -@@ -96,7 +96,7 @@ - lazy => 1, - default => sub { - my $self = shift; -- return $self->path if $self->uri; -+ return $self->{path} if $self->{uri}; - }, - ); + - CatalystX-Imports, Class::MOP doesn't consider anon subs in the symbol + table as methods, tests + fix, or explanation and documentation? + (rafl & phaylon) + + - Run another round of repository smokes against latest 5.80 trunk, manually + go through all the things which are broken (t0m). + + - Catalyst::Plugin::Authorization::ACL + - Catalyst::Plugin::Server + - Should hopefully be fixed now.. + + - Catalyst-Plugin-Cache dies due to mk_accessors('meta') + + - Catalyst-Action-REST appears to have real issues, investigate + + - Catalyst-Model-Search ditto + + - CatalystX-CRUD ditto + + - Catalyst-Engine-Apache - Deep recursion on subroutine + "Catalyst::Action::execute", investigate once all other tests pass + + - Catalyst-Plugin-Scheduler + - Catalyst-Engine-HTTPEngine + - Catalyst-View-HTML-Template + - Catalyst-View-XSLT + - Catalyst-Plugin-Setenv + - Catalyst-Plugin-RequireSSL - Bullshit 'cannot locate' errors + Bug in smoke test rig. + + - Issues with TWMC not being loaded when it used to be in 5.70 + (Bill Moseley) + + - Fix memory leaks (I already tried Devel::Leak::Object, but no joy). + + - In 5.70, you can have a component which doesn't inherit from ::Component, + and has no new method, and it'll do the right thing for you. We just + ignore that, we should still support that, but warn for/deprecate it so + it can go for 5.9X... + This all obviously needs better tests :/ + +Cleanups: + + - Catalyst-Plugin-Authorization-ACL, Can't locate object method "tree" + via package "Catalyst::Dispatcher", fix the plugin as tree was never + a public method. + + - Catalyst-Model-Akismet, incorrectly inherits from + Catalyst::Component::InstancePerContext, should compose as a role. + Fix the module. + + - 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. (done already?) - But I'd like a 2nd opinion from someone who knows core better than me - about if that is the correct fix.. (t0m / Caelum) + - Comments marked /Moose TODO/i in Catalyst::Request re {_body} (t0m) - - Common engine test failures, look into and get tests into core. + - 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) - - Catalyst-Plugin-Authorization-ACL, Can't locate object method "tree" via package "Catalyst::Dispatcher", fix the plugin as tree was never a public method. + - Find whatever in the test suite craps a file called GLOB(0xfoo) onto my + disk and fix it. (Believed to be in the optional tests?) (t0m) - - Catalyst-Model-Akismet, incorrectly inherits from Catalyst::Component::InstancePerContext, should compose as a role, fix the module. +Documentation: - - CatalystX-Imports, Class::MOP doesn't consider anon subs in the symbol table as methods, tests + fix? + - extends in components must be inside a BEGIN block. - - Catalyst::Plugin::Cache::Curried - - Catalyst::Controller::WrapCGI - - Both import 'meta' into controller package - mst thinks this can be fixed?. - but @marcus> and the generated accessors even call $_[0]->meta, so I don't think we can remove it. - . Add a test for this in core. - . Fix core to use CMOP to look up metaclass, rather than ->meta method, and - namespace::clean the meta method out, fix generated accessors to do the same? + - How to write Moosified Catalyst components. - - MooseX::Emulate::Class::Accessor::Fast - - Tests for uses of C::A::F from Catalyst repos. - t0m - - New release once these are done. + - Manual / Tutorial updates - - 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. (done already?) - - - Comments marked /Moose TODO/i in the code - - - Eliminate all instances of $instance->{$key} + - Fix the Roadmap to be less full of lies. - - Catalyst-Log-Log4perl - deep recursion in the test suite, investigate +Profiling: - - Profiling vs 5.70 and optimisation as needed. + - vs 5.70 and optimisation as needed. - - http://lists.scsys.co.uk/pipermail/catalyst-dev/2008-November/001546.html - - patch to list, andyg to look at? +Tests: + - Moosified test application?