Detect redispatch exceptions by a class check, not by checking the exception message.

[catagits/Catalyst-Runtime.git] / TODO

diff --git a/TODO b/TODO
index 9e655ed..7d34863 100644 (file)
--- a/TODO
+++ b/TODO
@@ -1,35 +1,10 @@
 Known Bugs:
 
-   - Bug ->go or ->visit causes actions which have Args or CaptureArgs caled
+   - Bug ->go or ->visit causes actions which have Args or CaptureArgs called
      twice when called via ->go or ->visit.
 
      Test app: http://github.com/bobtfish/catalyst-app-bug-go_chain/tree/master
 
-
-   - Certain errors in your application to do with using other code which does
-     not exist can cause an 'Unknown error' issue.
-
-     This is now fixed, by moving back to B::Hooks::EndOfScope, but this (used
-     to) cause issues with the following modules:
-
-     - 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
-
-     according to comments in r8991. Retest these before shipping anything..
-
-   - -restarter.t will fail if you don't have ::StashChange installed, and
-      also quite often at other times.
-
-
-   - Attribute called 'actions' in controllers - needs to be renamed to _actions.
-
-   - before finalize causes you to miserably fail to call plugins finalize
-     method (dmaki)
-
 Compatibility warnings to add:
 
    - $self->config should warn as config should only ever be called as a
@@ -47,4 +22,15 @@ Proposed functionality / feature additions:
       See also: Catalyst::Plugin::Log::Dispatch and
       http://github.com/willert/catalyst-plugin-log4perl-simple/tree
 
+TODO for brach namespace_handling_refactor:
+
+- refactor code in
+  * Catalyst::Dispatcher::get_containers           # No Idea
+  * Catalyst::Dispatcher::dispatch_type            # DONE
+
+  * Catalyst::Controller::_parse_ActionClass_attr  # DONE
+  * Catalyst::Dispatcher::_load_dispatch_types     # DONE
+  * Catalyst::setup_plugins                        # DONE
+  to use the same namespacing method
 
+- Ok, so can you add tests for ->config(actions => { foo => { ActionClass => '+Bar' }});