check for with/set_logger at import time

nichtich · Jan 11, 2013 · 84d7d9e · 84d7d9e
1 parent 6ae293d
commit 84d7d9e
Showing 1 changed file with 10 additions and 18 deletions.
diff --git a/lib/Log/Contextual.pm b/lib/Log/Contextual.pm
@@ -55,27 +55,19 @@ sub before_import {
 
    $router->before_import(%router_args);
 
-   $spec->add_export(
-      '&set_logger',
-      sub {
-         my $router = $class->router;
+   if ($exports->{'&set_logger'}) {
+      die ref($router) . " does not support set_logger()"
+        unless $router->does('Log::Contextual::Role::Router::SetLogger');
 
-         die ref($router) . " does not support set_logger()"
-           unless $router->does('Log::Contextual::Role::Router::SetLogger');
-
-         return $router->set_logger(@_);
-      }) if $exports->{'&set_logger'};
-
-   $spec->add_export(
-      '&with_logger',
-      sub {
-         my $router = $class->router;
+      $spec->add_export('&set_logger', sub { $router->set_logger(@_) })
+   }
 
-         die ref($router) . " does not support with_logger()"
-           unless $router->does('Log::Contextual::Role::Router::WithLogger');
+   if ($exports->{'&with_logger'}) {
+      die ref($router) . " does not support with_logger()"
+        unless $router->does('Log::Contextual::Role::Router::WithLogger');
 
-         return $router->with_logger(@_);
-      }) if $exports->{'&with_logger'};
+      $spec->add_export('&with_logger', sub { $router->with_logger(@_) })
+   }
 
    my @levels = @{$class->arg_levels($spec->config->{levels})};
    for my $level (@levels) {