FIXME's and comments

[catagits/Catalyst-Runtime.git] / TODO

diff --git a/TODO b/TODO
index 2c419a7..8490d57 100644 (file)
--- a/TODO
+++ b/TODO
@@ -57,6 +57,25 @@ http://github.com/willert/catalyst-plugin-log4perl-simple/tree
 
 ## GSOC
 
+### Next steps - planned:
+
+  - Test custom container
+    - writing some tests which verify that the models you think should be
+      there are there, and that they received their dependencies as arguments
+    - i.e. Model::Bar should get params { foo => $model_foo } when being
+      constructed, etc
+    - Need to test that if you have a standard component Frotz
+      and a customized component Fnar, and Fnar depends on Frotz
+    - And yeah, some tests that the customised components actually work via
+      $c->model('Foo'), and that COMPONENT is called (or not called)
+      as appropiate and that ACCEPT_CONTEXT is called (or not) as appropriate
+
+  - Kill flush_request_services
+    - we're already passing $ctx in when we locate services so the ::Request
+      lifecycle can just stash the object in, or retrieve the object from the
+      stash
+      (actually, we're not passing $ctx when we locate services. Problem!)
+
 ### Next steps - less planned:
 
   - Creating service()-like sugar for component
@@ -64,6 +83,8 @@ http://github.com/willert/catalyst-plugin-log4perl-simple/tree
   - Test cases for extending the container in an application.
     - Using the sugar added in the previous item
     - Test when Model::Foo depends_on Model::Bar
+    - Test for component Foo => ( lifecycle => 'Singleton', class => 'My::External::Class', dependencies => { config => depends_on("config") } )
+    - Fix ^^ so that you can get your component's namespaced config nicely.
 
   - Tests for using the container outside of Catalyst
     - Custom container which adds some (very simple) services which are initialized from
@@ -82,12 +103,16 @@ and what needs to be done to fix it!
 package MyApp::Container;
 use Catalyst::IOC;
     
-    container $self, as { 
+    container $self, as {
             container model => as {
                 component Foo => (); # As per default!
                 component Bar => (dependencies => ['/model/Foo']); # Magic!
                 component Baz => ( lifecycle => 'InstancePerContext );
                 component Quux => ( lifecycle => 'Singleton' ); # ACCEPT_CONTEXT not called
+                # Catalyst::Model::Adaptor example
+                conponent Fnar => ( lifecycle => 'Singleton', class => 'My::External::Class', dependencies => { config => depends_on('config')} );
+                #                                                                                               ^^ FIXME - gets whole config, not Model::Foo
+                #                                                                                                  There should be a 'nice' way to get the 'standard' config
             };                    
             # Note - implementation of BB may need to be changed to support making sure existing 
             # services actually get overridden. not sure how the default container behaves when doing that