antonioribeiro / health

src/Checkers/Filesystem.php

Doc Comments +3 added lines, -3 removed lines

@@ -25,9 +25,9 @@
     }
 
     /**
-     * @param $name
-     * @param $content
-     * @param null $folder
+     * @param string $name
+     * @param string $content
+     * @param string $folder
      * @return string
      */
     private function temporaryFile($name, $content, $folder = null)

Spacing +2 added lines, -2 removed lines

@@ -18,7 +18,7 @@ 
                 storage_path()
             );
 
-            if (! file_exists($file)) {
+            if (!file_exists($file)) {
                 return $this->makeResult(
                     false,
                     sprintf($this->target->getErrorMessage(), $file)
@@ -47,7 +47,7 @@ 
 
         file_put_contents($file, $content);
 
-        register_shutdown_function(function () use ($file) {
+        register_shutdown_function(function() use ($file) {
             if (file_exists($file)) {
                 unlink($file);
             }

src/Checkers/NotInDebugMode.php

Doc Comments +2 added lines, -2 removed lines

@@ -139,7 +139,7 @@ 
      * Convert uptime to seconds.
      *
      * @param $date
-     * @return int
+     * @return double
      */
     protected function uptimeInSeconds($date)
     {
@@ -152,7 +152,7 @@ 
      * Make uptime message.
      *
      * @param $current
-     * @param $saved
+     * @param \Illuminate\Support\Collection $saved
      * @return string
      */
     protected function makeMessage($current, $saved = null)

Spacing +5 added lines, -5 removed lines

@@ -25,7 +25,7 @@ 
             ($ss = $this->uptimeInSeconds($this->database));
 
         return $this->makeResult(
-            ! $rebooted,
+            !$rebooted,
             $this->makeMessage($current, $this->database)
         );
     }
@@ -44,7 +44,7 @@ 
             throw new DomainException((string) $error);
         }
 
-        return (! is_array($system_string) || empty($system_string))
+        return (!is_array($system_string) || empty($system_string))
             ? ''
             : $system_string[0];
     }
@@ -69,10 +69,10 @@ 
     protected function normalizeMatches($matches)
     {
         return collect($matches)
-            ->filter(function ($item, $key) {
-                return ! is_numeric($key);
+            ->filter(function($item, $key) {
+                return !is_numeric($key);
             })
-            ->map(function ($item, $key) {
+            ->map(function($item, $key) {
                 $return = $item[0];
 
                 if (Str::startsWith($key, 'load')) {

src/Checkers/ServerUptime.php

Doc Comments +2 added lines, -2 removed lines

@@ -139,7 +139,7 @@ 
      * Convert uptime to seconds.
      *
      * @param $date
-     * @return int
+     * @return double
      */
     protected function uptimeInSeconds($date)
     {
@@ -152,7 +152,7 @@ 
      * Make uptime message.
      *
      * @param $current
-     * @param $saved
+     * @param \Illuminate\Support\Collection $saved
      * @return string
      */
     protected function makeMessage($current, $saved = null)

Spacing +5 added lines, -5 removed lines

@@ -25,7 +25,7 @@ 
             ($ss = $this->uptimeInSeconds($this->database));
 
         return $this->makeResult(
-            ! $rebooted,
+            !$rebooted,
             $this->makeMessage($current, $this->database)
         );
     }
@@ -44,7 +44,7 @@ 
             throw new DomainException((string) $error);
         }
 
-        return (! is_array($system_string) || empty($system_string))
+        return (!is_array($system_string) || empty($system_string))
             ? ''
             : $system_string[0];
     }
@@ -69,10 +69,10 @@ 
     protected function normalizeMatches($matches)
     {
         return collect($matches)
-            ->filter(function ($item, $key) {
-                return ! is_numeric($key);
+            ->filter(function($item, $key) {
+                return !is_numeric($key);
             })
-            ->map(function ($item, $key) {
+            ->map(function($item, $key) {
                 $return = $item[0];
 
                 if (Str::startsWith($key, 'load')) {

src/Support/helpers.php

Doc Comments +3 added lines, -2 removed lines

@@ -57,7 +57,7 @@ 
      * Check if string is absolute path.
      *
      * @param $path
-     * @return string
+     * @return boolean
      */
     function is_absolute_path($path)
     {
@@ -84,6 +84,7 @@ 
     /**
      * Convert bytes to human readable.
      *
+     * @param double $bytes
      * @return string
      */
     function bytes_to_human($bytes)
@@ -102,7 +103,7 @@ 
     /**
      * Convert bytes to human readable.
      *
-     * @return string
+     * @return double
      */
     function human_to_bytes($str)
     {

Spacing +7 added lines, -7 removed lines

@@ -2,7 +2,7 @@ 
 
 use PragmaRX\Health\Service;
 
-if (! function_exists('instantiate')) {
+if (!function_exists('instantiate')) {
     /**
      * Instantiate a class.
      *
@@ -22,7 +22,7 @@ 
     }
 }
 
-if (! function_exists('package_dir')) {
+if (!function_exists('package_dir')) {
     /**
      * Get the package root directory.
      *
@@ -36,7 +36,7 @@ 
     }
 }
 
-if (! function_exists('package_resources_dir')) {
+if (!function_exists('package_resources_dir')) {
     /**
      * Get package resources directory.
      *
@@ -52,7 +52,7 @@ 
     }
 }
 
-if (! function_exists('is_absolute_path')) {
+if (!function_exists('is_absolute_path')) {
     /**
      * Check if string is absolute path.
      *
@@ -80,7 +80,7 @@ 
     }
 }
 
-if (! function_exists('bytes_to_human')) {
+if (!function_exists('bytes_to_human')) {
     /**
      * Convert bytes to human readable.
      *
@@ -98,7 +98,7 @@ 
     }
 }
 
-if (! function_exists('human_to_bytes')) {
+if (!function_exists('human_to_bytes')) {
     /**
      * Convert bytes to human readable.
      *
@@ -126,7 +126,7 @@ 
     }
 }
 
-if (! function_exists('ip_address_from_hostname')) {
+if (!function_exists('ip_address_from_hostname')) {
     function ip_address_from_hostname($host)
     {
         if (

src/Checkers/Cache.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
 
     private function getChecker()
     {
-        return function () {
+        return function() {
             return 'DUMMY DATA';
         };
     }

src/Checkers/Ping.php

Doc Comments +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
 
     /**
      * @param $hostname
-     * @return mixed
+     * @return string
      */
     protected function hosnameAndIp($hostname, $ipAdress)
     {

Spacing +2 added lines, -2 removed lines

@@ -90,7 +90,7 @@ 
                 ' -n 1 -i '.
                 $ttl.
                 ' -w '.
-                $timeout * 1000 .
+                $timeout * 1000.
                 ' '.
                 $host;
         } elseif (strtoupper(PHP_OS) === 'DARWIN') {
@@ -126,7 +126,7 @@ 
         $output = array_values(array_filter($output));
 
         // If the result line in the output is not empty, parse it.
-        if (! empty($output[1])) {
+        if (!empty($output[1])) {
             // Search for a 'time' value in the result line.
             $response = preg_match(
                 "/time(?:=|<)(?<time>[\.0-9]+)(?:|\s)ms/",

src/Checkers/DiskSpace.php

Doc Comments +3 added lines

@@ -30,6 +30,9 @@
         return $this->makeHealthyResult();
     }
 
+    /**
+     * @param double $free
+     */
     public function isEnough($free, $minimum)
     {
         return $free > human_to_bytes($minimum);

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
     {
         $free = disk_free_space($this->target->path);
 
-        if (! $this->isEnough($free, $this->target->minimum)) {
+        if (!$this->isEnough($free, $this->target->minimum)) {
             return $this->makeResult(
                 false,
                 sprintf(

src/Checkers/PortCheck.php

Doc Comments +2 added lines, -1 removed lines

@@ -28,7 +28,8 @@
      * Get hostname and IP.
      *
      * @param $hostname
-     * @return mixed
+     * @param boolean $ipAdress
+     * @return string
      */
     protected function hosnameAndIp($hostname, $ipAdress)
     {

Spacing +1 added lines, -1 removed lines

@@ -74,7 +74,7 @@
      */
     protected function portIsNotConnectable($ipAddress)
     {
-        return ! $this->portCheck(
+        return !$this->portCheck(
             $ipAddress,
             $this->target->port,
             $this->target->timeout ?? 1

src/Checkers/Process.php

Doc Comments +9 added lines, -3 removed lines

@@ -27,12 +27,18 @@ 
         return $this->makeHealthyResult();
     }
 
+    /**
+     * @param integer|null $processes
+     */
     private function checkMinMax($processes)
     {
         return $this->buildMessage('minimum', $processes)
             ?: $this->buildMessage('maximum', $processes);
     }
 
+    /**
+     * @param string $type
+     */
     private function buildMessage($type, $processes)
     {
         $instances = $this->target->instances;
@@ -75,7 +81,7 @@ 
 
     /**
      * @param $file
-     * @return bool
+     * @return integer
      */
     private function checkPidFileExistence($file)
     {
@@ -90,7 +96,7 @@ 
 
     /**
      * @param $file
-     * @return bool
+     * @return boolean|null
      */
     private function checkPidFileLockState($file)
     {
@@ -112,7 +118,7 @@ 
     }
 
     /**
-     * @return bool
+     * @return integer
      */
     private function processPidFileIsLocked()
     {

Spacing +3 added lines, -3 removed lines

@@ -20,7 +20,7 @@ 
     {
         $message = $this->checkMinMax($this->getProcessesRunningCount());
 
-        if (! empty($message)) {
+        if (!empty($message)) {
             return $this->makeResult(false, $message);
         }
 
@@ -37,7 +37,7 @@ 
     {
         $instances = $this->target->instances;
 
-        if (! $count = (int) $instances[$type]['count']) {
+        if (!$count = (int) $instances[$type]['count']) {
             return '';
         }
 
@@ -101,7 +101,7 @@ 
 
             fclose($filePointer);
 
-            if (! $locked) {
+            if (!$locked) {
                 throw new DomainException(
                     sprintf($this->target->pid_file_missing_not_locked, $file)
                 );