CS fixes

src/Monolog/Formatter/JsonFormatter.php

@@ -76,7 +76,7 @@ public function formatBatch(array $records)
     /**
      * Return a JSON-encoded array of records.
      *
-     * @param  array $records
+     * @param  array  $records
      * @return string
      */
     protected function formatBatchJson(array $records)
@@ -88,14 +88,14 @@ protected function formatBatchJson(array $records)
      * Use new lines to separate records instead of a
      * JSON-encoded array.
      *
-     * @param  array $records
+     * @param  array  $records
      * @return string
      */
     protected function formatBatchNewlines(array $records)
     {
         $instance = $this;
 
-        array_walk($records, function(&$value, $key) use ($instance) {
+        array_walk($records, function (&$value, $key) use ($instance) {
             $value = $instance->format($value);
         });
 

src/Monolog/Formatter/LogglyFormatter.php

@@ -42,6 +42,7 @@ public function format(array $record)
             $record["timestamp"] = $record["datetime"]->format("c");
             // @todo unset the 'datetime' parameter, retained for BC
         }
+
         return parent::format($record);
     }
 

src/Monolog/Handler/BrowserConsoleHandler.php

@@ -11,7 +11,6 @@
 
 namespace Monolog\Handler;
 
-use Monolog\Logger;
 use Monolog\Formatter\LineFormatter;
 
 /**
@@ -103,7 +102,8 @@ private static function generateScript()
                 );
             }
         }
-        return "(function(c){if (c && c.groupCollapsed) {\n" . implode("\n", $script) . "\n}})(console);";
+
+        return "(function (c) {if (c && c.groupCollapsed) {\n" . implode("\n", $script) . "\n}})(console);";
     }
 
     private static function handleStyles($formatted)
@@ -130,7 +130,7 @@ private static function handleCustomStyles($style, $string)
         static $colors = array('blue', 'green', 'red', 'magenta', 'orange', 'black', 'grey');
         static $labels = array();
 
-        return preg_replace_callback('/macro\s*:(.*?)(?:;|$)/', function($m) use($string, &$colors, &$labels) {
+        return preg_replace_callback('/macro\s*:(.*?)(?:;|$)/', function ($m) use ($string, &$colors, &$labels) {
             if (trim($m[1]) === 'autolabel') {
                 // Format the string as a label with consistent auto assigned background color
                 if (!isset($labels[$string])) {
@@ -160,6 +160,7 @@ private static function dump($title, array $dict)
             }
             $script[] = self::call('log', self::quote('%s: %o'), self::quote($key), $value);
         }
+
         return $script;
     }
 
@@ -172,6 +173,7 @@ private static function call()
     {
         $args = func_get_args();
         $method = array_shift($args);
+
         return self::call_array($method, $args);
     }
 

src/Monolog/Handler/DynamoDbHandler.php

@@ -14,7 +14,6 @@
 use Aws\Common\Aws;
 use Aws\DynamoDb\DynamoDbClient;
 use Monolog\Formatter\ScalarFormatter;
-use Monolog\Handler\AbstractProcessingHandler;
 use Monolog\Logger;
 
 /**

src/Monolog/Handler/FilterHandler.php

@@ -36,10 +36,10 @@ class FilterHandler extends AbstractHandler
     protected $bubble;
 
     /**
-     * @param callable|HandlerInterface $handler Handler or factory callable($record, $this).
+     * @param callable|HandlerInterface $handler        Handler or factory callable($record, $this).
      * @param int|array                 $minLevelOrList A list of levels to accept or a minimum level if maxLevel is provided
-     * @param int                       $maxLevel Maximum level to accept, only used if $minLevelOrList is an array
-     * @param Boolean                   $bubble Whether the messages that are handled can bubble up the stack or not
+     * @param int                       $maxLevel       Maximum level to accept, only used if $minLevelOrList is an array
+     * @param Boolean                   $bubble         Whether the messages that are handled can bubble up the stack or not
      */
     public function __construct($handler, $minLevelOrList = Logger::DEBUG, $maxLevel = Logger::EMERGENCY, $bubble = true)
     {
@@ -57,8 +57,8 @@ public function getAcceptedLevels()
     }
 
     /**
-     * @param int|array                 $minLevelOrList A list of levels to accept or a minimum level if maxLevel is provided
-     * @param int                       $maxLevel Maximum level to accept, only used if $minLevelOrList is an array
+     * @param int|array $minLevelOrList A list of levels to accept or a minimum level if maxLevel is provided
+     * @param int       $maxLevel       Maximum level to accept, only used if $minLevelOrList is an array
      */
     public function setAcceptedLevels($minLevelOrList = Logger::DEBUG, $maxLevel = Logger::EMERGENCY)
     {

src/Monolog/Handler/FlowdockHandler.php

@@ -32,9 +32,9 @@ class FlowdockHandler extends SocketHandler
     protected $apiToken;
 
     /**
-     * @param string $apiToken
-     * @param bool|int $level The minimum logging level at which this handler will be triggered
-     * @param bool $bubble Whether the messages that are handled can bubble up the stack or not
+     * @param string   $apiToken
+     * @param bool|int $level    The minimum logging level at which this handler will be triggered
+     * @param bool     $bubble   Whether the messages that are handled can bubble up the stack or not
      *
      * @throws MissingExtensionException if OpenSSL is missing
      */

src/Monolog/Handler/GelfHandler.php

@@ -31,9 +31,9 @@ class GelfHandler extends AbstractProcessingHandler
     protected $publisher;
 
     /**
-     * @param PublisherInterface|IMessagePublisher  $publisher a publisher object
-     * @param integer    $level     The minimum logging level at which this handler will be triggered
-     * @param boolean    $bubble    Whether the messages that are handled can bubble up the stack or not
+     * @param PublisherInterface|IMessagePublisher $publisher a publisher object
+     * @param integer                              $level     The minimum logging level at which this handler will be triggered
+     * @param boolean                              $bubble    Whether the messages that are handled can bubble up the stack or not
      */
     public function __construct($publisher, $level = Logger::DEBUG, $bubble = true)
     {

src/Monolog/Handler/HipChatHandler.php

@@ -285,7 +285,7 @@ private function combineRecords($records)
      * system where `mb_strlen()` is unavailable.
      *
      * @param string $str
-     * @param int $length
+     * @param int    $length
      *
      * @return bool
      */

src/Monolog/Handler/NativeMailerHandler.php

@@ -77,7 +77,7 @@ public function __construct($to, $subject, $from, $level = Logger::ERROR, $bubbl
     /**
      * Add headers to the message
      *
-     * @param string|array $headers Custom added headers
+     * @param  string|array $headers Custom added headers
      * @return null
      */
     public function addHeader($headers)
@@ -123,23 +123,25 @@ public function getEncoding()
     }
 
     /**
-     * @param string $contentType The content type of the email - Defaults to text/plain. Use text/html for HTML
-     * messages.
+     * @param  string $contentType The content type of the email - Defaults to text/plain. Use text/html for HTML
+     *                             messages.
      * @return self
      */
     public function setContentType($contentType)
     {
         $this->contentType = $contentType;
+
         return $this;
     }
 
     /**
-     * @param string $encoding
+     * @param  string $encoding
      * @return self
      */
     public function setEncoding($encoding)
     {
         $this->encoding = $encoding;
+
         return $this;
     }
 }

src/Monolog/Handler/RavenHandler.php

@@ -14,7 +14,6 @@
 use Monolog\Formatter\LineFormatter;
 use Monolog\Formatter\FormatterInterface;
 use Monolog\Logger;
-use Monolog\Handler\AbstractProcessingHandler;
 use Raven_Client;
 
 /**

src/Monolog/Handler/RollbarHandler.php

@@ -21,9 +21,9 @@ class RollbarHandler extends AbstractProcessingHandler
     protected $rollbarNotifier;
 
     /**
-     * @param RollbarNotifier  $rollbarNotifier RollbarNotifier object constructed with valid token
-     * @param integer          $level           The minimum logging level at which this handler will be triggered
-     * @param boolean          $bubble          Whether the messages that are handled can bubble up the stack or not
+     * @param RollbarNotifier $rollbarNotifier RollbarNotifier object constructed with valid token
+     * @param integer         $level           The minimum logging level at which this handler will be triggered
+     * @param boolean         $bubble          Whether the messages that are handled can bubble up the stack or not
      */
     public function __construct(RollbarNotifier $rollbarNotifier, $level = Logger::ERROR, $bubble = true)
     {

tests/Monolog/Formatter/ElasticaFormatterTest.php

@@ -12,7 +12,6 @@
 namespace Monolog\Formatter;
 
 use Monolog\Logger;
-use Monolog\Formatter\ElasticaFormatter;
 
 class ElasticaFormatterTest extends \PHPUnit_Framework_TestCase
 {

tests/Monolog/Formatter/GelfMessageFormatterTest.php

@@ -12,7 +12,6 @@
 namespace Monolog\Formatter;
 
 use Monolog\Logger;
-use Monolog\Formatter\GelfMessageFormatter;
 
 class GelfMessageFormatterTest extends \PHPUnit_Framework_TestCase
 {

tests/Monolog/Formatter/JsonFormatterTest.php

@@ -64,7 +64,7 @@ public function testFormatBatchNewlines()
             $this->getRecord(Logger::WARNING),
             $this->getRecord(Logger::DEBUG),
         );
-        array_walk($expected, function(&$value, $key) {
+        array_walk($expected, function (&$value, $key) {
             $value = json_encode($value);
         });
         $this->assertEquals(implode("\n", $expected), $formatter->formatBatch($records));

tests/Monolog/Formatter/LogglyFormatterTest.php

@@ -11,7 +11,6 @@
 
 namespace Monolog\Formatter;
 
-use Monolog\Logger;
 use Monolog\TestCase;
 
 class LogglyFormatterTest extends TestCase

tests/Monolog/Formatter/LogstashFormatterTest.php

@@ -12,7 +12,6 @@
 namespace Monolog\Formatter;
 
 use Monolog\Logger;
-use Monolog\Formatter\LogstashFormatter;
 
 class LogstashFormatterTest extends \PHPUnit_Framework_TestCase
 {

tests/Monolog/Functional/Handler/FirePHPHandlerTest.php

@@ -9,7 +9,7 @@
  * file that was distributed with this source code.
  */
 
-spl_autoload_register(function($class) {
+spl_autoload_register(function ($class) {
     $file = __DIR__.'/../../../../src/'.strtr($class, '\\', '/').'.php';
     if (file_exists($file)) {
         require $file;

tests/Monolog/Handler/AbstractProcessingHandlerTest.php

@@ -70,7 +70,7 @@ public function testProcessRecord()
         $handledRecord = null;
         $handler->expects($this->once())
             ->method('write')
-            ->will($this->returnCallback(function($record) use (&$handledRecord) {
+            ->will($this->returnCallback(function ($record) use (&$handledRecord) {
                 $handledRecord = $record;
             }))
         ;

tests/Monolog/Handler/BrowserConsoleHandlerTest.php

@@ -40,7 +40,7 @@ public function testStyling()
         $handler->handle($this->getRecord(Logger::DEBUG, 'foo[[bar]]{color: red}'));
 
         $expected = <<<EOF
-(function(c){if (c && c.groupCollapsed) {
+(function (c) {if (c && c.groupCollapsed) {
 c.log("%cfoo%cbar%c", "font-weight: normal", "color: red", "font-weight: normal");
 }})(console);
 EOF;
@@ -56,7 +56,7 @@ public function testEscaping()
         $handler->handle($this->getRecord(Logger::DEBUG, "[foo] [[\"bar\n[baz]\"]]{color: red}"));
 
         $expected = <<<EOF
-(function(c){if (c && c.groupCollapsed) {
+(function (c) {if (c && c.groupCollapsed) {
 c.log("%c[foo] %c\"bar\\n[baz]\"%c", "font-weight: normal", "color: red", "font-weight: normal");
 }})(console);
 EOF;
@@ -75,7 +75,7 @@ public function testAutolabel()
         $handler->handle($this->getRecord(Logger::DEBUG, '[[foo]]{macro: autolabel}'));
 
         $expected = <<<EOF
-(function(c){if (c && c.groupCollapsed) {
+(function (c) {if (c && c.groupCollapsed) {
 c.log("%c%cfoo%c", "font-weight: normal", "background-color: blue; color: white; border-radius: 3px; padding: 0 2px 0 2px", "font-weight: normal");
 c.log("%c%cbar%c", "font-weight: normal", "background-color: green; color: white; border-radius: 3px; padding: 0 2px 0 2px", "font-weight: normal");
 c.log("%c%cfoo%c", "font-weight: normal", "background-color: blue; color: white; border-radius: 3px; padding: 0 2px 0 2px", "font-weight: normal");
@@ -93,7 +93,7 @@ public function testContext()
         $handler->handle($this->getRecord(Logger::DEBUG, 'test', array('foo' => 'bar')));
 
         $expected = <<<EOF
-(function(c){if (c && c.groupCollapsed) {
+(function (c) {if (c && c.groupCollapsed) {
 c.groupCollapsed("%ctest", "font-weight: normal");
 c.log("%c%s", "font-weight: bold", "Context");
 c.log("%s: %o", "foo", "bar");
@@ -118,7 +118,7 @@ public function testConcurrentHandlers()
         $handler2->handle($this->getRecord(Logger::DEBUG, 'test4'));
 
         $expected = <<<EOF
-(function(c){if (c && c.groupCollapsed) {
+(function (c) {if (c && c.groupCollapsed) {
 c.log("%ctest1", "font-weight: normal");
 c.log("%ctest2", "font-weight: normal");
 c.log("%ctest3", "font-weight: normal");

tests/Monolog/Handler/ElasticSearchHandlerTest.php

@@ -11,7 +11,6 @@
 
 namespace Monolog\Handler;
 
-use Monolog\Handler\ElasticSearchHandler;
 use Monolog\Formatter\ElasticaFormatter;
 use Monolog\Formatter\NormalizerFormatter;
 use Monolog\TestCase;
@@ -185,7 +184,7 @@ public function testHandleIntegration()
         $handler = new ElasticSearchHandler($client, $this->options);
         try {
             $handler->handleBatch(array($msg));
-        } catch(\RuntimeException $e) {
+        } catch (\RuntimeException $e) {
             $this->markTestSkipped("Cannot connect to Elastic Search server on localhost");
         }
 
@@ -208,7 +207,7 @@ public function testHandleIntegration()
 
     /**
      * Return last created document id from ES response
-     * @param Response $response Elastica Response object
+     * @param  Response    $response Elastica Response object
      * @return string|null
      */
     protected function getCreatedDocId(Response $response)
@@ -221,10 +220,10 @@ protected function getCreatedDocId(Response $response)
 
     /**
      * Retrieve document by id from Elasticsearch
-     * @param Client $client Elastica client
-     * @param string $index
-     * @param string $type
-     * @param string $documentId
+     * @param  Client $client     Elastica client
+     * @param  string $index
+     * @param  string $type
+     * @param  string $documentId
      * @return array
      */
     protected function getDocSourceFromElastic(Client $client, $index, $type, $documentId)
@@ -234,6 +233,7 @@ protected function getDocSourceFromElastic(Client $client, $index, $type, $docum
         if (!empty($data['_source'])) {
             return $data['_source'];
         }
+
         return array();
     }
 }

tests/Monolog/Handler/FingersCrossedHandlerTest.php

@@ -102,7 +102,7 @@ public function testHandleBufferLimit()
     public function testHandleWithCallback()
     {
         $test = new TestHandler();
-        $handler = new FingersCrossedHandler(function($record, $handler) use ($test) {
+        $handler = new FingersCrossedHandler(function ($record, $handler) use ($test) {
                     return $test;
                 });
         $handler->handle($this->getRecord(Logger::DEBUG));
@@ -120,7 +120,7 @@ public function testHandleWithCallback()
      */
     public function testHandleWithBadCallbackThrowsException()
     {
-        $handler = new FingersCrossedHandler(function($record, $handler) {
+        $handler = new FingersCrossedHandler(function ($record, $handler) {
                     return 'foo';
                 });
         $handler->handle($this->getRecord(Logger::WARNING));

tests/Monolog/Handler/GelfHandlerLegacyTest.php

@@ -23,8 +23,6 @@ public function setUp()
         if (!class_exists('Gelf\MessagePublisher') || !class_exists('Gelf\Message')) {
             $this->markTestSkipped("mlehner/gelf-php not installed");
         }
-
-        require_once __DIR__ . '/GelfMocks.php';
     }
 
     /**

tests/Monolog/Handler/GelfHandlerTest.php

@@ -64,7 +64,6 @@ public function testDebug()
 
         $handler = $this->getHandler($messagePublisher);
 
-
         $handler->handle($record);
 
     }
@@ -87,7 +86,6 @@ public function testWarning()
 
         $handler = $this->getHandler($messagePublisher);
 
-
         $handler->handle($record);
     }