antonioribeiro / health

src/Checkers/Expression.php

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/Checkers/Docusign.php

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

src/Commands.php

Doc Comments +3 added lines, -3 removed lines

@@ -100,7 +100,7 @@ 
      * Format input to textual table.
      *
      * @param Command|null $command
-     * @param $columns
+     * @param string[] $columns
      * @param  \Illuminate\Contracts\Support\Arrayable|array $rows
      */
     private function table($command, $columns, $rows)
@@ -114,7 +114,7 @@ 
      * Write a string as information output.
      *
      * @param Command|null $command
-     * @param $string
+     * @param string $string
      */
     private function info($command, $string)
     {
@@ -127,7 +127,7 @@ 
      * Write a string as information output.
      *
      * @param Command|null $command
-     * @param $string
+     * @param string $string
      */
     private function error($command, $string)
     {

Spacing +4 added lines, -4 removed lines

@@ -58,7 +58,7 @@ 
         $columns = ['Resource', 'State', 'Message'];
 
         $rows = $this->getTargetsFomResources($this->healthService->health())
-            ->map(function ($target) {
+            ->map(function($target) {
                 return [
                     "{$target->resource->name} ({$target->display})",
                     $target->result->healthy
@@ -76,9 +76,9 @@ 
     {
         $checker = $this->healthService->getSilentChecker();
 
-        $errors = $this->getTargetsFomResources($checker()->filter(function ($resource) {
-            return ! $resource->isGlobal;
-        }))->reduce(function ($carry, $target) {
+        $errors = $this->getTargetsFomResources($checker()->filter(function($resource) {
+            return !$resource->isGlobal;
+        }))->reduce(function($carry, $target) {
             return $carry + ($target->result->healthy ? 0 : 1);
         }, 0);
 

src/Http/Controllers/Health.php

Doc Comments +2 added lines, -2 removed lines

@@ -39,7 +39,7 @@ 
     /**
      * Check and get one resource.
      *
-     * @param $slug
+     * @param string $slug
      * @return mixed
      * @throws \Exception
      */
@@ -53,7 +53,7 @@ 
     /**
      * Get all resources.
      *
-     * @return mixed
+     * @return \Illuminate\Support\Collection
      * @throws \Exception
      */
     public function allResources()

src/Service.php

Spacing +2 added lines, -2 removed lines

@@ -83,7 +83,7 @@ 
      */
     public function getSilentChecker()
     {
-        return function () {
+        return function() {
             return $this->checkResources();
         };
     }
@@ -132,7 +132,7 @@ 
      */
     public function string()
     {
-        return collect($this->health())->reduce(function ($current, $resource) {
+        return collect($this->health())->reduce(function($current, $resource) {
             return
                 $current.
                 ($current ? config('health.string.glue') : '').

src/Support/ResourceChecker.php

Doc Comments +3 added lines, -3 removed lines

@@ -152,7 +152,7 @@ 
     /**
      * Get current action.
      *
-     * @return mixed
+     * @return string
      */
     public function getCurrentAction()
     {
@@ -298,7 +298,7 @@ 
      * Get one resource.
      *
      * @param resource|Collection $resource
-     * @return resource
+     * @return Resource
      * @throws Exception
      */
     public function makeResource($resource)
@@ -313,7 +313,7 @@ 
     /**
      * Sort resources.
      *
-     * @param $resources
+     * @param Collection $resources
      * @return \Illuminate\Support\Collection
      */
     protected function sortResources($resources)

Spacing +16 added lines, -16 removed lines

@@ -79,21 +79,21 @@ 
      */
     public function checkResources($force = false)
     {
-        if (! ($resources = $this->getCachedResources($force))->isEmpty()) {
+        if (!($resources = $this->getCachedResources($force))->isEmpty()) {
             return $resources;
         }
 
-        if (! $this->allResourcesAreGood()) {
+        if (!$this->allResourcesAreGood()) {
             return $this->resources = collect();
         }
 
         $resources = $this->sortResources(
             $this->getNonGlobalResources()
-                ->each(function ($resource) {
+                ->each(function($resource) {
                     $this->checkResource($resource);
                 })
                 ->merge(
-                    $this->getGlobalResources()->each(function ($resource) {
+                    $this->getGlobalResources()->each(function($resource) {
                         return $resource->checkGlobal($this->getResources());
                     })
                 )
@@ -118,7 +118,7 @@ 
                 ? $resource
                 : $this->getResourceBySlug($resource);
 
-        $checked = $this->cache->remember($resource->slug, function () use (
+        $checked = $this->cache->remember($resource->slug, function() use (
             $resource
         ) {
             return $resource->check($this->getCurrentAction());
@@ -167,9 +167,9 @@ 
      */
     protected function allResourcesAreGood()
     {
-        return ! $this->getResources()
-            ->reject(function ($resource) {
-                return ! $resource instanceof Resource;
+        return !$this->getResources()
+            ->reject(function($resource) {
+                return !$resource instanceof Resource;
             })
             ->isEmpty();
     }
@@ -182,8 +182,8 @@ 
      */
     protected function getNonGlobalResources()
     {
-        return $this->getResources()->filter(function (Resource $resource) {
-            return ! $resource->isGlobal;
+        return $this->getResources()->filter(function(Resource $resource) {
+            return !$resource->isGlobal;
         });
     }
 
@@ -195,7 +195,7 @@ 
      */
     protected function getGlobalResources()
     {
-        return $this->getResources()->filter(function (Resource $resource) {
+        return $this->getResources()->filter(function(Resource $resource) {
             return $resource->isGlobal;
         });
     }
@@ -227,7 +227,7 @@ 
             ? $exception->getMessage()
             : static::UNKNOWN_ERROR;
 
-        if (! isset($resourceChecker)) {
+        if (!isset($resourceChecker)) {
             return [
                 null,
                 [
@@ -283,8 +283,8 @@ 
     {
         if (is_null($this->resources) || $this->resources->isEmpty()) {
             $this->resources = $this->resourceLoader->loadResources()->map(
-                function ($resource) {
-                    return $this->handleExceptions(function () use ($resource) {
+                function($resource) {
+                    return $this->handleExceptions(function() use ($resource) {
                         return $this->makeResource($resource);
                     });
                 }
@@ -319,8 +319,8 @@ 
     protected function sortResources($resources)
     {
         if ($sortBy = config('health.sort_by')) {
-            return $resources->sortBy(function ($resource) use ($sortBy) {
-                return $this->handleExceptions(function () use (
+            return $resources->sortBy(function($resource) use ($sortBy) {
+                return $this->handleExceptions(function() use (
                     $resource,
                     $sortBy
                 ) {

src/Notifications/HealthStatus.php

Spacing +1 added lines, -1 removed lines

@@ -103,7 +103,7 @@
                 config('health.notifications.from.icon_emoji')
             )
             ->content($this->getMessage())
-            ->attachment(function ($attachment) {
+            ->attachment(function($attachment) {
                 $attachment
                     ->title($this->getActionTitle(), $this->getActionLink())
                     ->content($this->result->errorMessage);

Doc Comments +4 added lines, -4 removed lines

@@ -132,8 +132,8 @@ 
     /**
      * Get the action message.
      *
-     * @param $item
-     * @return \Illuminate\Config\Repository|mixed
+     * @param HealthStatus $item
+     * @return string
      */
     private function getActionMessage($item)
     {
@@ -145,7 +145,7 @@ 
     /**
      * Get the action title.
      *
-     * @return mixed
+     * @return string
      */
     protected function getActionTitle()
     {
@@ -155,7 +155,7 @@ 
     /**
      * Get the email subject.
      *
-     * @return mixed
+     * @return string
      */
     protected function getSubject()
     {

src/Checkers/Horizon.php

Spacing +2 added lines, -2 removed lines

@@ -25,11 +25,11 @@
      */
     protected function horizonIsRunning()
     {
-        if (! $masters = app(MasterSupervisorRepository::class)->all()) {
+        if (!$masters = app(MasterSupervisorRepository::class)->all()) {
             return false;
         }
 
-        return collect($masters)->contains(function ($master) {
+        return collect($masters)->contains(function($master) {
             return $master->status === 'paused';
         })
             ? false