[catagits/Catalyst-Runtime.git] / TODO

TBD pre 5.8000_04 dev release:

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