double negation is hard to read

diff --git a/lib/Reaction/UI/ViewPort/Field/Role/Mutable.pm b/lib/Reaction/UI/ViewPort/Field/Role/Mutable.pm
index 0acac38..f1704ad 100644 (file)
--- a/lib/Reaction/UI/ViewPort/Field/Role/Mutable.pm
+++ b/lib/Reaction/UI/ViewPort/Field/Role/Mutable.pm
@@ -95,7 +95,7 @@ sub sync_to_action {
 };
 sub sync_from_action {
   my ($self) = @_;
-  return unless !$self->needs_sync; # && $self->has_attribute;
+  return if $self->needs_sync;
   if( !$self->has_message ){
     if(my $error = $self->model->error_for($self->attribute) ){
       $self->message( $error );