[catagits/Catalyst-Runtime.git] / TODO

  - Class::C3::Adopt::NEXT - some people are getting more than one warning
    per package, make a test case for this (t0m)

  - Catalyst::Plugin::Authentication fails tests. Believed to be as
    Catalyst::Plugin::Authentication::Credential::Password has a
    ->mk_accessors('_config'), which is also class data.
    Make a test case for this (t0m)

  - Looks like stash is not available during prepare_path when running under
    Catalyst::Test.. Run MojoMojo's t/01app.t and it will explode with
    "Can't use an undefined value as a HASH reference" - test case for core
    + fix maybe (marcus)

  - Common engine test failures, look into and get tests into core.

  - 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.

  - CatalystX-Imports, Class::MOP doesn't consider anon subs in the symbol table as methods, tests + fix?

  - Catalyst::Plugin::Cache::Curried
  - Catalyst::Controller::WrapCGI
    - Both import 'meta' into controller package - mst thinks this can be fixed?.
      but @marcus> and the generated accessors even call $_[0]->meta, so I don't think we can remove it. 
      . Add a test for this in core.
      . Fix core to use CMOP to look up metaclass, rather than ->meta method, and
        namespace::clean the meta method out, fix generated accessors to do the same?

  - MooseX::Emulate::Class::Accessor::Fast
    - Tests for uses of C::A::F from Catalyst repos. - t0m
    - New release once these are done.

  - 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?)
  
  - Comments marked /Moose TODO/i in the code
  
  - Eliminate all instances of $instance->{$key}

  - Catalyst-Log-Log4perl - deep recursion in the test suite, investigate

  - Profiling vs 5.70 and optimisation as needed.

  - http://lists.scsys.co.uk/pipermail/catalyst-dev/2008-November/001546.html
    - patch to list, andyg to look at?
Commit	Line	Data
cec7aff6	1	- Class::C3::Adopt::NEXT - some people are getting more than one warning
cec7aff6	2	per package, make a test case for this (t0m)
2105b371	3
cec7aff6	4	- Catalyst::Plugin::Authentication fails tests. Believed to be as
	5	Catalyst::Plugin::Authentication::Credential::Password has a
	6	->mk_accessors('_config'), which is also class data.
	7	Make a test case for this (t0m)
dae2b0fa	8
	9	- Looks like stash is not available during prepare_path when running under
	10	Catalyst::Test.. Run MojoMojo's t/01app.t and it will explode with
	11	"Can't use an undefined value as a HASH reference" - test case for core
	12	+ fix maybe (marcus)
	13
2105b371	14	- Common engine test failures, look into and get tests into core.
	15
	16	- Catalyst-Plugin-Authorization-ACL, Can't locate object method "tree" via package "Catalyst::Dispatcher", fix the plugin as tree was never a public method.
	17
	18	- Catalyst-Model-Akismet, incorrectly inherits from Catalyst::Component::InstancePerContext, should compose as a role, fix the module.
	19
2105b371	20	- CatalystX-Imports, Class::MOP doesn't consider anon subs in the symbol table as methods, tests + fix?
2105b371	21
8610de43	22	- Catalyst::Plugin::Cache::Curried
	23	- Catalyst::Controller::WrapCGI
	24	- Both import 'meta' into controller package - mst thinks this can be fixed?.
	25	but @marcus> and the generated accessors even call $_[0]->meta, so I don't think we can remove it.
	26	. Add a test for this in core.
	27	. Fix core to use CMOP to look up metaclass, rather than ->meta method, and
	28	namespace::clean the meta method out, fix generated accessors to do the same?
	29
dae2b0fa	30	- MooseX::Emulate::Class::Accessor::Fast
	31	- Tests for uses of C::A::F from Catalyst repos. - t0m
	32	- New release once these are done.
8610de43	33
2105b371	34	- Update Test suite to not assume MyApp ISA Controller
	35	- 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?)
	36
	37	- Comments marked /Moose TODO/i in the code
	38
	39	- Eliminate all instances of $instance->{$key}
	40
	41	- Catalyst-Log-Log4perl - deep recursion in the test suite, investigate
	42
	43	- Profiling vs 5.70 and optimisation as needed.
	44
8610de43	45	- http://lists.scsys.co.uk/pipermail/catalyst-dev/2008-November/001546.html
	46	- patch to list, andyg to look at?
	47