1 files changed, 67 insertions, 68 deletions
diff --git a/lib/Reaction/UI/ViewPort/Field/Mutable/ChooseMany.pm b/lib/Reaction/UI/ViewPort/Field/Mutable/ChooseMany.pm
index b3dca44..0833c73 100644
--- a/lib/Reaction/UI/ViewPort/Field/Mutable/ChooseMany.pm
+++ b/lib/Reaction/UI/ViewPort/Field/Mutable/ChooseMany.pm
@@ -7,77 +7,76 @@ my $listify = sub{
   return ref $_[0] eq 'ARRAY' ? $_[0] : [$_[0]];
 };
 
-class ChooseMany is 'Reaction::UI::ViewPort::Field', which {
-
-  does 'Reaction::UI::ViewPort::Field::Role::Mutable';
-  does 'Reaction::UI::ViewPort::Field::Role::Choices';
-
-  #MUST BE HERE, BELOW THE 'does', OR THE TRIGGER WILL NOT HAPPEN!
-  has '+value' => (isa => 'ArrayRef');
-
-  around value => sub {
-    my $orig = shift;
-    my $self = shift;
-    return $orig->($self) unless @_;
-    my $value = $listify->(shift);
-    $_ = $self->str_to_ident($_) for @$value;
-    my $checked = $self->attribute->check_valid_value($self->model, $value);
-    # i.e. fail if any of the values fail
-    confess "Not a valid set of values"
-      if (@$checked < @$value || grep { !defined($_) } @$checked);
-    $orig->($self, $checked);
-  };
-
-
-  around _value_string_from_value => sub {
-    my $orig = shift;
-    my $self = shift;
-    join(", ", (map {$self->obj_to_name($_->{value}) } @{ $self->current_value_choices }));
-  };
-
-  implements is_current_value => as {
-    my ($self, $check_value) = @_;
-    return unless $self->_model_has_value;
-    my @our_values = @{$self->value || []};
-    $check_value = $self->obj_to_str($check_value) if ref($check_value);
-    return grep { $self->obj_to_str($_) eq $check_value } @our_values;
-  };
-
-  implements current_value_choices => as {
-    my $self = shift;
-    my @all = grep { $self->is_current_value($_->{value}) } @{$self->value_choices};
-    return [ @all ];
-  };
-
-  implements available_value_choices => as {
-    my $self = shift;
-    my @all = grep { !$self->is_current_value($_->{value}) } @{$self->value_choices};
-    return [ @all ];
-  };
-
-  around handle_events => sub {
-    my $orig = shift;
-    my ($self, $events) = @_;
-    $events->{value} = [] if $events->{no_current_value};
-    my $ev_value = $listify->($events->{value});
-    if (delete $events->{add_all_values}) {
-      $events->{value} = [map {$self->obj_to_str($_)} @{$self->valid_values}];
-    } elsif (exists $events->{add_values} && delete $events->{do_add_values}) {
-      my $add = $listify->(delete $events->{add_values});
-      $events->{value} = [ @{$ev_value}, @$add ];
-    } elsif (delete $events->{remove_all_values}) {
-      $events->{value} = [];
-    }elsif (exists $events->{remove_values} && delete $events->{do_remove_values}) {
-      my $remove = $listify->(delete $events->{remove_values});
-      my %r = map { ($_ => 1) } @$remove;
-      $events->{value} = [ grep { !$r{$_} } @{$ev_value} ];
-    }
-
-    return $orig->(@_);
-  };
+use namespace::clean -except => [ qw(meta) ];
+extends 'Reaction::UI::ViewPort::Field';
+
+with 'Reaction::UI::ViewPort::Field::Role::Mutable';
+with 'Reaction::UI::ViewPort::Field::Role::Choices';
+
+#MUST BE HERE, BELOW THE 'does', OR THE TRIGGER WILL NOT HAPPEN!
+has '+value' => (isa => 'ArrayRef');
+
+around value => sub {
+  my $orig = shift;
+  my $self = shift;
+  return $orig->($self) unless @_;
+  my $value = $listify->(shift);
+  $_ = $self->str_to_ident($_) for @$value;
+  my $checked = $self->attribute->check_valid_value($self->model, $value);
+  # i.e. fail if any of the values fail
+  confess "Not a valid set of values"
+    if (@$checked < @$value || grep { !defined($_) } @$checked);
+  $orig->($self, $checked);
+};
+
 
+around _value_string_from_value => sub {
+  my $orig = shift;
+  my $self = shift;
+  join(", ", (map {$self->obj_to_name($_->{value}) } @{ $self->current_value_choices }));
+};
+sub is_current_value {
+  my ($self, $check_value) = @_;
+  return unless $self->_model_has_value;
+  my @our_values = @{$self->value || []};
+  $check_value = $self->obj_to_str($check_value) if ref($check_value);
+  return grep { $self->obj_to_str($_) eq $check_value } @our_values;
+};
+sub current_value_choices {
+  my $self = shift;
+  my @all = grep { $self->is_current_value($_->{value}) } @{$self->value_choices};
+  return [ @all ];
+};
+sub available_value_choices {
+  my $self = shift;
+  my @all = grep { !$self->is_current_value($_->{value}) } @{$self->value_choices};
+  return [ @all ];
 };
 
+around handle_events => sub {
+  my $orig = shift;
+  my ($self, $events) = @_;
+  $events->{value} = [] if $events->{no_current_value};
+  my $ev_value = $listify->($events->{value});
+  if (delete $events->{add_all_values}) {
+    $events->{value} = [map {$self->obj_to_str($_)} @{$self->valid_values}];
+  } elsif (exists $events->{add_values} && delete $events->{do_add_values}) {
+    my $add = $listify->(delete $events->{add_values});
+    $events->{value} = [ @{$ev_value}, @$add ];
+  } elsif (delete $events->{remove_all_values}) {
+    $events->{value} = [];
+  }elsif (exists $events->{remove_values} && delete $events->{do_remove_values}) {
+    my $remove = $listify->(delete $events->{remove_values});
+    my %r = map { ($_ => 1) } @$remove;
+    $events->{value} = [ grep { !$r{$_} } @{$ev_value} ];
+  }
+
+  return $orig->(@_);
+};
+
+__PACKAGE__->meta->make_immutable;
+
+
 1;
 
 =head1 NAME