antonioribeiro / health

src/Checkers/Base.php

Spacing +3 added lines, -3 removed lines

@@ -55,7 +55,7 @@ 
     {
         $dir = dirname($fileName);
 
-        if (! file_exists($dir)) {
+        if (!file_exists($dir)) {
             mkdir($dir, 0775, true);
         }
     }
@@ -142,7 +142,7 @@ 
      */
     public function load()
     {
-        if (! file_exists($file = $this->getFileName())) {
+        if (!file_exists($file = $this->getFileName())) {
             return collect();
         }
 
@@ -160,7 +160,7 @@ 
             $data = $this->database->toArray();
         }
 
-        if (! is_array($data)) {
+        if (!is_array($data)) {
             $data = $data->toArray();
         }
 

Doc Comments +2 added lines, -2 removed lines

@@ -33,7 +33,7 @@ 
     /**
      * Create base directory for files.
      *
-     * @param $fileName
+     * @param string $fileName
      */
     protected function makeDir($fileName)
     {
@@ -69,7 +69,7 @@ 
     /**
      * Make a result from an exception.
      *
-     * @param $exception
+     * @param \Exception $exception
      * @return Result
      */
     protected function makeResultFromException($exception)

src/Checkers/DirectoryAndFilePresence.php

Spacing +7 added lines, -7 removed lines

@@ -55,13 +55,13 @@ 
         $messages = [];
 
         $result = collect($this->getFiles())
-            ->map(function ($files, $type) use (&$messages) {
+            ->map(function($files, $type) use (&$messages) {
                 $isGood = true;
 
                 $files = collect($files);
 
                 foreach ($files as $file) {
-                    if (! is_null($file)) {
+                    if (!is_null($file)) {
                         foreach ($this->getCheckers($type) as $checker) {
                             if (is_string($message = $checker($file))) {
                                 $messages[] = $message;
@@ -73,7 +73,7 @@ 
 
                 return $isGood;
             })
-            ->filter(function ($value) {
+            ->filter(function($value) {
                 return $value === false;
             });
 
@@ -97,7 +97,7 @@ 
      */
     public function buildFileExistsChecker()
     {
-        return function ($file) {
+        return function($file) {
             return $this->fileExists($file);
         };
     }
@@ -109,7 +109,7 @@ 
      */
     public function buildFileDoesNotExistsChecker()
     {
-        return function ($file) {
+        return function($file) {
             return $this->fileDoesNotExists($file);
         };
     }
@@ -121,7 +121,7 @@ 
      */
     public function buildIsDirectoryChecker()
     {
-        return function ($file) {
+        return function($file) {
             return $this->isDirectory($file);
         };
     }
@@ -176,7 +176,7 @@ 
      */
     public function fileDoesNotExists($file)
     {
-        if (! file_exists($file)) {
+        if (!file_exists($file)) {
             return true;
         }
 

src/Checkers/Expression.php

Doc Comments +3 added lines

@@ -35,6 +35,9 @@
         return preg_match("|{$this->target->shouldReturn}|", $expressionResult);
     }
 
+    /**
+     * @return string
+     */
     public function executeExpression($expression)
     {
         return eval("return {$expression} ;");

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
         }
 
         if ($this->target->shouldReturn === false) {
-            return ! $expressionResult;
+            return !$expressionResult;
         }
 
         return preg_match("|{$this->target->shouldReturn}|", $expressionResult);

src/Support/Target.php

Doc Comments +3 added lines, -2 removed lines

@@ -37,6 +37,7 @@ 
 
     /**
      * @param \Exception|\Throwable
+     * @param \Throwable $exception
      */
     protected function exceptionResult($exception)
     {
@@ -49,7 +50,7 @@ 
     /**
      * Target factory.
      *
-     * @param $resource
+     * @param Resource $resource
      * @param $data
      * @return Target
      */
@@ -172,7 +173,7 @@ 
     /**
      * Get result error message.
      *
-     * @return mixed
+     * @return null|string
      */
     public function getErrorMessage()
     {

src/database/migrations/2018_09_09_000001_create_table_health_checks.php

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
      */
     public function up()
     {
-        Schema::create('health_checks', function (Blueprint $table) {
+        Schema::create('health_checks', function(Blueprint $table) {
             $table->increments('id');
 
             $table->string('resource_name');

src/Checkers/Mail.php

Spacing +1 added lines, -1 removed lines

@@ -69,7 +69,7 @@
      */
     private function sendMail()
     {
-        IlluminateMail::send($this->target->view, [], function ($message) {
+        IlluminateMail::send($this->target->view, [], function($message) {
             $fromAddress = array_get($this->target->config, 'from.address');
 
             $message->returnPath($fromAddress);

src/Support/Traits/ImportProperties.php

Spacing +2 added lines, -2 removed lines

@@ -11,10 +11,10 @@
      */
     public function importProperties($data)
     {
-        $data->each(function ($value, $key) {
+        $data->each(function($value, $key) {
             $key = camel_case($key);
 
-            if (! property_exists($this, $key)) {
+            if (!property_exists($this, $key)) {
                 $this->$key = $value;
             }
         });

src/Checkers/ServerLoad.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
             $current['load_5'] > $this->target->maxLoad['load_5'] ||
             $current['load_15'] > $this->target->maxLoad['load_15'];
 
-        return $this->makeResult(! $inTrouble, $this->makeMessage($current));
+        return $this->makeResult(!$inTrouble, $this->makeMessage($current));
     }
 
     protected function makeMessage($current, $saved = null)

src/Checkers/Database.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
 
     protected function findFirstModel()
     {
-        collect($this->target->models)->each(function ($model) {
+        collect($this->target->models)->each(function($model) {
             instantiate($model)->first();
         });