[Jifty-commit] r5798 - in jifty/trunk: lib/Jifty lib/Jifty/Subs
Jifty commits
jifty-commit at lists.jifty.org
Sat Sep 6 06:04:20 EDT 2008
Author: sartak
Date: Sat Sep 6 06:04:16 2008
New Revision: 5798
Modified:
jifty/trunk/ (props changed)
jifty/trunk/Makefile.PL
jifty/trunk/lib/Jifty/Subs.pm
jifty/trunk/lib/Jifty/Subs/Render.pm
jifty/trunk/lib/Jifty/Test.pm
jifty/trunk/lib/Jifty/Web.pm
Log:
Modified: jifty/trunk/Makefile.PL
==============================================================================
--- jifty/trunk/Makefile.PL (original)
+++ jifty/trunk/Makefile.PL Sat Sep 6 06:04:16 2008
@@ -208,7 +208,6 @@
install_script('bin/jifty');
install_share;
-# ls plugins -> Authentication-CAS Multipage TabView
my @plugins = defined $ENV{JIFTY_PLUGINS} ? split(/ /, $ENV{JIFTY_PLUGINS}) : qw(TabView);
# Test all of our sub-dist tests too
@@ -219,7 +218,7 @@
die unless -d "plugins/$_";
# install_share("plugins/$_/share");
chdir("plugins/$_");
- system("$^X -I../../lib Makefile.PL INSTALLDIRS=".$ENV{INSTALLDIRS});
+ system("$^X -I../../lib Makefile.PL");
chdir('../..');
}
@@ -227,13 +226,8 @@
postamble(<<"END");
install ::
-@{[ join("\n", map { "\tcd plugins/$_;".' $(MAKE) install DESTDIR=$(DESTDIR) PREFIX=$(PREFIX)' } @plugins) ]}
+@{[ join("\n", map { "\tcd plugins/$_; make install" } @plugins) ]}
-distclean :: realclean
-@{[ "\t".'$(NOECHO) $(NOOP)']}
-
-realclean ::
-@{[ join("\n", map { "\tcd plugins/$_;".' $(MAKE) realclean' } @plugins) ]}
END
Modified: jifty/trunk/lib/Jifty/Subs.pm
==============================================================================
Modified: jifty/trunk/lib/Jifty/Subs/Render.pm
==============================================================================
--- jifty/trunk/lib/Jifty/Subs/Render.pm (original)
+++ jifty/trunk/lib/Jifty/Subs/Render.pm Sat Sep 6 06:04:16 2008
@@ -56,27 +56,6 @@
# XXX - We don't yet use $timestamp here.
my ( $timestamp, $msg ) = @$rv;
- for my $render_info (values %{$render->{$channel}}) {
- if ($render_info->{coalesce} and $render_info->{mode} eq "Replace") {
- my $hash = Digest::MD5::md5_hex( YAML::Dump([$render_info->{region}, $render_info->{render_with}] ) );
- Jifty->log->debug("Coalesced duplicate region @{[$render_info->{region}]} with @{[$render_info->{render_with}]} from $channel event $msg") if exists $coalesce{$hash};
- $coalesce{$hash} = [ $timestamp, $event_class, $msg, $render_info ] unless $coalesce{$hash} and $coalesce{$hash}[0] > $timestamp;
- } else {
- Jifty->log->debug("Rendering $channel event $msg in @{[$render_info->{region}]} with @{[$render_info->{render_with}]}");
- render_single( $event_class, $msg, $render_info, $callback );
- $sent++;
- }
- }
- }
- }
-
- for my $c (values %coalesce) {
- my (undef, $event_class, $msg, $render_info) = @{$c};
- Jifty->log->debug("Rendering @{[$render_info->{region}]} with @{[$render_info->{render_with}]} for $event_class");
- render_single( $event_class, $msg, $render_info, $callback );
- $sent++;
- }
-
return ($sent);
}
@@ -84,33 +63,30 @@
Renders a single region, based on the region information in C<INFO>.
-=cut
-
-sub render_single {
- my ($class, $msg, $render_info, $callback) = @_;
-
- my $region = Jifty::Web::PageRegion->new(
- name => $render_info->{region},
- path => $render_info->{render_with},
- );
- # So we don't warn about "duplicate region"s
- delete Jifty->web->{'regions'}{ $region->qualified_name };
-
- my $event_object = $class->new($msg);
- # Region's arguments come from explicit arguments only
- $region->arguments( $render_info->{arguments} );
-
- my $region_content = '';
- $region->enter;
- # Also provide an 'event' argument, and fill in the render
- # arguments. These don't show up in the region's arguments if
- # inspected, but do show up in the request.
- $region->render_as_subrequest(
- \$region_content,
- { %{$region->arguments}, event => $event_object, $event_object->render_arguments },
- );
- $callback->( $render_info->{mode}, $region->qualified_name, $region_content, $render_info->{attrs} );
- $region->exit;
+ for my $render_info (values %{$render->{$channel}}) {
+ my $region = Jifty::Web::PageRegion->new(
+ name => $render_info->{region},
+ path => $render_info->{render_with},
+ );
+ delete Jifty->web->{'regions'}{ $region->qualified_name };
+
+ # Finally render the region. In addition to the user-supplied arguments
+ # in $render_info, we always pass the target $region and the event object
+ # into its %ARGS.
+ my $region_content = '';
+ my $event_object = $event_class->new($msg);
+ $region->render_as_subrequest( \$region_content,
+ { %{ $render_info->{arguments} || {} },
+ event => $event_object,
+ $event_object->render_arguments,
+ }
+ );
+ $callback->( $render_info->{mode}, $region->qualified_name, $region_content);
+ $sent++;
+ }
+ }
+ }
+ return ($sent);
}
1;
Modified: jifty/trunk/lib/Jifty/Test.pm
==============================================================================
Modified: jifty/trunk/lib/Jifty/Web.pm
==============================================================================
More information about the Jifty-commit
mailing list