r31695@martha (orig r1238): groditi | 2009-09-25 14:51:47 -0400
groditi [Tue, 15 Dec 2009 21:06:32 +0000 (21:06 +0000)]
 work arounds not necessary with newest MethodAttributes release (0.18)

Changes
Makefile.PL
lib/Reaction/UI/Controller/Collection.pm
lib/Reaction/UI/Controller/Collection/CRUD.pm

diff --git a/Changes b/Changes
index e2e42f2..bd40cd9 100644 (file)
--- a/Changes
+++ b/Changes
@@ -11,6 +11,8 @@ Revision history for Reaction
           - Add example of explicitly stating action to ComponentUI
         - Don't override custom location in push_viewport
         - Wire layout_args, which was forgotten + example of how to use it
+        - CRUD functionality is now implemented as roles, so it can be used 
+          without needing to use Controller::Collection::CRUD
 0.002000 - 29 Apr 2008
         - Update CheckUniques role to use around instead of overrides
         - Stop using ACCEPT_CONTEXT, use InstancePerContext instead
index da2cb84..d20d44a 100644 (file)
@@ -33,6 +33,7 @@ requires 'MooseX::Types' => '0.10';
 requires 'MooseX::Types::URI' => '0.02';
 requires 'MooseX::Types::Common' => '0.001000';
 requires 'MooseX::Types::DateTime' => '0.03';
+requires 'MooseX::MethodAttributes' => '0.18';
 requires 'Path::Class::Dir';
 requires 'Path::Class::File';
 requires 'Scalar::Util' => '1.19';
index d37f958..38cd61d 100644 (file)
@@ -8,7 +8,7 @@ use aliased 'Reaction::UI::ViewPort::Collection::Grid';
 __PACKAGE__->config(
   action => {
     list => { Chained => 'base', PathPart => '' },
-    object => { Chained => 'base', PathPart => 'id', CaptureArgs => 1, },
+    object => { Chained => 'base', PathPart => 'id' },
     view => { Chained => 'object', },
   },
 );
index c8d1147..bebfc75 100644 (file)
@@ -16,10 +16,6 @@ __PACKAGE__->config(
 
 with(
   'Reaction::UI::Controller::Role::Action::Create',
-);
-
-with(
-#  'Reaction::UI::Controller::Role::Action::Create',
   'Reaction::UI::Controller::Role::Action::Update',
   'Reaction::UI::Controller::Role::Action::Delete',
   'Reaction::UI::Controller::Role::Action::DeleteAll',