Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
a few more nested helper tests
  • Loading branch information
kraih committed Aug 10, 2014
1 parent 121f050 commit 0b62dab
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 5 deletions.
9 changes: 4 additions & 5 deletions lib/Mojolicious/Renderer.pm
Expand Up @@ -72,12 +72,11 @@ sub get_helper {
my ($self, $name) = @_;

if (my $helper = $self->helpers->{$name}) { return $helper }
if (my $proxy = $self->{proxy}{$name}) { return $proxy }

my $lookup = $self->{lookup} ||= {};
return undef
unless $lookup->{$name} || grep {/^\Q$name\E\./} keys %{$self->helpers};
return $lookup->{$name}
||= sub { Mojolicious::Renderer::_Proxy->new(c => shift, p => $name) };
return undef unless grep {/^\Q$name\E\./} keys %{$self->helpers};
return $self->{proxy}{$name}
= sub { Mojolicious::Renderer::_Proxy->new(c => shift, p => $name) };
}

sub render {
Expand Down
2 changes: 2 additions & 0 deletions t/mojolicious/renderer.t
Expand Up @@ -67,6 +67,8 @@ $first->app->log->level('fatal');
$first->app->helper('myapp.defaults' => sub { shift->app->defaults(@_) });
ok $first->app->renderer->get_helper('myapp'), 'found helper';
ok $first->app->renderer->get_helper('myapp.defaults'), 'found helper';
ok !$first->app->renderer->get_helper('myap.'), 'no helper';
ok !$first->app->renderer->get_helper('yapp'), 'no helper';
$first->myapp->defaults(foo => 'bar');
is $first->myapp->defaults('foo'), 'bar', 'right result';
is $first->app->myapp->defaults('foo'), 'bar', 'right result';
Expand Down

0 comments on commit 0b62dab

Please sign in to comment.