Merge branch 'master' of https://github.com/cesargb/laravel-logs-rotate

Author: cesargb

src/Helpers/Log.php

@@ -18,7 +18,7 @@ public static function getLaravelLogFiles()
         $files = [];
 
         foreach (self::getHandlers() as $handler) {
-            if ($handler instanceof StreamHandler && !$handler instanceof RotatingFileHandler) {
+            if ($handler instanceof StreamHandler && ! $handler instanceof RotatingFileHandler) {
                 $files[] = $handler->getUrl();
             }
         }
@@ -29,7 +29,7 @@ public static function getLaravelLogFiles()
     public static function closeHandlers()
     {
         foreach (self::getHandlers() as $handler) {
-            if ($handler instanceof StreamHandler && !$handler instanceof RotatingFileHandler) {
+            if ($handler instanceof StreamHandler && ! $handler instanceof RotatingFileHandler) {
                 if (method_exists($handler, 'close')) {
                     $handler->close();
                 }

tests/Commands/RotateFileTest.php

@@ -61,7 +61,7 @@ public function itCanRotateFileMax()
     {
         $file = $this->tmpDir.'/file1';
 
-        for ($n = 0; $n < 5; ++$n) {
+        for ($n = 0; $n < 5; $n++) {
             file_put_contents($file, 'test');
 
             $resultCode = Artisan::call('rotate:files', [
@@ -76,7 +76,7 @@ public function itCanRotateFileMax()
 
         $this->assertEquals(filesize($file), 0);
 
-        for ($n = 1; $n < 4; ++$n) {
+        for ($n = 1; $n < 4; $n++) {
             $this->assertFileExists($file.'.'.$n.'.gz');
         }
 

tests/Handlers/RotativeHandlerTest.php

@@ -54,7 +54,7 @@ public function testItCanRotateLogsWithMaxfiles()
 
         $this->app['config']->set('rotate.log_compress_files', true);
 
-        for ($n = 0; $n < 10; ++$n) {
+        for ($n = 0; $n < 10; $n++) {
             $this->writeLog();
 
             $this->assertGreaterThan(0, filesize(app()->storagePath().'/logs/laravel.log'));

tests/RotateTest.php

@@ -103,12 +103,12 @@ public function testItNotRotateLogsDaily()
     public function testItCanRotateLogsCustomStreamFile()
     {
         $this->app['config']->set('logging.channels.custom', [
-                'driver' => 'monolog',
-                'handler' => StreamHandler::class,
-                'with' => [
-                    'stream' => app()->storagePath().'/logs/custom.log',
-                ],
-            ]);
+            'driver' => 'monolog',
+            'handler' => StreamHandler::class,
+            'with' => [
+                'stream' => app()->storagePath().'/logs/custom.log',
+            ],
+        ]);
 
         $this->app['config']->set('logging.default', 'custom');
 
@@ -129,12 +129,12 @@ public function testItCanRotateLogsCustomStreamFile()
     public function testItNotRotateLogsCustomStreamStd()
     {
         $this->app['config']->set('logging.channels.custom', [
-                'driver' => 'monolog',
-                'handler' => StreamHandler::class,
-                'with' => [
-                    'stream' => 'php://stdout',
-                ],
-            ]);
+            'driver' => 'monolog',
+            'handler' => StreamHandler::class,
+            'with' => [
+                'stream' => 'php://stdout',
+            ],
+        ]);
 
         $this->app['config']->set('logging.default', 'custom');