antonioribeiro / health

src/Checkers/Http.php

Doc Comments +3 added lines, -3 removed lines

@@ -89,7 +89,7 @@ 
      *
      * @param $url
      * @param $ssl
-     * @return mixed|\Psr\Http\Message\ResponseInterface
+     * @return \Psr\Http\Message\ResponseInterface
      * @throws \GuzzleHttp\Exception\GuzzleException
      */
     private function fetchResponse($url, $ssl)
@@ -160,7 +160,7 @@ 
      * Make a url with a proper scheme.
      *
      * @param $url
-     * @param $secure
+     * @param boolean $secure
      * @return mixed
      */
     private function makeUrlWithScheme($url, $secure)
@@ -188,7 +188,7 @@ 
      * Send a request and get the result.
      *
      * @param $url
-     * @param $ssl
+     * @param boolean $ssl
      * @return bool
      * @internal param $response
      */

Spacing +3 added lines, -3 removed lines

@@ -45,7 +45,7 @@ 
                     $this->secure
                 );
 
-                if (! $healthy) {
+                if (!$healthy) {
                     return $this->makeResult($healthy, $message);
                 }
             }
@@ -179,7 +179,7 @@ 
      */
     private function onStatsCallback()
     {
-        return function (TransferStats $stats) {
+        return function(TransferStats $stats) {
             $this->totalTime = $stats->getTransferTime();
         };
     }
@@ -196,7 +196,7 @@ 
     {
         return
             $this->fetchResponse($url, $ssl)->getStatusCode() == 200 &&
-            ! $this->requestTimedout();
+            !$this->requestTimedout();
     }
 
     /**

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

src/Support/Cache.php

Doc Comments +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
      * Flush cache.
      *
      * @param bool $force
-     * @param null $key
+     * @param string $key
      */
     public function flush($force = false, $key = null)
     {

Spacing +2 added lines, -2 removed lines

@@ -29,7 +29,7 @@ 
      */
     public function flush($force = false, $key = null)
     {
-        if (! isset($this->flushed[$key]) && ($force || $this->needsToFlush())) {
+        if (!isset($this->flushed[$key]) && ($force || $this->needsToFlush())) {
             try {
                 $this->forceFlush($key);
             } catch (Exception $exception) {
@@ -123,7 +123,7 @@ 
      */
     public function remember($key, \Closure $callback)
     {
-        if (! $this->cacheIsEnabled()) {
+        if (!$this->cacheIsEnabled()) {
             return $callback();
         }
 

src/Support/ResourceLoader.php

Doc Comments +3 added lines, -2 removed lines

@@ -33,7 +33,7 @@ 
     /**
      * Get enabled resources.
      *
-     * @param $resources
+     * @param \Illuminate\Support\Collection $resources
      * @return \Illuminate\Support\Collection
      * @throws \Exception
      */
@@ -105,6 +105,7 @@ 
     /**
      * Replace executable code.
      *
+     * @param \Illuminate\Support\Collection $files
      * @return \Illuminate\Support\Collection
      */
     private function replaceExecutableCode($files)
@@ -175,7 +176,7 @@ 
     /**
      * Sanitize resources.
      *
-     * @param $resources
+     * @param \Illuminate\Support\Collection $resources
      * @return \Illuminate\Support\Collection
      */
     private function sanitizeResources($resources)

Spacing +7 added lines, -7 removed lines

@@ -42,7 +42,7 @@ 
         if (
             is_array($filters = config($configKey = 'health.resources.enabled'))
         ) {
-            return collect($resources)->filter(function ($resource, $name) use (
+            return collect($resources)->filter(function($resource, $name) use (
                 $filters
             ) {
                 foreach ($filters as $filter) {
@@ -79,7 +79,7 @@ 
      */
     public function load()
     {
-        if (! empty($this->resources)) {
+        if (!empty($this->resources)) {
             return $this->resources;
         }
 
@@ -97,7 +97,7 @@ 
     {
         return $this->replaceExecutableCode(
             $this->loadResourcesFiles()
-        )->mapWithKeys(function ($value, $key) {
+        )->mapWithKeys(function($value, $key) {
             return [$this->removeExtension($key) => $value];
         });
     }
@@ -109,9 +109,9 @@ 
      */
     private function replaceExecutableCode($files)
     {
-        return $files->map(function ($item) {
+        return $files->map(function($item) {
             return collect($item)
-                ->map(function ($value) {
+                ->map(function($value) {
                     return (new Resolver())->recursivelyFindAndReplaceExecutableCode(
                         $value
                     );
@@ -180,8 +180,8 @@ 
      */
     private function sanitizeResources($resources)
     {
-        return $resources->map(function ($resource) {
-            return collect($resource)->mapWithKeys(function ($value, $key) {
+        return $resources->map(function($resource) {
+            return collect($resource)->mapWithKeys(function($value, $key) {
                 return [$this->sanitizeKey($key) => $value];
             });
         });

src/Support/Traits/ToArray.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
      */
     public function __getToArrayCallBack()
     {
-        return function ($subject) {
+        return function($subject) {
             static::$__depth++;
 
             if ($subject instanceof Arrayable) {

src/Checkers/Health.php

Spacing +2 added lines, -2 removed lines

@@ -43,7 +43,7 @@ 
      */
     private function computeHealthForAllTargets($resource)
     {
-        return $resource->targets->reduce(function ($carry, $target) {
+        return $resource->targets->reduce(function($carry, $target) {
             return $target->result->healthy;
         }, true);
     }
@@ -55,7 +55,7 @@ 
      */
     protected function isHealthy()
     {
-        $healthy = $this->target->resource->resources->reduce(function (
+        $healthy = $this->target->resource->resources->reduce(function(
             $carry,
             $item
         ) {

src/Checkers/Docusign.php

Doc Comments +7 added lines, -1 removed lines

@@ -37,6 +37,9 @@ 
         return ! class_exists(ApiClient::class);
     }
 
+    /**
+     * @param \DocuSign\eSign\Model\LoginInformation $login
+     */
     private function getAccountIdFromLogin($login)
     {
         return $login->getLoginAccounts()[0]->getAccountId();
@@ -80,7 +83,7 @@ 
     }
 
     /**
-     * @param $config
+     * @param ApiClient $config
      * @return \DocuSign\eSign\Model\LoginInformation
      */
     protected function getLoginInformation($config)
@@ -106,6 +109,9 @@ 
         );
     }
 
+    /**
+     * @return string
+     */
     private function makeFileName($file)
     {
         if (is_absolute_path($file)) {

Spacing +2 added lines, -2 removed lines

@@ -20,7 +20,7 @@ 
             return $this->makeResult(false, $this->target->notInstalledMessage);
         }
 
-        if (! $this->login()) {
+        if (!$this->login()) {
             throw new DomainException(
                 'Unable to authenticate to the DocuSign Api'
             );
@@ -31,7 +31,7 @@ 
 
     private function docusignIsNotInstalled()
     {
-        return ! class_exists(ApiClient::class);
+        return !class_exists(ApiClient::class);
     }
 
     private function getAccountIdFromLogin($login)

src/Support/Traits/Database.php

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
      */
     public function __load()
     {
-        if (! file_exists($file = $this->getDatabaseFileName())) {
+        if (!file_exists($file = $this->getDatabaseFileName())) {
             return collect();
         }
 

Doc Comments +3 added lines

@@ -47,6 +47,9 @@
         return config('health.database.enabled');
     }
 
+    /**
+     * @param \PragmaRX\Health\Support\Target $target
+     */
     protected function saveResultsToDatabase($target, $result)
     {
         $model = config('health.database.model', HealthCheck::class);