antonioribeiro / health

src/Listeners/NotifyHealthIssue.php

Doc Comments +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
 class NotifyHealthIssue
 {
     /**
-     * @return static
+     * @return \Illuminate\Support\Collection
      */
     private function getNotifiableUsers()
     {

Spacing +2 added lines, -2 removed lines

@@ -14,7 +14,7 @@ 
     private function getNotifiableUsers()
     {
         return collect(config('health.notifications.users.emails'))->map(
-            function ($item) {
+            function($item) {
                 $model = instantiate(
                     config('health.notifications.users.model')
                 );
@@ -35,7 +35,7 @@ 
     public function handle(RaiseHealthIssue $event)
     {
         try {
-            $event->failure->targets->each(function ($target) use ($event) {
+            $event->failure->targets->each(function($target) use ($event) {
                 if (!$target->result->healthy) {
                     Notification::send(
                         $this->getNotifiableUsers(),

src/ServiceProvider.php

Doc Comments +2 added lines, -2 removed lines

@@ -142,7 +142,7 @@ 
     /**
      * Get the resource checker closure for instantiation.
      *
-     * @param $resourceLoader
+     * @param ResourceLoader $resourceLoader
      * @param $cache
      * @return \Closure
      */
@@ -354,7 +354,7 @@ 
     /**
      * Get the services provided by the provider.
      *
-     * @return array
+     * @return string[]
      */
     public function provides()
     {

Spacing +8 added lines, -8 removed lines

@@ -130,7 +130,7 @@ 
 
         $cache = call_user_func($this->cacheClosure);
 
-        $this->healthServiceClosure = function () use (
+        $this->healthServiceClosure = function() use (
             $resourceChecker,
             $cache
         ) {
@@ -162,7 +162,7 @@ 
      */
     private function getCacheClosure()
     {
-        $cacheClosure = function () {
+        $cacheClosure = function() {
             return new Cache();
         };
 
@@ -178,7 +178,7 @@ 
      */
     private function getResourceCheckerClosure($resourceLoader, $cache)
     {
-        $resourceCheckerInstance = function () use ($resourceLoader, $cache) {
+        $resourceCheckerInstance = function() use ($resourceLoader, $cache) {
             return new ResourceChecker($resourceLoader, $cache);
         };
 
@@ -257,9 +257,9 @@ 
 
     private function registerResourcesRoutes()
     {
-        collect($this->resourceLoader->getResources())->each(function ($item) {
+        collect($this->resourceLoader->getResources())->each(function($item) {
             if (isset($item['routes'])) {
-                collect($item['routes'])->each(function ($route, $key) {
+                collect($item['routes'])->each(function($route, $key) {
                     $this->registerRoute($route, $key);
                 });
             }
@@ -273,11 +273,11 @@ 
     {
         $commands = $this->commands;
 
-        Artisan::command('health:panel', function () use ($commands) {
+        Artisan::command('health:panel', function() use ($commands) {
             $commands->panel($this);
         })->describe('Show all resources and their current health states.');
 
-        Artisan::command('health:check', function () use ($commands) {
+        Artisan::command('health:check', function() use ($commands) {
             $commands->check($this);
         })->describe('Check resources health and send error notifications.');
     }
@@ -295,7 +295,7 @@ 
      */
     private function registerRoutes()
     {
-        collect($routes = $this->getRoutes())->each(function ($route) {
+        collect($routes = $this->getRoutes())->each(function($route) {
             $this->registerRoute($route);
         });
 

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

@@ -24,7 +24,7 @@ 
             ($ss = $this->uptimeInSeconds($this->database));
 
         return $this->makeResult(
-            ! $rebooted,
+            !$rebooted,
             $this->makeMessage($current, $this->database)
         );
     }
@@ -43,7 +43,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];
     }
@@ -68,10 +68,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 (starts_with($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

@@ -24,7 +24,7 @@ 
             ($ss = $this->uptimeInSeconds($this->database));
 
         return $this->makeResult(
-            ! $rebooted,
+            !$rebooted,
             $this->makeMessage($current, $this->database)
         );
     }
@@ -43,7 +43,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];
     }
@@ -68,10 +68,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 (starts_with($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 +20 added lines, -20 removed lines

@@ -59,7 +59,7 @@ 
      */
     protected function hosnameAndIp($hostname, $ipAdress)
     {
-        return $hostname . ($hostname != $ipAdress ? " ({$ipAdress})" : '');
+        return $hostname.($hostname != $ipAdress ? " ({$ipAdress})" : '');
     }
 
     /**
@@ -82,35 +82,35 @@ 
         if (strtoupper(substr(PHP_OS, 0, 3)) === 'WIN') {
             // -n = number of pings; -i = ttl; -w = timeout (in milliseconds).
             $exec_string =
-                $this->pingBin .
-                ' -n 1 -i ' .
-                $ttl .
-                ' -w ' .
-                ($timeout * 1000) .
-                ' ' .
+                $this->pingBin.
+                ' -n 1 -i '.
+                $ttl.
+                ' -w '.
+                ($timeout * 1000).
+                ' '.
                 $host;
         } elseif (strtoupper(PHP_OS) === 'DARWIN') {
             // Exec string for Darwin based systems (OS X).
             // -n = numeric output; -c = number of pings; -m = ttl; -t = timeout.
             $exec_string =
-                $this->pingBin .
-                ' -n -c 1 -m ' .
-                $ttl .
-                ' -t ' .
-                $timeout .
-                ' ' .
+                $this->pingBin.
+                ' -n -c 1 -m '.
+                $ttl.
+                ' -t '.
+                $timeout.
+                ' '.
                 $host;
         } else {
             // Exec string for other UNIX-based systems (Linux).
             // -n = numeric output; -c = number of pings; -t = ttl; -W = timeout
             $exec_string =
-                $this->pingBin .
-                ' -n -c 1 -t ' .
-                $ttl .
-                ' -W ' .
-                $timeout .
-                ' ' .
-                $host .
+                $this->pingBin.
+                ' -n -c 1 -t '.
+                $ttl.
+                ' -W '.
+                $timeout.
+                ' '.
+                $host.
                 ' 2>&1';
         }
 

src/Checkers/Artisan.php

Doc Comments +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
     /**
      * Check resource.
      *
-     * @return bool
+     * @return \PragmaRX\Health\Support\Result
      */
     public function check()
     {