Move all the public attributes in Catalyst::Dispatcher back to being private, remove...

[catagits/Catalyst-Runtime.git] / TODO

diff --git a/TODO b/TODO
index 9c225aa..41ce344 100644 (file)
--- a/TODO
+++ b/TODO
@@ -6,9 +6,18 @@ Pending patches:
   
   - 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:
 
+  - 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).
+
   - Common engine test failures, look into and get tests into core.
 
   - CatalystX-Imports, Class::MOP doesn't consider anon subs in the symbol 
@@ -18,6 +27,10 @@ Back-compat investigation / known issues:
   - 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
@@ -50,10 +63,6 @@ Back-compat investigation / known issues:
 
 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.
@@ -62,14 +71,12 @@ Cleanups:
     - 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 Catalyst::Request re {_body} (t0m)
-
   - 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 tests?) (t0m)
+    disk and fix it. (Believed to be in the optional TEST_HTTP tests?) (t0m)
 
 Documentation:
 
@@ -88,3 +95,5 @@ Profiling:
 Tests:
 
   - Moosified test application?
+
+  - Test warning from back-compat methods in Catalyst::Dispatcher