X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=TODO;h=842e74c9463ef87c56762fc2c1232ddc7dfa3ac3;hb=ca35f958d7ba5bdb610a2d9ea3a0050e7b6c8ba8;hp=2db1ad6cd737b5bbf277390830b6f7461d537e1c;hpb=878cfbf2ec360b01decfe922d58e5091fe070ea5;p=catagits%2FCatalyst-Runtime.git diff --git a/TODO b/TODO index 2db1ad6..842e74c 100644 --- a/TODO +++ b/TODO @@ -24,14 +24,40 @@ subclass of Catalyst::Log, no ::Plugin:: needed. See also: Catalyst::Plugin::Log::Dispatch and http://github.com/willert/catalyst-plugin-log4perl-simple/tree -# REFACTORING +## Capture arguments that the plack engine component was run with somewhere, + to more easily support custom args from scripts (e.g. Gitalist's + --git_dir) -## The horrible hack for plugin setup - replacing it: +## throw away the restarter and allow using the restarters Plack provides + +## remove per-request state from the engine instance + +## be smarter about how we use PSGI - not every response needs to be delayed + and streaming + +# The horrible hack for plugin setup - replacing it: * Have a look at the Devel::REPL BEFORE_PLUGIN stuff I wonder if what we need is that combined with plugins-as-roles -## App / ctx split: +# PSGI + +## To do at release time + + - Release psgi branch of Catalyst-Devel + - Release new Task::Catalyst + - Release 5.9 branch of Catalyst-Manual + - Release Catalyst::Engine::HTTP::Prefork with deprecation notice + + exit in Makefile.PL if Catalyst > 5.89 is installed. + +## Blockers + + * I've noticed a small difference with Catalyst::Test. The latest stable + version include two headers, 'host' and 'https'. They are missing from + this version - Pedro Melo on list + ^^ Cannot replicate this? Mailed back to ask for tests.. + +# App / ctx split: NOTE - these are notes that t0m thought up after doing back compat for catalyst_component_class, may be inaccurate, wrong or missing things @@ -75,12 +101,8 @@ http://github.com/willert/catalyst-plugin-log4perl-simple/tree - expand_component_module - - loading two containers simultaneously - - - sugar is still not completely implemented - - Some back compat - - wrappers around setup_component, setup_components, locate_components in Catalyst.pm + - wrappers around setup_component, setup_components in Catalyst.pm - $instance->expand_modules - search_extra - Crazy tests for things such as: @@ -95,6 +117,7 @@ http://github.com/willert/catalyst-plugin-log4perl-simple/tree - per request life cycle - sugar - we should discuss the syntax with rafl and edenc + - what's missing? - when / when not COMPONENT should be called @@ -108,15 +131,10 @@ http://github.com/willert/catalyst-plugin-log4perl-simple/tree ### Next steps - planned: - some imports need to get the importing package in Catalyst::IOC - - - test values directly instead of serializing them over http in t/lib/TestCustomContainer.pm - - - 16:01 < shadowpaste> "t0m" at 217.168.150.38 pasted "more explicit? thoughts?" (13 lines) at http://paste.scsys.co.uk/131520 - - - make ACCEPT_CONTEXT and COMPONENT optional in Catalyst::IOC::BlockInjection and Catalyst::IOC::ConstructorInjection - - Create COMPONENTSingleton life cycle + - done - needs testing - Back compat for Catalyst.pm moved methods (locate_components) + - done - needs testing - Test custom container - writing some tests which verify that the models you think should be @@ -131,7 +149,8 @@ http://github.com/willert/catalyst-plugin-log4perl-simple/tree ### Next steps - less planned: - - Creating service()-like sugar for component + - make ACCEPT_CONTEXT and COMPONENT optional in Catalyst::IOC::BlockInjection and Catalyst::IOC::ConstructorInjection + - Create COMPONENTSingleton life cycle - Test cases for extending the container in an application. - Using the sugar added in the previous item