Forráskód Böngészése

made some tweaks to error levels

Fabien Potencier 14 éve
szülő
commit
fb24b95bd5

+ 1 - 1
src/Symfony/Bridge/Doctrine/Logger/DbalLogger.php

@@ -52,6 +52,6 @@ class DbalLogger extends DebugStack
      */
     public function log($message)
     {
-        $this->logger->info($message);
+        $this->logger->debug($message);
     }
 }

+ 2 - 5
src/Symfony/Bundle/FrameworkBundle/Debug/TraceableEventDispatcher.php

@@ -58,11 +58,8 @@ class TraceableEventDispatcher extends ContainerAwareEventDispatcher implements
             } else {
                 $typeDefinition = '[?] '.var_export($listener, true);
             }
-            $msg = sprintf('The given callback (%s) for event "%s" is not callable.', $typeDefinition, $eventName);
-            if (null !== $this->logger) {
-                $this->logger->err($msg);
-            }
-            throw new \RuntimeException($msg);
+
+            throw new \RuntimeException(sprintf('The given callback (%s) for event "%s" is not callable.', $typeDefinition, $eventName));
         }
 
         parent::addListener($eventName, $listener, $priority);

+ 1 - 1
src/Symfony/Bundle/FrameworkBundle/EventListener/RouterListener.php

@@ -78,7 +78,7 @@ class RouterListener
             $parameters = $this->router->match($request->getPathInfo());
 
             if (null !== $this->logger) {
-                $this->logger->debug(sprintf('Matched route "%s" (parameters: %s)', $parameters['_route'], $this->parametersToString($parameters)));
+                $this->logger->info(sprintf('Matched route "%s" (parameters: %s)', $parameters['_route'], $this->parametersToString($parameters)));
             }
 
             $request->attributes->add($parameters);

+ 1 - 5
src/Symfony/Component/HttpKernel/Controller/ControllerResolver.php

@@ -54,7 +54,7 @@ class ControllerResolver implements ControllerResolverInterface
     {
         if (!$controller = $request->attributes->get('_controller')) {
             if (null !== $this->logger) {
-                $this->logger->err('Unable to look for the controller as the "_controller" parameter is missing');
+                $this->logger->warn('Unable to look for the controller as the "_controller" parameter is missing');
             }
 
             return false;
@@ -70,10 +70,6 @@ class ControllerResolver implements ControllerResolverInterface
             throw new \InvalidArgumentException(sprintf('Method "%s::%s" does not exist.', get_class($controller), $method));
         }
 
-        if (null !== $this->logger) {
-            $this->logger->debug(sprintf('Using controller "%s::%s"', get_class($controller), $method));
-        }
-
         return array($controller, $method);
     }
 

+ 1 - 1
src/Symfony/Component/HttpKernel/Profiler/Profiler.php

@@ -89,7 +89,7 @@ class Profiler
     public function saveProfile(Profile $profile)
     {
         if (!($ret = $this->storage->write($profile)) && null !== $this->logger) {
-            $this->logger->err('Unable to store the profiler information.');
+            $this->logger->warn('Unable to store the profiler information.');
         }
 
         return $ret;

+ 1 - 1
src/Symfony/Component/Security/Http/Firewall/AnonymousAuthenticationListener.php

@@ -49,7 +49,7 @@ class AnonymousAuthenticationListener implements ListenerInterface
         $this->context->setToken(new AnonymousToken($this->key, 'anon.', array()));
 
         if (null !== $this->logger) {
-            $this->logger->debug(sprintf('Populated SecurityContext with an anonymous Token'));
+            $this->logger->info(sprintf('Populated SecurityContext with an anonymous Token'));
         }
     }
 }

+ 2 - 2
src/Symfony/Component/Security/Http/Firewall/ChannelListener.php

@@ -48,7 +48,7 @@ class ChannelListener implements ListenerInterface
 
         if ('https' === $channel && !$request->isSecure()) {
             if (null !== $this->logger) {
-                $this->logger->debug('Redirecting to HTTPS');
+                $this->logger->info('Redirecting to HTTPS');
             }
 
             $response = $this->authenticationEntryPoint->start($request);
@@ -60,7 +60,7 @@ class ChannelListener implements ListenerInterface
 
         if ('http' === $channel && $request->isSecure()) {
             if (null !== $this->logger) {
-                $this->logger->debug('Redirecting to HTTP');
+                $this->logger->info('Redirecting to HTTP');
             }
 
             $response = $this->authenticationEntryPoint->start($request);

+ 1 - 1
src/Symfony/Component/Security/Http/Firewall/UsernamePasswordFormAuthenticationListener.php

@@ -56,7 +56,7 @@ class UsernamePasswordFormAuthenticationListener extends AbstractAuthenticationL
     {
         if ($this->options['post_only'] && 'post' !== strtolower($request->getMethod())) {
             if (null !== $this->logger) {
-                $this->logger->err(sprintf('Authentication method not supported: %s.', $request->getMethod()));
+                $this->logger->debug(sprintf('Authentication method not supported: %s.', $request->getMethod()));
             }
 
             return null;

+ 3 - 3
src/Symfony/Component/Security/Http/RememberMe/AbstractRememberMeServices.php

@@ -112,7 +112,7 @@ abstract class AbstractRememberMeServices implements RememberMeServicesInterface
             }
 
             if (null !== $this->logger) {
-                $this->logger->debug('Remember-me cookie accepted.');
+                $this->logger->info('Remember-me cookie accepted.');
             }
 
             return new RememberMeToken($user, $this->providerKey, $this->key);
@@ -126,11 +126,11 @@ abstract class AbstractRememberMeServices implements RememberMeServicesInterface
             }
         } catch (UnsupportedUserException $unSupported) {
             if (null !== $this->logger) {
-                $this->logger->err('User class for remember-me cookie not supported.');
+                $this->logger->warn('User class for remember-me cookie not supported.');
             }
         } catch (AuthenticationException $invalid) {
             if (null !== $this->logger) {
-                $this->logger->err('Remember-Me authentication failed: '.$invalid->getMessage());
+                $this->logger->debug('Remember-Me authentication failed: '.$invalid->getMessage());
             }
         }
 

+ 1 - 2
tests/Symfony/Tests/Component/HttpKernel/Controller/ControllerResolverTest.php

@@ -25,12 +25,11 @@ class ControllerResolverTest extends \PHPUnit_Framework_TestCase
 
         $request = Request::create('/');
         $this->assertFalse($resolver->getController($request), '->getController() returns false when the request has no _controller attribute');
-        $this->assertEquals(array('Unable to look for the controller as the "_controller" parameter is missing'), $logger->getLogs('debug'));
+        $this->assertEquals(array('Unable to look for the controller as the "_controller" parameter is missing'), $logger->getLogs('warn'));
 
         $request->attributes->set('_controller', 'Symfony\Tests\Component\HttpKernel\ControllerResolverTest::testGetController');
         $controller = $resolver->getController($request);
         $this->assertInstanceOf('Symfony\Tests\Component\HttpKernel\ControllerResolverTest', $controller[0], '->getController() returns a PHP callable');
-        $this->assertContains('Using controller "Symfony\Tests\Component\HttpKernel\ControllerResolverTest::testGetController"', $logger->getLogs('debug'));
 
         $request->attributes->set('_controller', $lambda = function () {});
         $controller = $resolver->getController($request);