cleanup of hard to read layout_args code
[catagits/Reaction.git] / lib / Reaction / UI / WidgetClass.pm
index 9e40936..f9c1b5a 100644 (file)
@@ -4,289 +4,225 @@ use Reaction::ClassExporter;
 use Reaction::Class;
 use Reaction::UI::Widget;
 use Data::Dumper;
+use Devel::Declare;
+use HTML::Entities ();
+use aliased 'Reaction::UI::WidgetClass::_OVER';
 
 no warnings 'once';
 
-class WidgetClass, which {
-
-  overrides exports_for_package => sub {
-    my ($self, $package) = @_;
-    return (super(),
-      func => sub {
-                my ($k, $m) = @_;
-                my $sig = "should be: func(data_key => 'method_name')";
-                confess "Data key not present, ${sig}" unless defined($k);
-                confess "Data key must be string, ${sig}" unless !ref($k);
-                confess "Method name not present, ${sig}" unless defined($m);
-                confess "Method name must be string, ${sig}" unless !ref($m);
-                [ $k, $m ];
-              }, # XXX zis is not ze grand design. OBSERVABLE.
-      string => sub (&) { -string => [ @_ ] }, # meh (maybe &;@ later?)
-      wrap => sub { $self->do_wrap_sub($package, @_); }, # should have class.
-    );
-  };
+use namespace::clean -except => [ qw(meta) ];
 
-  overrides default_base => sub { ('Reaction::UI::Widget') };
-
-  overrides do_class_sub => sub {
-    my ($self, $package, $class) = @_;
-    # intercepts 'foo renders ...'
-    local *renders::AUTOLOAD = sub {
-      our $AUTOLOAD;
-      shift;
-      $AUTOLOAD =~ /^renders::(.*)$/;
-      $self->do_renders_meth($package, $class, $1, @_);
-    };
-    # intercepts 'foo over ...'
-    local *over::AUTOLOAD = sub {
-      our $AUTOLOAD;
-      shift;
-      $AUTOLOAD =~ /^over::(.*)$/;
-      $self->do_over_meth($package, $class, $1, @_);
-    };
-    # $_ returns '-topic:_', $_{foo} returns '-topic:foo'
-    local $_ = '-topic:_';
-    my %topichash;
-    tie %topichash, 'Reaction::UI::WidgetClass::TopicHash';
-    local *_ = \%topichash;
-    super;
-  };
 
-  implements do_wrap_sub => as { confess "Unimplemented" };
-
-  implements do_renders_meth => as {
-    my ($self, $package, $class, $fname, $content, $args, $extra) = @_;
-
-    my $sig = 'should be: renders [ <content spec> ], \%args?';
-
-    confess "Too many args to renders, ${sig}" if defined($extra);
-    confess "First arg not an arrayref, ${sig}" unless ref($content) eq 'ARRAY';
-    confess "Args must be hashref, ${sig}"
-      if (defined($args) && (ref($args) ne 'HASH'));
-
-    $sig .= '
-  where content spec is [ fragment_name over func(...), \%args? ]
-  or [ qw(list of fragment names), \%args ]'; # explain the mistake, yea
-
-    my $inner_args = ((ref($content->[-1]) eq 'HASH') ? pop(@$content) : {});
-     # [ blah over func(...), { ... } ] or [ qw(foo bar), { ... } ]
-
-    # predeclare since content_gen gets populated somewhere in an if
-    # and inner_args_gen wants to be closed over by content_gen
-
-    my ($content_gen, $inner_args_gen);
-
-    my %args_extra; # again populated (possibly) within the if
-
-    confess "Content spec invalid, ${sig}"
-      unless defined($content->[0]) && !ref($content->[0]);
-
-    if (my ($key) = ($content->[0] =~ /^-(.*)?/)) {
-
-      # if first content value is -foo, pull it off the front and then
-      # figure out is it's a type we know how to handle
-
-      shift(@$content);
-      if ($key eq 'over') { # fragment_name over func
-        my ($fragment, $func) = @$content;
-        confess "Fragment name invalid, ${sig}" if ref($fragment);
-        my $content_meth = "render_${fragment}";
-        # grab result of func
-        # - if arrayref, render fragment per entry
-        # - if obj and can('next') call that until undef
-        # - else scream loudly
-        my ($func_key, $func_meth) = @$func;
-        $content_gen = sub {
-          my ($widget, $args) = @_;
-          my $topic = eval { $args->{$func_key}->$func_meth };
-          confess "Error calling ${func_meth} on ${func_key} argument "
-                 .($args->{$func_key}||'').": $@"
-            if $@;
-          my $iter_sub;
-          if (ref $topic eq 'ARRAY') {
-            my @copy = @$topic; # non-destructive on original data
-            $iter_sub = sub { shift(@copy); };
-          } elsif (Scalar::Util::blessed($topic) && $topic->can('next')) {
-            $iter_sub = sub { $topic->next };
-          } else {
-            #confess "func(${func_key} => ${func_meth}) for topic within fragment ${fname} did not return arrayref or iterator object";
-            # Coercing to a single-arg list instead for the mo. Mistake?
-            my @copy = ($topic);
-            $iter_sub = sub { shift(@copy); };
-          }
-          my $inner_args = $inner_args_gen->($args);
-          return sub {
-            my $next = $iter_sub->();
-            return undef unless $next;
-            return sub {
-              my ($rctx) = @_;
-              local $inner_args->{'_'} = $next; # ala local $_, why copy?
-              $widget->$content_meth($rctx, $inner_args);
-            };
-          };
-        };
-      } elsif ($key eq 'string') {
-
-        # string { ... }
-
-        my $sub = $content->[0]->[0]; # string {} returns (-string => [ $cr ])
-        $content_gen = sub {
-          my ($widget, $args) = @_;
-          my $done = 0;
-          my $inner_args = $inner_args_gen->($args);
-          return sub {
-            return if $done++; # a string content only happens once
-            return sub { # setup $_{foo} etc. and alias $_ to $_{_}
-              my ($rctx) = @_;
-              local *_ = \%{$inner_args};
-              local $_ = $inner_args->{'_'};
-              $sub->($rctx);
-            };
-          };
-        };
-
-      # must also handle just $_ later for wrap
-      } else {
-        # unrecognised -foo
-        confess "Unrecognised content spec type ${key}, ${sig}";
+# for local() for fragment wrap
+our ($next_call, $fragment_args, $current_widget, $do_render, $new_args);
+
+after 'do_import' => sub {
+  my ($self, $package) = @_;
+  Devel::Declare->install_declarator(
+    $package, 'fragment', DECLARE_NAME,
+    sub { },
+    sub {
+      WidgetClass->handle_fragment(@_);
+    }
+  );
+};
+
+after 'setup_and_cleanup' => sub {
+  my ($self, $package) = @_;
+  {
+    no strict 'refs';
+    delete ${"${package}::"}{'fragment'};
+  }
+  #Devel::Declare->teardown_for($package);
+};
+override exports_for_package => sub {
+  my ($self, $package) = @_;
+  return (super(),
+    over => sub {
+      my ($collection) = @_;
+      confess "too many args, should be: over \$collection" if @_ > 1;
+      _OVER->new(collection => $collection);
+    },
+    render => sub {
+      my ($name, $over) = @_;
+
+      my $sig = "should be: render 'name' or render 'name' => over \$coll";
+      if (!defined $name) { confess "name undefined: $sig"; }
+      if (ref $name) { confess "name is a ${\ref $name} ref, not a plain string: $sig"; }
+      if (defined $over && !(blessed($over) && $over->isa(_OVER))) {
+        confess "invalid args after name, $sig";
       }
-    } else {
-
-      # handling the renders [ qw(list of frag names), \%args ] case
-
-#warn @$content;
-      confess "Invalid content spec, ${sig}"
-        if grep { ref($_) } @$content;
-      $content_gen = sub {
-        my ($widget, $args) = @_;
-        my @fragment_methods = map { "render_${_}" } @$content;
-        my $inner_args = $inner_args_gen->($args);
-        return sub {
-          my $next = shift(@fragment_methods);
-          return undef unless $next;
-          return sub {
-            my ($rctx) = @_;
-            $widget->$next($rctx, $inner_args);
-          };
-        };
-      };
-
-      foreach my $key (@$content) {
-        my $frag_meth = "render_${key}";
-        $args_extra{$key} = sub {
-          my ($widget, $args) = @_;
-          my $inner_args = $inner_args_gen->($args);
-          return sub {
-            my ($rctx) = @_;
-            $widget->$frag_meth($rctx, $inner_args);
-          };
-        };
+      $do_render->($package, $current_widget, $name, $over);
+    },
+    arg => sub {
+      my ($name, $value) = @_;
+
+      my $sig = "should be: arg 'name' => \$value";
+      if (@_ < 2) {
+        $name ||= 'undef';
+        $value ||= 'undef';
+        confess "Not enough arguments, $sig, got: $name => $value";
       }
+      if (!defined $name) { confess "name undefined, $sig"; }
+      if (ref $name) { confess "name is not a string, $sig"; }
+
+      $new_args->{$name} = $value;
+    },
+    localized => sub {
+      my($value) = @_;
+      return $_{self}->view->localize($value);
+    },
+    call_next => sub {
+      confess "args passed, should be just call_next; or call_next();"
+        if @_;
+      $next_call->(@$fragment_args);
+    },
+    event_id => sub {
+      my ($name) = @_;
+      $_{viewport}->event_id_for($name);
+    },
+    event_uri => sub {
+      my ($events) = @_;
+      my $vp = $_{viewport};
+      my %args = map{ $vp->event_id_for($_) => $events->{$_} } keys %$events;
+      $vp->ctx->req->uri_with(\%args);
+    },
+    attrs => sub {
+      my ($attrs) = @_;
+      return join(' ', map {
+        my $text = HTML::Entities::encode_entities( $attrs->{$_} );
+        qq{$_="${text}"};
+      } keys %$attrs);
+    },
+    implements => sub {
+      my ($name, $sub) = @_;
+      $package->meta->add_method($name, $sub);
+    },
+  );
+};
+override default_base => sub { ('Reaction::UI::Widget') };
+sub handle_fragment {
+  my ($self, $name, $proto, $code) = @_;
+#warn ($self, $name, $code);
+  return ("_fragment_${name}" => $self->wrap_as_fragment($code));
+};
+sub wrap_as_fragment {
+  my ($self, $code) = @_;
+  return sub {
+    local $next_call;
+    if (ref $_[0] eq 'CODE') { # inside 'around' modifier
+      $next_call = shift;
     }
+    local $fragment_args = \@_;
+
+    # $self->$method($do_render, \%_, $new_args)
+    local $current_widget = $_[0];
+    local $do_render = $_[1];
+    local *_ = \%{$_[2]};
+    local $_ = $_[2]->{_};
+    local $new_args = $_[3];
+    $code->(@_);
+  };
+};
+
+__PACKAGE__->meta->make_immutable;
+
 
-    # populate both args generators here primarily for clarity
+1;
 
-    my $args_gen = $self->mk_args_generator($args);
-    $inner_args_gen = $self->mk_args_generator($inner_args);
+=head1 NAME
 
-    my $methname = "render_${fname}";
+Reaction::UI::WidgetClass - Create a new widget class
 
-    $args_extra{'_'} = $content_gen;
+=head1 DESCRIPTION
 
-    my @extra_keys = keys %args_extra;
-    my @extra_gen = values %args_extra;
+Turns the importing package into a widget class. It will export:
 
-    my $meth = sub {
-      my ($self, $rctx, $args) = @_;
-      confess "No rendering context passed" unless $rctx;
-      my $r_args = $args_gen->($args);
-#warn Dumper($r_args).' ';
-      @{$r_args}{@extra_keys} = map { $_->($self, $args); } @extra_gen;
-      $r_args->{'_'} = $content_gen->($self, $args);
-#warn Dumper($r_args).' ';
-      $rctx->render($fname, $r_args);
-    };
+=over 4
 
-    $class->meta->add_method($methname => $meth);
-  };
+=item All of L<Moose>
 
-  implements do_over_meth => as {
-    my ($self, $package, $class, @args) = @_;
-    #warn Dumper(\@args);
-    return (-over => @args);
-  };
+=item All of L<Reaction::Class>
 
-  implements mk_args_generator => as {
-    my ($self, $argspec) = @_;
-#warn Dumper($argspec);
-    # only handling [ $k, $v ] (func()) and -topic:$x ($_{$x}) for the moment
-
-    my $sig = 'should be: key => $_ or key => $_{name} or key => func("name", "method")';
-
-    my (@func_to, @func_spec, @copy_from, @copy_to, @sub_spec, @sub_to);
-    foreach my $key (keys %$argspec) {
-      my $val = $argspec->{$key};
-      if (ref($val) eq 'ARRAY') {
-        push(@func_spec, $val);
-        push(@func_to, $key);
-      } elsif (!ref($val) && ($val =~ /^-topic:(.*)$/)) {
-        my $topic_key = $1;
-        push(@copy_from, $topic_key);
-        push(@copy_to, $key);
-      }  elsif (ref($val) eq 'CODE') {
-      #LOOK AT ME
-        my $sub = sub{
-          my $inner_args = shift;
-          local *_ = \%{$inner_args};
-          local $_ = $inner_args->{'_'};
-          return $val->();
-        };
-        push(@sub_spec, $sub);
-        push(@sub_to, $key);
-      } else {
-        confess "Invalid args member for ${key}, ${sig}";
-      }
-    }
-#warn Dumper(\@func_to, \@func_spec, \@copy_from, \@copy_to);
-    return sub {
-      my ($outer_args) = @_;
-      my $args = { %$outer_args };
-#warn Dumper(\@func_to, \@func_spec, \@copy_from, \@copy_to).' ';
-      @{$args}{@copy_to} = @{$outer_args}{@copy_from};
-      @{$args}{@func_to} = (map {
-        my ($key, $meth) = @{$_};
-        $outer_args->{$key}->$meth; # [ 'a, 'b' ] ~~ ->{'a'}->b
-      } @func_spec);
-      #LOOK AT ME
-      @{$args}{@sub_to} = (map { $_->($outer_args) } @sub_spec);
-#warn Dumper($args).' ';
-      return $args;
-    };
-  };
+=item L<strict> and L<warnings>
 
-};
+=item See L</EXPORTS> for this package's own exports
 
-1;
+=back
 
-package Reaction::UI::WidgetClass::TopicHash;
+It will also set the value of C<default_base> as new superclass. The default is
+C<Reaction::UI::Widget>.
 
-use Tie::Hash;
-use base qw(Tie::StdHash);
+=head1 EXPORTS
 
-sub FETCH {
-  my ($self, $key) = @_;
-  return "-topic:${key}";
-}
+=head2 over
 
-1;
+  over $collection
 
-__END__;
+Used in combination with L</render> to render a fragment for a series of values:
 
-=head1 NAME
+  render fragment_name => over [1, 2, 3];
 
-Reaction::UI::WidgetClass
+=head2 render
 
-=head1 DESCRIPTION
+  render $fragment_name;
+  render $fragment_name, $over;
+
+With only the fragment name as argument, it renders that fragment. If an C<$over>
+collection is specified with the L</over> keyword, the fragment is rendered once
+for every value in the collection. The value will be accessible in the topic
+argument C<_>.
+
+=head2 arg
+
+  arg $arg_name, $arg_value;
+
+Sets the fragment argument C<$arg_name> to C<$arg_value>;
+
+=head2 localized
+
+  localize $value;
+
+Calls the view's C<localize> method to localize the passed value.
+
+=head2 call_next
+
+  call_next;
+
+Calls the parent fragment.
+
+=head2 event_id
+
+  event_id $event_name;
+
+Fetches the event id for the event C<$event_name> from the viewport via its C<event_id_for>
+method.
+
+=head2 event_uri
+
+  event_uri \%events;
+
+Returns an L<URI> object with the event ids corresponding to the keys in the C<%events> 
+argument and the values being the values of the hash reference.
+
+=head2 attrs
+
+  attrs \%attrs;
+
+Builds a string of rendered element attributes out of the C<%attrs> hash reference argument.
+
+=head2 implements
+
+  implements fragment foo { ... };
+  implements bar => sub { ... };
+
+Implements a method or a fragment in the widget class.
+
+=head2 fragment
+
+  fragment foo { ... };
+
+Creates a new fragment named C<foo> with a implementation in the block.
 
 =head1 AUTHORS