Permalink
Browse files

Add some FIXME and TODO comments.

  • Loading branch information...
1 parent e9d8e34 commit 7b40676d7689633f82709bbbe38d1d69706df48d @rafl rafl committed Feb 12, 2009
@@ -1,4 +1,6 @@
package CatalystX::Actions::Meta::Method;
+#TODO: I guess at some point this would become a Catalyst::Action or a role
+# that gets applied to it
use Moose;
@@ -13,7 +13,7 @@ has action_method_metaclass => (
default => 'CatalystX::Actions::Meta::Method',
);
-#FIXME
+#FIXME: BUILD apparently isn't the right thing to use in a role
use CatalystX::Actions::Meta::Method;
sub BUILD {
@@ -7,6 +7,9 @@ use namespace::clean -except => 'meta';
after register_actions => sub {
my ($self, $app) = @_;
+
+ #TODO: don't assume the action_methods method exists
+ #TODO: walk the superclasses to collect all actions
my @methods = find_meta($self)->action_methods;
for my $method (@methods) {

0 comments on commit 7b40676

Please sign in to comment.