antonioribeiro / health

src/Checkers/Queue.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 namespace PragmaRX\Health\Checkers;
 
-use Illuminate\Queue\Worker;
 use Illuminate\Queue\WorkerOptions;
 use PragmaRX\Health\Support\Result;
 

src/Checkers/Broadcasting.php

Spacing +5 added lines, -5 removed lines

@@ -14,7 +14,7 @@ 
 
     protected function bootRouter()
     {
-        $this->target->routes->each(function ($route, $name) {
+        $this->target->routes->each(function($route, $name) {
             $this->registerRoute($route, $name);
         });
     }
@@ -30,7 +30,7 @@ 
 
         $this->bootRouter();
 
-        $isHealthy = ! $this->pingTimedout();
+        $isHealthy = !$this->pingTimedout();
 
         $this->createPing();
 
@@ -95,7 +95,7 @@ 
      */
     public function pong($secret)
     {
-        $this->database = $this->database->map(function ($item) use ($secret) {
+        $this->database = $this->database->map(function($item) use ($secret) {
             if ($item['secret'] == $secret) {
                 $item['ponged_at'] = Carbon::now();
             }
@@ -115,10 +115,10 @@ 
     {
         $timedout = false;
 
-        $this->database = $this->database->filter(function ($item) use (
+        $this->database = $this->database->filter(function($item) use (
             &$timedout
         ) {
-            if (! $item['ponged_at']) {
+            if (!$item['ponged_at']) {
                 if (
                     Carbon::now()->diffInSeconds(
                         $this->parseDate($item['pinged_at'])

src/Support/Traits/Routing.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
             'middleware' => isset($route['middleware']) ? $route['middleware'] : [],
         ];
 
-        $this->getRouter()->group($attributes, function () use ($route, $name) {
+        $this->getRouter()->group($attributes, function() use ($route, $name) {
             $action = isset($route['controller'])
                 ? "{$route['controller']}@{$route['action']}"
                 : $route['action'];

src/Support/Resource.php

Doc Comments +5 added lines, -5 removed lines

@@ -87,7 +87,7 @@ 
      * Resource factory.
      *
      * @param Collection $data
-     * @return resource
+     * @return Resource
      * @throws \Exception
      */
     public static function factory(Collection $data)
@@ -176,7 +176,7 @@ 
      * Check all targets for a resource.
      *
      * @param string $action
-     * @return resource
+     * @return Resource
      */
     public function check($action = 'resource')
     {
@@ -195,7 +195,7 @@ 
      * Check global resources.
      *
      * @param $resources
-     * @return resource
+     * @return Resource
      */
     public function checkGlobal($resources)
     {
@@ -279,7 +279,7 @@ 
      * Set current action.
      *
      * @param string $currentAction
-     * @return resource
+     * @return Resource
      */
     public function setCurrentAction(string $currentAction)
     {
@@ -292,7 +292,7 @@ 
      * Resources setter.
      *
      * @param $resources
-     * @return resource
+     * @return Resource
      */
     protected function setResources($resources)
     {

Spacing +9 added lines, -9 removed lines

@@ -146,12 +146,12 @@ 
         $current = collect();
 
         $targets = $targets
-            ->map(function (Collection $targetList) {
-                return $targetList->map(function ($target, $name) {
+            ->map(function(Collection $targetList) {
+                return $targetList->map(function($target, $name) {
                     return Target::factory($this, $target, $name);
                 });
             })
-            ->reduce(function ($current, $targetList) {
+            ->reduce(function($current, $targetList) {
                 foreach ($targetList as $target) {
                     $current[] = $target;
                 }
@@ -181,7 +181,7 @@ 
      */
     public function check($action = 'resource')
     {
-        $this->setCurrentAction($action)->targets->each(function (
+        $this->setCurrentAction($action)->targets->each(function(
             Target $target
         ) {
             $target->check($target);
@@ -210,14 +210,14 @@ 
      */
     public function isHealthy()
     {
-        return $this->targets->reduce(function ($carry, $target) {
+        return $this->targets->reduce(function($carry, $target) {
             return $carry && $target->result->healthy;
         }, true);
     }
 
     protected function keysToCamel($array)
     {
-        return collect($array)->mapWithKeys(function ($item, $key) {
+        return collect($array)->mapWithKeys(function($item, $key) {
             return [Str::camel($key) => $item];
         });
     }
@@ -239,7 +239,7 @@ 
      */
     protected function sendNotifications()
     {
-        return collect(config('health.notifications.channels'))->each(function (
+        return collect(config('health.notifications.channels'))->each(function(
             $channel
         ) {
             try {
@@ -258,9 +258,9 @@ 
     protected function canNotify()
     {
         return
-            ! $this->notified &&
+            !$this->notified &&
             $this->notificationsAreEnabled() &&
-            ! $this->isHealthy();
+            !$this->isHealthy();
     }
 
     /**

src/Support/Traits/ImportProperties.php

Spacing +2 added lines, -2 removed lines

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

src/Listeners/NotifyHealthIssue.php

Spacing +3 added lines, -3 removed lines

@@ -16,7 +16,7 @@ 
     private function getNotifiableUsers()
     {
         return collect(config('health.notifications.users.emails'))->map(
-            function ($item) {
+            function($item) {
                 $model = instantiate(
                     config('health.notifications.users.model')
                 );
@@ -44,8 +44,8 @@ 
             $notifierClass = HealthStatus::class;
         }
         try {
-            $event->failure->targets->each(function ($target) use ($event, $notifierClass) {
-                if (! $target->result->healthy) {
+            $event->failure->targets->each(function($target) use ($event, $notifierClass) {
+                if (!$target->result->healthy) {
                     Notification::send(
                         $this->getNotifiableUsers(),
                         $notifierClass->newInstance($target, $event->channel)

src/Checkers/Mail.php

Spacing +1 added lines, -1 removed lines

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

src/ServiceProvider.php

Spacing +7 added lines, -7 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);
         };
 
@@ -259,9 +259,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);
                 });
             }
@@ -292,7 +292,7 @@ 
      */
     private function registerRoutes()
     {
-        collect(($routes = $this->getRoutes()))->each(function ($route) {
+        collect(($routes = $this->getRoutes()))->each(function($route) {
             $this->registerRoute($route);
         });
 
@@ -317,7 +317,7 @@ 
 
         $this->app->singleton(
             'pragmarx.health.commands',
-            function () {
+            function() {
                 return $this->instantiateCommands();
             }
         );

src/Support/Timer.php

Doc Comments +3 added lines

@@ -15,6 +15,9 @@
             : \PHP_Timer::start();
     }
 
+    /**
+     * @return double
+     */
     public static function stop()
     {
         return class_exists('SebastianBergmann\Timer\Timer')

Spacing +2 added lines, -2 removed lines

@@ -43,7 +43,7 @@ 
 
     public static function getSBInstance()
     {
-        if (! empty(static::$sbTimer)) {
+        if (!empty(static::$sbTimer)) {
             return static::$sbTimer;
         }
 
@@ -52,7 +52,7 @@ 
 
     public static function getStaticMethodNames()
     {
-        return collect((new \ReflectionClass(SBTimer::class))->getMethods(\ReflectionMethod::IS_STATIC))->map(function ($method) {
+        return collect((new \ReflectionClass(SBTimer::class))->getMethods(\ReflectionMethod::IS_STATIC))->map(function($method) {
             return $method->name;
         });
     }