X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?p=catagits%2FCatalyst-Runtime.git;a=blobdiff_plain;f=TODO;h=7d3486343290ad54d89c890a0ac50fa2f9662203;hp=41ce344070eada7f3c85186e28c9134867fb0b56;hb=79f5d5718d885ee1c3f288159a31562f9cf5b02f;hpb=c41cfce33f5510dee4937dfaaff2fd331130a84b

diff --git a/TODO b/TODO
index 41ce344..7d34863 100644
--- a/TODO
+++ b/TODO
@@ -1,99 +1,36 @@
-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)
-    
-  - Double-applying mk_accessors breaks, t/double_apply.t for MX::Emulate::CAF
-  
-Back-compat investigation / known issues:
+Known Bugs:
 
-  - Plugins with new methods.
-    - Moose 'not inlining new method' warnings need to be fixed.
-      - Warning in Moose could also be more helpful..
-    - Why does the current test not fail / warn?
-    - Generally unhappy with the on_end_of_scope immutable approach,
-      try removing new method from plugins (and emitting warning).
+   - Bug ->go or ->visit causes actions which have Args or CaptureArgs called
+     twice when called via ->go or ->visit.
 
-  - Common engine test failures, look into and get tests into core.
+     Test app: http://github.com/bobtfish/catalyst-app-bug-go_chain/tree/master
 
-  - CatalystX-Imports, Class::MOP doesn't consider anon subs in the symbol 
-    table as methods, tests + fix, or explanation and documentation?
-    (rafl & phaylon)
+Compatibility warnings to add:
 
-  - 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.
+   - $self->config should warn as config should only ever be called as a
+     class method.
 
-  - Issues with TWMC not being loaded when it used to be in 5.70 
-    (Bill Moseley)
+Proposed functionality / feature additions:
 
-  - Fix memory leaks (I already tried Devel::Leak::Object, but no joy).
+    - 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.
 
-  -  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 :/
+      Simple example: Catalyst::Plugin::Log::Colorful should just be a
+      subclass of Catalyst::Log, no ::Plugin:: needed.
 
-Cleanups:
+      See also: Catalyst::Plugin::Log::Dispatch and
+      http://github.com/willert/catalyst-plugin-log4perl-simple/tree
 
-  - 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?)
+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)
+  * Catalyst::Controller::_parse_ActionClass_attr  # DONE
+  * Catalyst::Dispatcher::_load_dispatch_types     # DONE
+  * Catalyst::setup_plugins                        # DONE
+  to use the same namespacing method
 
-Documentation:
-
-   - extends in components must be inside a BEGIN block.
-
-   - How to write Moosified Catalyst components.
-
-   - Manual / Tutorial updates
-
-   - Fix the Roadmap to be less full of lies.
-
-Profiling:
-
-  - vs 5.70 and optimisation as needed.
-
-Tests:
-
-  - Moosified test application?
-
-  - Test warning from back-compat methods in Catalyst::Dispatcher
+- Ok, so can you add tests for ->config(actions => { foo => { ActionClass => '+Bar' }});