X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?p=catagits%2FCatalyst-Runtime.git;a=blobdiff_plain;f=TODO;h=3b26ba36eae03319a63a93a37a6e1c5ba73f8e3f;hp=d22b09fae5eae60a0ca005921f242a03d349847c;hb=df3ea11bb1ad740663f2a4c909fba71612aa6a48;hpb=f6ba9b548282aeef7c4a03e4ae849586d7e3c60b diff --git a/TODO b/TODO index d22b09f..3b26ba3 100644 --- a/TODO +++ b/TODO @@ -1,41 +1,91 @@ - - Fix t/caf_backcompat_plugin_accessor_override.t - - - 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 + +Back-compat investigation / known issues: - - Common engine test failures, look into and get tests into core. + - Get engines tested: + - Catalyst-Engine-Zeus + - Catalyst-Engine-XMPP2 + - Catalyst-Engine-HTTPEngine + - Catalyst-Engine-HTTP-Prefork + - Catalyst-Engine-SCGI + - Catalyst-Engine-Wx - - Catalyst-Plugin-Authorization-ACL, Can't locate object method "tree" via package "Catalyst::Dispatcher", fix the plugin as tree was never a public method. + - CatalystX-Imports, Class::MOP doesn't consider anon subs in the symbol + table as methods, tests + fix, or explanation and documentation? + (rafl & phaylon) - - Catalyst-Model-Akismet, incorrectly inherits from Catalyst::Component::InstancePerContext, should compose as a role, fix the module. + - Run another round of repository smokes against latest 5.80 trunk, manually + go through all the things which are broken (t0m). + + - Catalyst-Plugin-Session-State-Cookie + Catalyst-Plugin-Session-Store-FastMmap + Catalyst-Plugin-Session-PerUser + Catalyst-Plugin-Session-Store-File + Catalyst-Authentication-Credential-HTTP + Catalyst-Plugin-SmartURI + - All fixed by Scope::Upper + + - Catalyst-Log-Log4perl Deep recursion on subroutine "MockApp::setup" + (rafl) + + - Catalyst-Plugin-Cache dies due to mk_accessors('meta') + + - CatalystX-CRUD fails tests against 5.80 (karpet) - - CatalystX-Imports, Class::MOP doesn't consider anon subs in the symbol table as methods, tests + fix? + - 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::Store::DBIC fails as + overriding the setup_finished class data method in a plugin no longer + works correctly (see t/cdi_backcompat_accessor_override.t). - - 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? + - Issues with TWMC not being loaded when it used to be in 5.70 + (Bill Moseley) - - MooseX::Emulate::Class::Accessor::Fast - - Tests for uses of C::A::F from Catalyst repos. - t0m - - New release once these are done. + - 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 should + still support that, but warn for/deprecate it so it can go for 5.9X... + This obviously needs better tests :/ + +Cleanups: + - 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} + - After that set up attr handlers that will output helpful error messages + when you do it as well as how to fix it. + + - 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) + + - 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) + + - Deprecate runtime plugins with warning. + +Documentation: + + - extends in components with attributes must be inside a BEGIN block. + + - How to write Moosified Catalyst components. + + - Manual / Tutorial updates + + - Fix the Roadmap to be less full of lies. + + - With 5.7 people did extends qw/Moose::Object Catalyst::Component/, now + Catalyst::Component isa Moose::Object so now isa doesn't linearize + anymore, docs of what doesn't work and why (rafl) + +Profiling: - - Catalyst-Log-Log4perl - deep recursion in the test suite, investigate + - vs 5.70 and optimisation as needed. - - Profiling vs 5.70 and optimisation as needed. +Tests: - - http://lists.scsys.co.uk/pipermail/catalyst-dev/2008-November/001546.html - - patch to list, andyg to look at? + - Moosified test application? + - Test warning from back-compat methods in Catalyst::Dispatcher