understand / understand-laravel5

src/Understand/UnderstandLaravel5/ExceptionEncoder.php

Doc Comments +2 added lines, -2 removed lines

@@ -157,7 +157,7 @@ 
     }
 
     /**
-     * @param $relativePath
+     * @param string|null $relativePath
      * @param $line
      * @param int $linesAround
      * @return array|void
@@ -217,7 +217,7 @@ 
      * Return stack trace file
      *
      * @param array $trace
-     * @return mixed
+     * @return string|null
      */
     protected function getStackTraceFile(array $trace)
     {

Spacing +16 added lines, -16 removed lines

@@ -42,7 +42,7 @@ 
      */
     public function exceptionToArray($exception)
     {
-        if ( ! ($exception instanceof Exception || $exception instanceof Throwable))
+        if (!($exception instanceof Exception || $exception instanceof Throwable))
         {
             throw new InvalidArgumentException('$exception must be instance of Exception or Throwable');
         }
@@ -71,7 +71,7 @@ 
         $stackTrace = $this->getCurrentStackTrace($level);
         $firstLineSet = false;
 
-        foreach($stackTrace as $trace)
+        foreach ($stackTrace as $trace)
         {
             if ($firstLineSet)
             {
@@ -98,7 +98,7 @@ 
         $stackTrace = debug_backtrace(DEBUG_BACKTRACE_PROVIDE_OBJECT, $this->stackTraceLimit);
         $vendorExcluded = false;
 
-        foreach($stackTrace as $index => $trace)
+        foreach ($stackTrace as $index => $trace)
         {
             // exclude Understand service provider and helper classes
             if (isset($trace['class']) && strpos($trace['class'], 'Understand\UnderstandLaravel5\\') === 0)
@@ -106,7 +106,7 @@ 
                 unset($stackTrace[$index]);
             }
 
-            if ( ! isset($trace['file']))
+            if (!isset($trace['file']))
             {
                 $vendorExcluded = true;
             }
@@ -117,7 +117,7 @@ 
             }
 
             // exclude `vendor` folder until project path reached
-            if (strpos($trace['file'], $this->projectRoot . 'vendor' . DIRECTORY_SEPARATOR) === 0)
+            if (strpos($trace['file'], $this->projectRoot.'vendor'.DIRECTORY_SEPARATOR) === 0)
             {
                 unset($stackTrace[$index]);
             }
@@ -185,12 +185,12 @@ 
      */
     public function getCode($relativePath, $line, $linesAround = 6)
     {
-        if ( ! $relativePath || ! $line)
+        if (!$relativePath || !$line)
         {
             return;
         }
 
-        $filePath = $this->projectRoot . $relativePath;
+        $filePath = $this->projectRoot.$relativePath;
 
         try
         {
@@ -200,11 +200,11 @@ 
             $codeLines = [];
 
             $from = max(0, $line - $linesAround - 2);
-            $to = min($line + $linesAround -1, $file->key() + 1);
+            $to = min($line + $linesAround - 1, $file->key() + 1);
 
             $file->seek($from);
 
-            while ($file->key() < $to && ! $file->eof())
+            while ($file->key() < $to && !$file->eof())
             {
                 $file->next();
                 // `key()` returns 0 as the first line
@@ -256,7 +256,7 @@ 
      */
     protected function stackTraceCallToString(array $trace)
     {
-        if (! isset($trace['type']))
+        if (!isset($trace['type']))
         {
             return 'function';
         }
@@ -282,7 +282,7 @@ 
     {
         $params = [];
 
-        if (! isset($trace['args']))
+        if (!isset($trace['args']))
         {
             return $params;
         }
@@ -291,7 +291,7 @@ 
         {
             if (is_array($arg))
             {
-                $params[] = 'array(' . count($arg) . ')';
+                $params[] = 'array('.count($arg).')';
             }
             else if (is_object($arg))
             {
@@ -299,19 +299,19 @@ 
             }
             else if (is_string($arg))
             {
-                $params[] = 'string(' . $arg . ')';
+                $params[] = 'string('.$arg.')';
             }
             else if (is_int($arg))
             {
-                $params[] = 'int(' . $arg . ')';
+                $params[] = 'int('.$arg.')';
             }
             else if (is_float($arg))
             {
-                $params[] = 'float(' . $arg . ')';
+                $params[] = 'float('.$arg.')';
             }
             else if (is_bool($arg))
             {
-                $params[] = 'bool(' . ($arg ? 'true' : 'false') . ')';
+                $params[] = 'bool('.($arg ? 'true' : 'false').')';
             }
             else if ($arg instanceof \__PHP_Incomplete_Class)
             {

src/Understand/UnderstandLaravel5/FieldProvider.php

Doc Comments +3 added lines, -4 removed lines

@@ -87,7 +87,7 @@ 
     protected $app;
 
     /**
-     * @return void
+     * @return Application
      */
     public function __construct()
     {
@@ -108,7 +108,7 @@ 
     /**
      * Set session store
      *
-     * @param type $service
+     * @param SessionStore $service
      */
     public function setSessionStore(SessionStore $service)
     {
@@ -157,7 +157,6 @@ 
      * Register a custom HTML macro.
      *
      * @param string $name
-     * @param  mixed  $macro
      * @return void
      */
     public function extend($name, $provider)
@@ -168,7 +167,7 @@ 
     /**
      * Set token provider
      *
-     * @param UniqueProcessIdentifier $tokenProvider
+     * @param TokenProvider $tokenProvider
      */
     public function setTokenProvider(TokenProvider $tokenProvider)
     {

Spacing +14 added lines, -14 removed lines

@@ -230,7 +230,7 @@ 
      */
     protected function getSessionId()
     {
-        if ( ! $this->session)
+        if (!$this->session)
         {
             return null;
         }
@@ -256,7 +256,7 @@ 
      */
     protected function getSqlQueries()
     {
-        if ( ! $this->dataCollector)
+        if (!$this->dataCollector)
         {
             return [];
         }
@@ -276,7 +276,7 @@ 
      */
     protected function getRouteName()
     {
-        if ( ! $this->router)
+        if (!$this->router)
         {
             return null;
         }
@@ -291,23 +291,23 @@ 
      */
     protected function getUrl()
     {
-        if ( ! $this->request)
+        if (!$this->request)
         {
             return null;
         }
 
         $url = $this->request->path();
 
-        if ( ! starts_with($url, '/'))
+        if (!starts_with($url, '/'))
         {
-            $url = '/' . $url;
+            $url = '/'.$url;
         }
 
         $queryString = $this->request->getQueryString();
 
         if ($queryString)
         {
-            $url .= '?' . $queryString;
+            $url .= '?'.$queryString;
         }
 
         return $url;
@@ -320,7 +320,7 @@ 
      */
     protected function getRequestMethod()
     {
-        if ( ! $this->request)
+        if (!$this->request)
         {
             return null;
         }
@@ -335,7 +335,7 @@ 
      */
     protected function getServerIp()
     {
-        if ( ! $this->request)
+        if (!$this->request)
         {
             return null;
         }
@@ -350,7 +350,7 @@ 
      */
     protected function getClientIp()
     {
-        if ( ! $this->request)
+        if (!$this->request)
         {
             return null;
         }
@@ -365,7 +365,7 @@ 
      */
     protected function getClientUserAgent()
     {
-        if ( ! $this->request)
+        if (!$this->request)
         {
             return null;
         }
@@ -391,7 +391,7 @@ 
      */
     protected function getFromSession($key)
     {
-        if ( ! $this->session)
+        if (!$this->session)
         {
             return null;
         }
@@ -409,9 +409,9 @@ 
     {
         $parts = [];
 
-        foreach(['class', 'file', 'line'] as $field)
+        foreach (['class', 'file', 'line'] as $field)
         {
-            $parts[] = isset($log[$field]) ? (string)$log[$field] : null;
+            $parts[] = isset($log[$field]) ? (string) $log[$field] : null;
         }
 
         return sha1(implode('#', $parts));

src/Understand/UnderstandLaravel5/Logger.php

Doc Comments -2 removed lines

@@ -28,7 +28,6 @@ 
     /**
      * @param FieldProvider $fieldProvider
      * @param BaseHandler $handler
-     * @param bool $silent
      */
     public function __construct(FieldProvider $fieldProvider, BaseHandler $handler)
     {
@@ -144,7 +143,6 @@ 
     /**
      * Send data to storage
      *
-     * @param array $requestData
      * @return mixed
      */
     protected function send(array $event)

Spacing +2 added lines, -2 removed lines

@@ -107,9 +107,9 @@
      */
     protected function formatMessage($message)
     {
-        if ( ! is_bool($message))
+        if (!is_bool($message))
         {
-            return (string)$message;
+            return (string) $message;
         }
         
         // cast boolean values to "1" or "0" strings

src/Understand/UnderstandLaravel5/ExceptionLogger.php

Spacing +5 added lines, -5 removed lines

@@ -55,7 +55,7 @@ 
      */
     public function log($error)
     {
-        if ( ! $this->canHandle($error))
+        if (!$this->canHandle($error))
         {
             return;
         }
@@ -73,7 +73,7 @@ 
      */
     public function logError($level, $message, $context)
     {
-        if ( ! $this->canHandle($message))
+        if (!$this->canHandle($message))
         {
             return;
         }
@@ -93,7 +93,7 @@ 
         }
         else
         {
-            $log = (array)$message;
+            $log = (array) $message;
             $log = $this->encoder->setCurrentStackTrace($log);
         }
 
@@ -101,7 +101,7 @@ 
 
         if ($context)
         {
-            $log['context'] = (array)$context;
+            $log['context'] = (array) $context;
         }
 
         return $this->dispatch($log);
@@ -150,6 +150,6 @@ 
      */
     protected function canHandle($error)
     {
-        return (bool)$this->config->get('understand-laravel.enabled');
+        return (bool) $this->config->get('understand-laravel.enabled');
     }
 }
\ No newline at end of file

src/Understand/UnderstandLaravel5/EventLogger.php

Spacing +2 added lines, -2 removed lines

@@ -48,7 +48,7 @@ 
         }
         else
         {
-            $log = (array)$message;
+            $log = (array) $message;
         }
 
         $log['tags'] = ['laravel_log'];
@@ -56,7 +56,7 @@ 
 
         if ($context)
         {
-            $log['context'] = (array)$context;
+            $log['context'] = (array) $context;
         }
 
         $additionalFields = $this->getMetaFields();

src/config/understand-laravel.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
     /**
      * Project root folder
      */
-    'project_root' => base_path() . DIRECTORY_SEPARATOR,
+    'project_root' => base_path().DIRECTORY_SEPARATOR,
 
     /**
      * Collect SQL queries without bindings

src/Understand/UnderstandLaravel5/UnderstandLaravel5ServiceProvider.php

Doc Comments +4 added lines, -4 removed lines

@@ -215,8 +215,8 @@ 
     /**
      * Detect Laravel version
      * 
-     * @param array $versions
-     * @return type
+     * @param string[] $versions
+     * @return boolean
      */
     protected function detectLaravelVersion(array $versions)
     {
@@ -317,7 +317,7 @@ 
     }
 
     /**
-     * @param $level
+     * @param string $level
      * @param $message
      * @param $context
      * @return bool
@@ -337,7 +337,7 @@ 
     /**
      * Get the services provided by the provider.
      *
-     * @return array
+     * @return string[]
      */
     public function provides()
     {

Spacing +7 added lines, -7 removed lines

@@ -24,7 +24,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');
 
@@ -62,7 +62,7 @@ 
      */
     protected function registerConfig()
     {
-        $configPath = __DIR__ . '/../../config/understand-laravel.php';
+        $configPath = __DIR__.'/../../config/understand-laravel.php';
         $this->mergeConfigFrom($configPath, 'understand-laravel');
     }
 
@@ -209,7 +209,7 @@ 
             return new Handlers\SyncHandler($inputToken, $apiUrl, $sslBundlePath);
         }
 
-        throw new \ErrorException('understand-laravel handler misconfiguration:' . $handlerType);
+        throw new \ErrorException('understand-laravel handler misconfiguration:'.$handlerType);
     }
     
     /**
@@ -298,13 +298,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']);
@@ -325,9 +325,9 @@ 
      */
     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)
+        if (!$ignoredEventTypes)
         {
             return false;
         }