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

Author: cesargb

src/Handlers/AbstractHandler.php

@@ -41,15 +41,15 @@ protected function validate()
 
     private function validateFile(): bool
     {
-        if (!is_file($this->file)) {
+        if (! is_file($this->file)) {
             return false;
         }
 
         if (filesize($this->file) == 0) {
             return false;
         }
 
-        if (!is_writable($this->file)) {
+        if (! is_writable($this->file)) {
             return false;
         }
 
@@ -59,7 +59,7 @@ private function validateFile(): bool
     private function validateDirectory(): bool
     {
         if (is_dir($this->dir_to_archive)) {
-            if (!is_writable($this->dir_to_archive)) {
+            if (! is_writable($this->dir_to_archive)) {
                 event(new RotateHasFailed($this->file, new Exception('Directory '.$this->dir_to_archive.' to archive logs is not writable')));
 
                 return false;
@@ -74,7 +74,7 @@ private function validateDirectory(): bool
             return false;
         }
 
-        if (!mkdir($this->dir_to_archive, 0777, true)) {
+        if (! mkdir($this->dir_to_archive, 0777, true)) {
             event(new RotateHasFailed($this->file, new Exception('Directory '.$this->dir_to_archive.' to archive logs is not writable')));
 
             return false;
@@ -101,19 +101,19 @@ protected function moveData($fileSource, $fileDestination)
             return false;
         }
 
-        if (!flock($fdSource, LOCK_EX)) {
+        if (! flock($fdSource, LOCK_EX)) {
             fclose($fdSource);
 
             return false;
         }
 
-        if (!copy($fileSource, $fileDestination)) {
+        if (! copy($fileSource, $fileDestination)) {
             fclose($fdSource);
 
             return false;
         }
 
-        if (!ftruncate($fdSource, 0)) {
+        if (! ftruncate($fdSource, 0)) {
             fclose($fdSource);
 
             unlink($fileDestination);

src/Handlers/RotativeHandler.php

@@ -20,7 +20,7 @@ public function __construct($file, $max_files = 0, bool $compress = true, $dir_t
 
     public function run()
     {
-        if (!$this->validate()) {
+        if (! $this->validate()) {
             return false;
         }
 
@@ -53,7 +53,7 @@ protected function rotate()
             if ($fd_tmp !== false) {
                 $fd_compress = gzopen($this->file_rotated, 'w');
 
-                while (!feof($fd_tmp)) {
+                while (! feof($fd_tmp)) {
                     gzwrite($fd_compress, fread($fd_tmp, 1024 * 512));
                 }
 
@@ -75,7 +75,7 @@ protected function getRotatedFileName()
 
         $curFiles = glob($patternGlob);
 
-        for ($n = count($curFiles); $n > 0; --$n) {
+        for ($n = count($curFiles); $n > 0; $n--) {
             $file_to_move = str_replace('*', $n, $patternGlob);
 
             if (file_exists($file_to_move)) {
@@ -94,7 +94,7 @@ private function getPatternGlob($fileInfo): string
     {
         $patternGlob = $fileInfo['dirname'].'/'.$fileInfo['filename'];
 
-        if (!empty($fileInfo['extension'])) {
+        if (! empty($fileInfo['extension'])) {
             $patternGlob .= '.'.$fileInfo['extension'];
         }
 

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();
                 }

src/RotateServiceProvider.php

@@ -42,7 +42,7 @@ public function register()
 
     private function registerSchedule()
     {
-        if (!config('rotate.schedule.enable', true)) {
+        if (! config('rotate.schedule.enable', true)) {
             return;
         }
 

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/ScheduleTest.php

@@ -13,7 +13,7 @@ public function testHasSchedule()
 
     private function scheduleRegistered(): bool
     {
-        return !is_null($this->schedule());
+        return ! is_null($this->schedule());
     }
 
     private function schedule()

tests/TestCase.php

@@ -26,7 +26,7 @@ public function setUp(): void
 
         $this->cleanLogs();
 
-        if (!file_exists($this->tmpDir)) {
+        if (! file_exists($this->tmpDir)) {
             mkdir($this->tmpDir);
         }
     }
@@ -49,7 +49,7 @@ protected function cleanLogs()
             $filesToRemove = glob(dirname($fileLog).'/*');
 
             foreach ($filesToRemove as $f) {
-                if (is_file($f) && !is_dir($f)) {
+                if (is_file($f) && ! is_dir($f)) {
                     unlink($f);
                 }
             }
@@ -58,15 +58,15 @@ protected function cleanLogs()
         $filesToRemove = glob($this->tmpDir.'/*');
 
         foreach ($filesToRemove as $f) {
-            if (is_file($f) && !is_dir($f)) {
+            if (is_file($f) && ! is_dir($f)) {
                 unlink($f);
             }
         }
 
         $filesToRemove = glob($this->tmpDir.'/archive/*');
 
         foreach ($filesToRemove as $f) {
-            if (is_file($f) && !is_dir($f)) {
+            if (is_file($f) && ! is_dir($f)) {
                 unlink($f);
             }
         }