Update the TODO list with everything discussed in #catalyst-dev over the weekend...
[catagits/Catalyst-Runtime.git] / TODO
1   - Class::Accessor::Fast extra compatibility methods for MX::Emulate::CAF - groditi
2     - Tests for uses of C::A::F from Catalyst repos. - t0m
3
4   - Finish off and CPAN Class::C3::Adopt::NEXT - rafl 
5     - More POD - t0m
6     - Make core use it.
7     - Simple test for next compat in core.
8
9   - Common engine test failures, look into and get tests into core.
10
11   - Catalyst-Plugin-Authorization-ACL, Can't locate object method "tree" via package "Catalyst::Dispatcher", fix the plugin as tree was never a public method.
12
13   - Catalyst-Model-Akismet, incorrectly inherits from Catalyst::Component::InstancePerContext, should compose as a role, fix the module.
14
15   - CatalystX-Imports, Class::MOP doesn't consider anon subs in the symbol table as methods, tests + fix?
16
17   - Catalyst::Plugin::Cache::Curried
18   - Catalyst::Controller::WrapCGI
19     - Both import 'meta' into controller package - mst thinks this can be fixed?.
20       but @marcus> and the generated accessors even call $_[0]->meta, so I don't think we can remove it. 
21       . Add a test for this in core.
22       . Fix core to use CMOP to look up metaclass, rather than ->meta method, and
23         namespace::clean the meta method out, fix generated accessors to do the same?
24
25   - Catalyst::Plugin::SmartURI
26     - Core tests for behavior which causes it to fail (Caelum)
27
28   - Update Test suite to not assume MyApp ISA Controller
29     - 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?)
30   
31   - Comments marked /Moose TODO/i in the code
32   
33   - Eliminate all instances of $instance->{$key}
34
35   - Catalyst-Log-Log4perl - deep recursion in the test suite, investigate
36
37   - Profiling vs 5.70 and optimisation as needed.
38
39   - http://lists.scsys.co.uk/pipermail/catalyst-dev/2008-November/001546.html
40     - patch to list, andyg to look at?
41