X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FCatalyst%2FView%2FTT.pm;h=f4d5762cf4164485d0566222b35936e06a6f0c10;hb=fe3df9dcf23365f7d3e685d128042393703a3ad7;hp=d5dde6ef52a4e05f2a0d7fbfecf19a2d19cb69b9;hpb=ad50568f7d4bf0ef2e0c78e599eb7c6ebf037890;p=catagits%2FCatalyst-View-TT.git diff --git a/lib/Catalyst/View/TT.pm b/lib/Catalyst/View/TT.pm index d5dde6e..f4d5762 100644 --- a/lib/Catalyst/View/TT.pm +++ b/lib/Catalyst/View/TT.pm @@ -8,10 +8,13 @@ use Data::Dump 'dump'; use Template; use Template::Timer; use MRO::Compat; +use Scalar::Util qw/blessed weaken/; -our $VERSION = '0.30'; +our $VERSION = '0.38'; +$VERSION = eval $VERSION; __PACKAGE__->mk_accessors('template'); +__PACKAGE__->mk_accessors('expose_methods'); __PACKAGE__->mk_accessors('include_path'); *paths = \&include_path; @@ -24,46 +27,52 @@ Catalyst::View::TT - Template View Class # use the helper to create your View - myapp_create.pl view TT TT + myapp_create.pl view Web TT + +# add custom configration in View/Web.pm + + __PACKAGE__->config( + # any TT configuration items go here + TEMPLATE_EXTENSION => '.tt', + CATALYST_VAR => 'c', + TIMER => 0, + ENCODING => 'utf-8' + # Not set by default + PRE_PROCESS => 'config/main', + WRAPPER => 'site/wrapper', + render_die => 1, # Default for new apps, see render method docs + expose_methods => [qw/method_in_view_class/], + ); -# configure in lib/MyApp.pm (Could be set from configfile instead) +# add include path configuration in MyApp.pm - MyApp->config( - name => 'MyApp', - root => MyApp->path_to('root'), - 'View::TT' => { - # any TT configurations items go here + __PACKAGE__->config( + 'View::Web' => { INCLUDE_PATH => [ - MyApp->path_to( 'root', 'src' ), - MyApp->path_to( 'root', 'lib' ), + __PACKAGE__->path_to( 'root', 'src' ), + __PACKAGE__->path_to( 'root', 'lib' ), ], - TEMPLATE_EXTENSION => '.tt', - CATALYST_VAR => 'c', - TIMER => 0, - # Not set by default - PRE_PROCESS => 'config/main', - WRAPPER => 'site/wrapper', }, ); - + # render view from lib/MyApp.pm or lib/MyApp::Controller::SomeController.pm - + sub message : Global { my ( $self, $c ) = @_; $c->stash->{template} = 'message.tt2'; $c->stash->{message} = 'Hello World!'; - $c->forward( $c->view('TT') ); + $c->forward( $c->view('Web') ); } # access variables from template The message is: [% message %]. - + # example when CATALYST_VAR is set to 'Catalyst' - Context is [% Catalyst %] - The base is [% Catalyst.req.base %] - The name is [% Catalyst.config.name %] - + Context is [% Catalyst %] + The base is [% Catalyst.req.base %] + The name is [% Catalyst.config.name %] + # example when CATALYST_VAR isn't set Context is [% c %] The base is [% base %] @@ -88,6 +97,7 @@ sub new { my $config = { EVAL_PERL => 0, TEMPLATE_EXTENSION => '', + CLASS => 'Template', %{ $class->config }, %{$arguments}, }; @@ -123,18 +133,19 @@ sub new { } my $self = $class->next::method( - $c, { %$config }, + $c, { %$config }, ); # Set base include paths. Local'd in render if needed $self->include_path($config->{INCLUDE_PATH}); - + + $self->expose_methods($config->{expose_methods}); $self->config($config); # Creation of template outside of call to new so that we can pass [ $self ] # as INCLUDE_PATH config item, which then gets ->paths() called to get list # of include paths to search for templates. - + # Use a weakend copy of self so we dont have loops preventing GC from working my $copy = $self; Scalar::Util::weaken($copy); @@ -168,6 +179,7 @@ sub new { @{ $p->{copy_config} }; } } + local $@; eval "require $prov"; if(!$@) { push @providers, "$prov"->new($p->{args}); @@ -183,10 +195,10 @@ sub new { $config->{LOAD_TEMPLATES} = \@providers; } } - - $self->{template} = - Template->new($config) || do { - my $error = Template->error(); + + $self->{template} = + $config->{CLASS}->new($config) || do { + my $error = $config->{CLASS}->error(); $c->log->error($error); $c->error($error); return undef; @@ -207,13 +219,13 @@ sub process { return 0; } - my $output = $self->render($c, $template); - - if (UNIVERSAL::isa($output, 'Template::Exception')) { - my $error = qq/Couldn't render template "$output"/; - $c->log->error($error); - $c->error($error); - return 0; + local $@; + my $output = eval { $self->render($c, $template) }; + if (my $err = $@) { + return $self->_rendering_error($c, $template . ': ' . $err); + } + if (blessed($output) && $output->isa('Template::Exception')) { + $self->_rendering_error($c, $output); } unless ( $c->response->content_type ) { @@ -225,43 +237,73 @@ sub process { return 1; } +sub _rendering_error { + my ($self, $c, $err) = @_; + my $error = qq/Couldn't render template "$err"/; + $c->log->error($error); + $c->error($error); + return 0; +} + sub render { my ($self, $c, $template, $args) = @_; - $c->log->debug(qq/Rendering template "$template"/) if $c->debug; + $c->log->debug(qq/Rendering template "$template"/) if $c && $c->debug; my $output; - my $vars = { + my $vars = { (ref $args eq 'HASH' ? %$args : %{ $c->stash() }), $self->template_vars($c) }; - local $self->{include_path} = + local $self->{include_path} = [ @{ $vars->{additional_template_paths} }, @{ $self->{include_path} } ] if ref $vars->{additional_template_paths}; - unless ($self->template->process( $template, $vars, \$output ) ) { - return $self->template->error; - } else { - return $output; + unless ( $self->template->process( $template, $vars, \$output ) ) { + if (exists $self->{render_die}) { + die $self->template->error if $self->{render_die}; + return $self->template->error; + } + $c->log->debug('The Catalyst::View::TT render() method will start dying on error in a future release. Unless you are calling the render() method manually, you probably want the new behaviour, so set render_die => 1 in config for ' . blessed($self) . '. If you wish to continue to return the exception rather than throwing it, add render_die => 0 to your config.') if $c->debug; + return $self->template->error; } + return $output; } sub template_vars { my ( $self, $c ) = @_; + return () unless $c; my $cvar = $self->config->{CATALYST_VAR}; - defined $cvar + my %vars = defined $cvar ? ( $cvar => $c ) : ( c => $c, base => $c->req->base, name => $c->config->{name} - ) + ); + + if ($self->expose_methods) { + my $meta = $self->meta; + foreach my $method_name (@{$self->expose_methods}) { + my $method = $meta->find_method_by_name( $method_name ); + unless ($method) { + Catalyst::Exception->throw( "$method_name not found in TT view" ); + } + my $method_body = $method->body; + my $weak_ctx = $c; + weaken $weak_ctx; + my $sub = sub { + $self->$method_body($weak_ctx, @_); + }; + $vars{$method_name} = $sub; + } + } + return %vars; } - 1; __END__ @@ -270,25 +312,25 @@ __END__ This is the Catalyst view class for the L