understand / understand-laravel5

src/Understand/UnderstandLaravel5/UnderstandLaravel5ServiceProvider.php

Doc Comments +2 added lines, -2 removed lines

@@ -122,8 +122,8 @@
     /**
      * Detect Laravel version
      * 
-     * @param array $versions
-     * @return type
+     * @param string[] $versions
+     * @return boolean
      */
     protected function detectLaravelVersion(array $versions)
     {

Indentation +23 added lines, -23 removed lines

@@ -9,20 +9,20 @@ 
 class UnderstandLaravel5ServiceProvider extends UnderstandServiceProvider
 {
 
-	/**
-	 * Indicates if loading of the provider is deferred.
-	 *
-	 * @var bool
-	 */
-	protected $defer = false;
+    /**
+     * Indicates if loading of the provider is deferred.
+     *
+     * @var bool
+     */
+    protected $defer = false;
 
-	/**
-	 * Bootstrap the application events.
-	 *
-	 * @return void
-	 */
-	public function boot()
-	{
+    /**
+     * Bootstrap the application events.
+     *
+     * @return void
+     */
+    public function boot()
+    {
         $configPath = __DIR__ . '/../../config/understand-laravel.php';
         $this->publishes([$configPath => config_path('understand-laravel.php')], 'config');
         $enabled = $this->app['config']->get('understand-laravel.enabled');
@@ -36,23 +36,23 @@ 
         {
             $this->listenQueryEvents();
         }
-	}
+    }
 
-	/**
-	 * Register the service provider.
-	 *
-	 * @return void
-	 */
-	public function register()
-	{
-		$this->registerConfig();
+    /**
+     * Register the service provider.
+     *
+     * @return void
+     */
+    public function register()
+    {
+        $this->registerConfig();
         $this->registerFieldProvider();
         $this->registerDataCollector();
         $this->registerTokenProvider();
         $this->registerLogger();
         $this->registerExceptionEncoder();
         $this->registerEventLoggers();
-	}
+    }
 
     /**
      * Register field provider

Spacing +3 added lines, -3 removed lines

@@ -23,7 +23,7 @@ 
 	 */
 	public function boot()
 	{
-        $configPath = __DIR__ . '/../../config/understand-laravel.php';
+        $configPath = __DIR__.'/../../config/understand-laravel.php';
         $this->publishes([$configPath => config_path('understand-laravel.php')], 'config');
         $enabled = $this->app['config']->get('understand-laravel.enabled');
 
@@ -235,13 +235,13 @@ 
         }
 
         // `\Log::info`, `\Log::debug` and NOT `\Exception` or `\Throwable`
-        if (in_array($level, ['info', 'debug']) && ! ($message instanceof Exception || $message instanceof Throwable))
+        if (in_array($level, ['info', 'debug']) && !($message instanceof Exception || $message instanceof Throwable))
         {
             $this->app['understand.eventLogger']->logEvent($level, $message, $context);
         }
         // `\Log::notice`, `\Log::warning`, `\Log::error`, `\Log::critical`, `\Log::alert`, `\Log::emergency` and `\Exception`, `\Throwable`
         // '5.5', '5.6', '5.7', '5.8'
-        else if ( ! $this->detectLaravelVersion(['5.0', '5.1', '5.2', '5.3', '5.4']) && isset($context['exception']) && ($context['exception'] instanceof Exception || $context['exception'] instanceof Throwable))
+        else if (!$this->detectLaravelVersion(['5.0', '5.1', '5.2', '5.3', '5.4']) && isset($context['exception']) && ($context['exception'] instanceof Exception || $context['exception'] instanceof Throwable))
         {
             $exception = $context['exception'];
             unset($context['exception']);

src/Understand/UnderstandLaravel5/UnderstandLumenServiceProvider.php

Doc Comments +2 added lines, -2 removed lines

@@ -129,8 +129,8 @@
     /**
      * Detect Lumen version
      *
-     * @param array $versions
-     * @return type
+     * @param string[] $versions
+     * @return boolean
      */
     protected function detectLumenVersion(array $versions)
     {

Unused Use Statements -3 removed lines

@@ -3,11 +3,8 @@
 namespace Understand\UnderstandLaravel5;
 
 use Exception;
-use Illuminate\Foundation\AliasLoader;
 use Illuminate\Support\Str;
 use Understand\UnderstandLaravel5\Handlers\MonologHandler;
-use UnderstandMonolog\Handler\UnderstandAsyncHandler;
-use UnderstandMonolog\Handler\UnderstandSyncHandler;
 
 class UnderstandLumenServiceProvider extends UnderstandServiceProvider
 {

Spacing +9 added lines, -9 removed lines

@@ -80,7 +80,7 @@ 
             return $fieldProvider;
         });
 
-        if (! class_exists('UnderstandFieldProvider'))
+        if (!class_exists('UnderstandFieldProvider'))
         {
             class_alias('Understand\UnderstandLaravel5\Facades\UnderstandFieldProvider', 'UnderstandFieldProvider');
         }
@@ -95,7 +95,7 @@ 
     {
         parent::registerEventLoggers();
 
-        if (! class_exists('UnderstandExceptionLogger'))
+        if (!class_exists('UnderstandExceptionLogger'))
         {
             class_alias('Understand\UnderstandLaravel5\Facades\UnderstandExceptionLogger', 'UnderstandExceptionLogger');
         }
@@ -110,7 +110,7 @@ 
     {
         parent::registerLogger();
 
-        if (! class_exists('UnderstandLogger'))
+        if (!class_exists('UnderstandLogger'))
         {
             class_alias('Understand\UnderstandLaravel5\Facades\UnderstandLogger', 'UnderstandLogger');
         }
@@ -157,7 +157,7 @@ 
 
             // the illuminate.log event is raised
             // by our MonologHandler, not by Lumen
-            $this->app['events']->listen('illuminate.log', function ($log)
+            $this->app['events']->listen('illuminate.log', function($log)
             {
                 $this->handleEvent($log['level'], $log['message'], $log['context']);
             });
@@ -165,7 +165,7 @@ 
         else
         {
             // starting from Lumen 5.6 MessageLogged event class was introduced
-            $this->app['events']->listen('Illuminate\Log\Events\MessageLogged', function ($log)
+            $this->app['events']->listen('Illuminate\Log\Events\MessageLogged', function($log)
             {
                 $this->handleEvent($log->level, $log->message, $log->context);
             });
@@ -175,17 +175,17 @@ 
         // https://github.com/illuminate/queue/commit/ce2b5518902b1bcb9ef650c41900fc8c6392eb0c
         if ($this->app->runningInConsole()) {
             if ($this->detectLumenVersion(['5.0', '5.1'])) {
-                $this->app['events']->listen('illuminate.queue.after', function () {
+                $this->app['events']->listen('illuminate.queue.after', function() {
                     $this->app['understand.tokenProvider']->generate();
                     $this->app['understand.dataCollector']->reset();
                 });
 
-                $this->app['events']->listen('illuminate.queue.failed', function () {
+                $this->app['events']->listen('illuminate.queue.failed', function() {
                     $this->app['understand.tokenProvider']->generate();
                     $this->app['understand.dataCollector']->reset();
                 });
             } else {
-                $this->app['events']->listen('Illuminate\Queue\Events\JobProcessing', function () {
+                $this->app['events']->listen('Illuminate\Queue\Events\JobProcessing', function() {
                     $this->app['understand.tokenProvider']->generate();
                     $this->app['understand.dataCollector']->reset();
                 });
@@ -242,7 +242,7 @@ 
         }
 
         // `\Log::info`, `\Log::debug` and NOT `\Exception` or `\Throwable`
-        if (in_array($level, ['info', 'debug']) && ! ($message instanceof Exception || $message instanceof Throwable))
+        if (in_array($level, ['info', 'debug']) && !($message instanceof Exception || $message instanceof Throwable))
         {
             $this->app['understand.eventLogger']->logEvent($level, $message, $context);
         }

Braces +3 added lines, -6 removed lines

@@ -161,8 +161,7 @@ 
             {
                 $this->handleEvent($log['level'], $log['message'], $log['context']);
             });
-        }
-        else
+        } else
         {
             // starting from Lumen 5.6 MessageLogged event class was introduced
             $this->app['events']->listen('Illuminate\Log\Events\MessageLogged', function ($log)
@@ -211,8 +210,7 @@ 
                     'time' => $time,
                 ]);
             });
-        }
-        else
+        } else
         {
             // https://laravel.com/api/5.3/Illuminate/Database/Events/QueryExecuted.html
             $this->app['events']->listen('Illuminate\Database\Events\QueryExecuted', function($event)
@@ -253,8 +251,7 @@ 
             unset($context['exception']);
 
             $this->app['understand.exceptionLogger']->logError($level, $exception, $context);
-        }
-        else
+        } else
         {
             $this->app['understand.exceptionLogger']->logError($level, $message, $context);
         }

src/Understand/UnderstandLaravel5/UnderstandServiceProvider.php

Doc Comments +2 added lines, -2 removed lines

@@ -117,7 +117,7 @@ 
     }
 
     /**
-     * @param $level
+     * @param string $level
      * @param $message
      * @param $context
      * @return bool
@@ -136,7 +136,7 @@ 
     /**
      * Get the services provided by the provider.
      *
-     * @return array
+     * @return string[]
      */
     public function provides()
     {

Spacing +6 added lines, -6 removed lines

@@ -11,7 +11,7 @@ 
      */
     protected function registerConfig()
     {
-        $configPath = __DIR__ . '/../../config/understand-laravel.php';
+        $configPath = __DIR__.'/../../config/understand-laravel.php';
         $this->mergeConfigFrom($configPath, 'understand-laravel');
     }
 
@@ -22,7 +22,7 @@ 
      */
     protected function registerTokenProvider()
     {
-        $this->app->singleton('understand.tokenProvider', function ()
+        $this->app->singleton('understand.tokenProvider', function()
         {
             return new TokenProvider();
         });
@@ -35,7 +35,7 @@ 
      */
     protected function registerDataCollector()
     {
-        $this->app->singleton('understand.dataCollector', function ()
+        $this->app->singleton('understand.dataCollector', function()
         {
             return new DataCollector();
         });
@@ -48,7 +48,7 @@ 
      */
     protected function registerExceptionEncoder()
     {
-        $this->app->bind('understand.exceptionEncoder', function ()
+        $this->app->bind('understand.exceptionEncoder', function()
         {
             return new ExceptionEncoder;
         });
@@ -113,7 +113,7 @@ 
             return new Handlers\SyncHandler($inputToken, $apiUrl, $sslBundlePath);
         }
 
-        throw new \ErrorException('understand-laravel handler misconfiguration:' . $handlerType);
+        throw new \ErrorException('understand-laravel handler misconfiguration:'.$handlerType);
     }
 
     /**
@@ -124,7 +124,7 @@ 
      */
     protected function shouldIgnoreEvent($level, $message, $context)
     {
-        $ignoredEventTypes = (array)$this->app['config']->get('understand-laravel.ignored_logs');
+        $ignoredEventTypes = (array) $this->app['config']->get('understand-laravel.ignored_logs');
 
         if (!$ignoredEventTypes) {
             return false;

src/Understand/UnderstandLaravel5/FieldProvider.php

Braces +14 added lines, -28 removed lines

@@ -266,8 +266,7 @@ 
         {
             // Laravel
             return Application::VERSION;
-        }
-        else
+        } else
         {
             $re = '/Lumen \((.*)\) \(.*\)/m';
 
@@ -327,17 +326,14 @@ 
                 if ($value instanceof \DateTimeInterface)
                 {
                     $binding = $value->format('Y-m-d H:i:s');
-                }
-                elseif (is_bool($value))
+                } elseif (is_bool($value))
                 {
                     $binding = (int) $value;
-                }
-                else
+                } else
                 {
                     $binding = (string)$value;
                 }
-            }
-            catch (\Exception $e)
+            } catch (\Exception $e)
             {
                 $binding = '[handler error]';
             }
@@ -414,8 +410,7 @@ 
             try
             {
                 $queryString[$key] = $this->parseRequestFieldValue($key, $value);
-            }
-            catch (\Exception $e)
+            } catch (\Exception $e)
             {
                 $queryString[$key] = '[handler error]';
             }
@@ -448,12 +443,10 @@ 
         if (method_exists($this->request, 'json') && method_exists($this->request, 'isJson') && $this->request->isJson())
         {
             $source = $this->request->json();
-        }
-        else if ($this->request->request instanceof \IteratorAggregate)
+        } else if ($this->request->request instanceof \IteratorAggregate)
         {
             $source = $this->request->request;
-        }
-        else
+        } else
         {
             return;
         }
@@ -470,8 +463,7 @@ 
             try
             {
                 $postData[$key] = $this->parseRequestFieldValue($key, $value);
-            }
-            catch (\Exception $e)
+            } catch (\Exception $e)
             {
                 $postData[$key] = '[handler error]';
             }
@@ -640,10 +632,8 @@ 
             {
                 return $userId;
             }
-        }
-        catch (\Throwable $e)
-        {}
-        catch (\Exception $e)
+        } catch (\Throwable $e)
+        {} catch (\Exception $e)
         {}
 
         try
@@ -652,10 +642,8 @@ 
             {
                 return $user->id;
             }
-        }
-        catch (\Throwable $e)
-        {}
-        catch (\Exception $e)
+        } catch (\Throwable $e)
+        {} catch (\Exception $e)
         {}
 
         try
@@ -664,10 +652,8 @@ 
             {
                 return $user->id;
             }
-        }
-        catch (\Throwable $e)
-        {}
-        catch (\Exception $e)
+        } catch (\Throwable $e)
+        {} catch (\Exception $e)
         {}
     }