on second thought, make all empty specs return a lexical sort
[catagits/Reaction.git] / lib / Reaction / UI / ViewPort / Action.pm
index 8ec12e4..6a2ac58 100644 (file)
@@ -2,114 +2,49 @@ package Reaction::UI::ViewPort::Action;
 
 use Reaction::Class;
 
-use aliased 'Reaction::UI::ViewPort::Object';
-
-BEGIN { *DEBUG_EVENTS = \&Reaction::UI::ViewPort::DEBUG_EVENTS; }
-
-use aliased 'Reaction::UI::ViewPort::Field::Mutable::Text';
-use aliased 'Reaction::UI::ViewPort::Field::Mutable::Array';
-use aliased 'Reaction::UI::ViewPort::Field::Mutable::String';
-use aliased 'Reaction::UI::ViewPort::Field::Mutable::Number';
-use aliased 'Reaction::UI::ViewPort::Field::Mutable::Integer';
-use aliased 'Reaction::UI::ViewPort::Field::Mutable::Boolean';
-use aliased 'Reaction::UI::ViewPort::Field::Mutable::Password';
-use aliased 'Reaction::UI::ViewPort::Field::Mutable::DateTime';
-use aliased 'Reaction::UI::ViewPort::Field::Mutable::ChooseOne';
-use aliased 'Reaction::UI::ViewPort::Field::Mutable::ChooseMany';
-
-use aliased 'Reaction::UI::ViewPort::Field::Mutable::File';
-#use aliased 'Reaction::UI::ViewPort::Field::Mutable::TimeRange';
-
+use MooseX::Types::Moose qw/Int/;
 use Reaction::Types::Core qw/NonEmptySimpleStr/;
 
 use namespace::clean -except => [ qw(meta) ];
-extends Object;
 
+extends 'Reaction::UI::ViewPort::Object::Mutable';
+with 'Reaction::UI::ViewPort::Action::Role::OK';
 
-has model  => (is => 'ro', isa => 'Reaction::InterfaceModel::Action', required => 1);
-#has '+model' => (isa => 'Reaction::InterfaceModel::Action');
-has method => ( isa => NonEmptySimpleStr, is => 'rw', default => sub { 'post' } );
+#this has to fucking go. it BLOWS.
+has method => (
+  is => 'rw',
+  isa => NonEmptySimpleStr,
+  default => sub { 'post' }
+);
 
-has on_apply_callback => (is => 'rw', isa => 'CodeRef');
-has on_close_callback => (is => 'rw', isa => 'CodeRef');
+has changed => (
+  is => 'rw',
+  isa => Int,
+  reader => 'is_changed',
+  default => sub{0}
+);
 
-has ok_label           => (is => 'rw', isa => 'Str', lazy_build => 1);
-has apply_label        => (is => 'rw', isa => 'Str', lazy_build => 1);
-has close_label        => (is => 'rw', isa => 'Str', lazy_fail  => 1);
-has close_label_close  => (is => 'rw', isa => 'Str', lazy_build => 1);
-has close_label_cancel => (is => 'rw', isa => 'Str', lazy_build => 1);
-
-has changed => (is => 'rw', isa => 'Int', reader => 'is_changed', default => sub{0});
-sub BUILD {
-  my $self = shift;
-  $self->close_label($self->close_label_close);
-};
-sub _build_ok_label { 'ok'     };
-sub _build_apply_label { 'apply'  };
-sub _build_close_label_close { 'close'  };
-sub _build_close_label_cancel { 'cancel' };
 sub can_apply {
   my ($self) = @_;
   foreach my $field ( @{ $self->fields } ) {
-    if ($field->needs_sync) {
-      if (DEBUG_EVENTS) {
-        $self->ctx->log->debug(
-          "Failing out of can_apply on ${\ref($self)} at ${\$self->location}"
-          ." because field for ${\$field->attribute->name} needs sync"
-        );
-      }
-      return 0;
-    }
+    return 0 if $field->needs_sync;
     # if e.g. a datetime field has an invalid value that can't be re-assembled
     # into a datetime object, the action may be in a consistent state but
     # not synchronized from the fields; in this case, we must not apply
   }
-  if (DEBUG_EVENTS) {
-    my $ret = $self->model->can_apply;
-    $self->ctx->log->debug(
-      "model can_apply returned ${ret}"
-      ." on ${\ref($self)} at ${\$self->location}"
-    );
-    return $ret;
-  }
   return $self->model->can_apply;
-};
+}
+
 sub do_apply {
   shift->model->do_apply;
-};
-sub ok {
-  my $self = shift;
-  $self->close(@_) if $self->apply(@_);
-};
-sub apply {
-  my $self = shift;
-  if ($self->can_apply && (my $result = $self->do_apply)) {
-    $self->changed(0);
-    $self->close_label($self->close_label_close);
-    $self->on_apply_callback->($self => $result) if $self->has_on_apply_callback;
-    return 1;
-  } else {
-    $self->changed(1);
-    $self->close_label($self->close_label_cancel);
-    return 0;
-  }
-};
-sub close {
-  my $self = shift;
-  return unless $self->has_on_close_callback;
-  $self->on_close_callback->($self);
-};
-sub can_close { 1 };
-
-override accept_events => sub {
-  (($_[0]->has_on_close_callback ? ('ok', 'close') : ()), 'apply', super());
-}; # can't do a close-type operation if there's nowhere to go afterwards
+}
 
 after apply_child_events => sub {
   # interrupt here because fields will have been updated
   my ($self) = @_;
   $self->sync_action_from_fields;
 };
+
 sub sync_action_from_fields {
   my ($self) = @_;
   foreach my $field (@{$self->fields}) {
@@ -119,77 +54,14 @@ sub sync_action_from_fields {
   foreach my $field (@{$self->fields}) {
     $field->sync_from_action; # get errors from $action if applicable
   }
-};
-sub _build_fields_for_type_Num {
-  my ($self, $attr, $args) = @_;
-  $self->_build_simple_field(attribute => $attr, class => Number, %$args);
-};
-sub _build_fields_for_type_Int {
-  my ($self, $attr, $args) = @_;
-  $self->_build_simple_field(attribute => $attr, class => Integer, %$args);
-};
-sub _build_fields_for_type_Bool {
-  my ($self,  $attr, $args) = @_;
-  $self->_build_simple_field(attribute => $attr, class => Boolean, %$args);
-};
-sub _build_fields_for_type_Reaction_Types_Core_SimpleStr {
-  my ($self, $attr, $args) = @_;
-  $self->_build_simple_field(attribute => $attr, class => String, %$args);
-};
-sub _build_fields_for_type_Reaction_Types_File_File {
-  my ($self, $attr, $args) = @_;
-  $self->_build_simple_field(attribute => $attr, class => File, %$args);
-};
-sub _build_fields_for_type_Str {
-  my ($self, $attr, $args) = @_;
-  if ($attr->has_valid_values) { # There's probably a better way to do this
-    $self->_build_simple_field(attribute => $attr, class => ChooseOne, %$args);
-  } else {
-    $self->_build_simple_field(attribute => $attr, class => Text, %$args);
-  }
-};
-sub _build_fields_for_type_Reaction_Types_Core_Password {
-  my ($self, $attr, $args) = @_;
-  $self->_build_simple_field(attribute => $attr, class => Password, %$args);
-};
-sub _build_fields_for_type_Reaction_Types_DateTime_DateTime {
-  my ($self, $attr, $args) = @_;
-  $self->_build_simple_field(attribute => $attr, class => DateTime, %$args);
-};
-sub _build_fields_for_type_Enum {
-  my ($self, $attr, $args) = @_;
-    $self->_build_simple_field(attribute => $attr, class => ChooseOne, %$args);
-};
+}
 
-#this needs to be fixed. somehow. beats the shit our of me. really.
-#implements build_fields_for_type_Reaction_InterfaceModel_Object => as {
-sub _build_fields_for_type_DBIx_Class_Row {
-  my ($self, $attr, $args) = @_;
-  $self->_build_simple_field(attribute => $attr, class => ChooseOne, %$args);
-};
-sub _build_fields_for_type_ArrayRef {
-  my ($self, $attr, $args) = @_;
-  if ($attr->has_valid_values) {
-    $self->_build_simple_field(attribute => $attr, class => ChooseMany,  %$args);
-  } else {
-    $self->_build_simple_field
-      (
-       attribute => $attr,
-       class     => Array,
-       layout    => 'field/mutable/hidden_array',
-       %$args);
-  }
-};
-
-#implements _build_fields_for_type_DateTime_Spanset => as {
-#  my ($self, $attr, $args) = @_;
-#    $self->_build_simple_field(attribute => $attr, class => TimeRange,  %$args);
-#};
 
 __PACKAGE__->meta->make_immutable;
 
+1;
 
-  1;
+__END__;
 
 =head1 NAME
 
@@ -197,84 +69,42 @@ Reaction::UI::ViewPort::Action
 
 =head1 SYNOPSIS
 
-  use aliased 'Reaction::UI::ViewPort::Action';
-
-  $self->push_viewport(Action,
-    layout => 'register',
-    model => $action,
-    next_action => [ $self, 'redirect_to', 'accounts', $c->req->captures ],
-    ctx => $c,
-    field_order => [
-      qw / contact_title company_name email address1 address2 address3
-           city country post_code telephone mobile fax/ ],
-  );
-
 =head1 DESCRIPTION
 
-This subclass of L<Reaction::UI::ViewPort::Object> is used for rendering a
-collection of C<Reaction::UI::ViewPort::Field::Mutable::*> objects for user editing.
+This subclass of L<Reaction::UI::ViewPort::Object::Mutable> is used for 
+rendering a complete form supporting Apply, Close and OK.
 
 =head1 ATTRIBUTES
 
-=head2 model
-
-L<Reaction::InterfaceModel::Action>
-
-=head2 ok_label
-
-Default: 'ok'
-
-=head2 apply_label
-
-Default: 'apply'
-
-=head2 close_label_close
-
-Default: 'close'
-
-=head2 close_label_cancel
-
-This label is only shown when C<changed> is true.
-
-Default: 'cancel'
+=head2 method
 
-=head2 fields
-
-=head2 can_apply
-
-=head2 can_close
+post / get
 
 =head2 changed
 
 Returns true if a field has been edited.
 
-=head2 next_action
-
-=head2 on_apply_callback
-
-CodeRef.
-
 =head1 METHODS
 
-=head2 ok
+=head2 can_apply
 
-Calls C<apply>, and then C<close> if successful.
+=head2 do_apply
 
-=head2 close
+=head2 sync_action_from_fields
 
-Pop viewport and proceed to C<next_action>.
+=head1 SEE ALSO
 
-=head2 apply
+L<Reaction::UI::ViewPort>
 
-Attempt to save changes and update C<changed> attribute if required.
+L<Reaction::UI::ViewPort::Object>
 
-=head1 SEE ALSO
+L<Reaction::UI::ViewPort::Object::Mutable>
 
-L<Reaction::UI::ViewPort::Object>
+L<Reaction::InterfaceModel::Action::Role::Apply>
 
-L<Reaction::UI::ViewPort>
+L<Reaction::InterfaceModel::Action::Role::Close>
 
-L<Reaction::InterfaceModel::Action>
+L<Reaction::InterfaceModel::Action::Role::OK>
 
 =head1 AUTHORS
 
@@ -285,3 +115,4 @@ See L<Reaction::Class> for authors.
 See L<Reaction::Class> for the license.
 
 =cut
+