understand / understand-laravel5

src/Understand/UnderstandLaravel5/Handlers/BaseHandler.php

Doc Comments +1 added lines, -1 removed lines

@@ -104,7 +104,7 @@
     /**
      * Parse respnse into array
      *
-     * @param type $response
+     * @param string $response
      * @param string $requestData
      * @return array
      */

Spacing +2 added lines, -2 removed lines

@@ -131,7 +131,7 @@ 
     {
         if (!$responseArr)
         {
-            throw new HandlerException('Cannot create connection to ' . $this->apiUrl);
+            throw new HandlerException('Cannot create connection to '.$this->apiUrl);
         }
 
         if (isset($responseArr['error']))
@@ -139,7 +139,7 @@ 
             throw new HandlerException($responseArr['error']);
         }
 
-        throw new HandlerException('Error. ' . ' Request data: ' . json_decode($requestData));
+        throw new HandlerException('Error. '.' Request data: '.json_decode($requestData));
     }
 
 }

src/Understand/UnderstandLaravel5/Handlers/LaravelQueueHandler.php

Doc Comments +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
      * Serialize data and send to storage
      *
      * @param array $requestData
-     * @return array
+     * @return type|null
      */
     public function handle(array $requestData)
     {

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
         }
         catch (\Exception $ex)
         {
-            if ( ! $this->silent)
+            if (!$this->silent)
             {
                 throw new \Understand\UnderstandLaravel5\Exceptions\HandlerException($ex->getMessage(), $ex->getCode(), $ex);
             }

Braces +1 added lines, -2 removed lines

@@ -16,8 +16,7 @@
             \Queue::push('Understand\UnderstandLaravel5\Handlers\LaravelQueueListener@listen', [
                 'requestData' => $requestData
             ]);
-        }
-        catch (\Exception $ex)
+        } catch (\Exception $ex)
         {
             if ( ! $this->silent)
             {

src/Understand/UnderstandLaravel5/Logger.php

Doc Comments +3 added lines, -5 removed lines

@@ -28,8 +28,8 @@ 
     protected $silent = true;
 
     /**
-     * @param \Understand\UnderstandLaravel\FieldProvider $fieldProvider
-     * @param \Understand\UnderstandLaravel\Handlers\BaseHandler $handler
+     * @param FieldProvider $fieldProvider
+     * @param BaseHandler $handler
      * @param bool $silent
      */
     public function __construct(FieldProvider $fieldProvider, BaseHandler $handler, $silent = true)
@@ -56,7 +56,6 @@ 
     /**
      * Send multiple events
      *
-     * @param array $data
      * @return array
      */
     public function bulkLog(array $events, array $additional = [])
@@ -107,7 +106,7 @@ 
     /**
      * Set field provider
      *
-     * @param \Understand\UnderstandLaravel\FieldProvider $fieldProvider
+     * @param FieldProvider $fieldProvider
      */
     public function setFieldProvider(FieldProvider $fieldProvider)
     {
@@ -117,7 +116,6 @@ 
     /**
      * Send data to storage
      *
-     * @param array $requestData
      * @return mixed
      */
     protected function send(array $event)

Braces +1 added lines, -2 removed lines

@@ -125,8 +125,7 @@
         try
         {
             return $this->handler->handle($event);
-        }
-        catch (\Exception $ex)
+        } catch (\Exception $ex)
         {
             if (! $this->silent)
             {

Spacing +3 added lines, -3 removed lines

@@ -110,9 +110,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
@@ -158,7 +158,7 @@ 
         }
         catch (\Exception $ex)
         {
-            if (! $this->silent)
+            if (!$this->silent)
             {
                 throw new $ex;
             }

src/Understand/UnderstandLaravel5/ModelEventListener.php

Doc Comments +2 added lines, -2 removed lines

@@ -21,9 +21,9 @@
     protected $additionalFields;
 
     /**
-     * @param Understand\UnderstandLaravel\Logger $logger
+     * @param Logger $logger
      *
-     * @param \Understand\UnderstandLaravel\Logger $logger
+     * @param Logger $logger
      * @param array $additionalFields
      */
     public function __construct(Logger $logger, array $additionalFields)

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
     public function logModelEvent($eventName, Model $model, $modelLevelEventName = null)
     {
         $log = [
-            'message' => is_null($modelLevelEventName) ? null : (string)$modelLevelEventName,
+            'message' => is_null($modelLevelEventName) ? null : (string) $modelLevelEventName,
             'id' => (int) $model->id,
             'model_event' => $eventName,
             'model_name' => get_class($model),

src/Understand/UnderstandLaravel5/ExceptionLogger.php

Spacing +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
             return;
         }
 
-        if ( ! $this->config->get('understand-laravel.log_types.exception_log.enabled'))
+        if (!$this->config->get('understand-laravel.log_types.exception_log.enabled'))
         {
             return;
         }

src/Understand/UnderstandLaravel5/FieldProvider.php

Braces +3 added lines, -6 removed lines

@@ -315,8 +315,7 @@ 
             {
                 return $userId;
             }
-        }
-        catch (\Exception $e) 
+        } catch (\Exception $e) 
         {}
 
         try 
@@ -325,8 +324,7 @@ 
             {
                 return $user->id;
             }
-        } 
-        catch (\Exception $e) 
+        } catch (\Exception $e) 
         {}
         
         try
@@ -335,8 +333,7 @@ 
             {
                 return $user->id;
             }
-        }
-        catch (\Exception $e) 
+        } catch (\Exception $e) 
         {}
     }
 

Doc Comments +2 added lines, -4 removed lines

@@ -72,7 +72,6 @@ 
     /**
      * Create field provider instance and set default providers to provider list
      *
-     * @param type $app
      * @return void
      */
     public function __construct()
@@ -86,7 +85,7 @@ 
     /**
      * Set session store
      *
-     * @param type $service
+     * @param SessionStore $service
      */
     public function setSessionStore(SessionStore $service)
     {
@@ -127,7 +126,6 @@ 
      * Register a custom HTML macro.
      *
      * @param string $name
-     * @param  mixed  $macro
      * @return void
      */
     public function extend($name, $provider)
@@ -138,7 +136,7 @@ 
     /**
      * Set token provider
      *
-     * @param UniqueProcessIdentifier $tokenProvider
+     * @param TokenProvider $tokenProvider
      */
     public function setTokenProvider(TokenProvider $tokenProvider)
     {

Spacing +11 added lines, -11 removed lines

@@ -199,7 +199,7 @@ 
      */
     protected function getSessionId()
     {
-        if ( ! $this->session)
+        if (!$this->session)
         {
             return null;
         }
@@ -219,7 +219,7 @@ 
      */
     protected function getRouteName()
     {
-        if ( ! $this->router)
+        if (!$this->router)
         {
             return null;
         }
@@ -234,23 +234,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;
@@ -263,7 +263,7 @@ 
      */
     protected function getRequestMethod()
     {
-        if ( ! $this->request)
+        if (!$this->request)
         {
             return null;
         }
@@ -278,7 +278,7 @@ 
      */
     protected function getServerIp()
     {
-        if ( ! $this->request)
+        if (!$this->request)
         {
             return null;
         }
@@ -293,7 +293,7 @@ 
      */
     protected function getClientIp()
     {
-        if ( ! $this->request)
+        if (!$this->request)
         {
             return null;
         }
@@ -308,7 +308,7 @@ 
      */
     protected function getClientUserAgent()
     {
-        if ( ! $this->request)
+        if (!$this->request)
         {
             return null;
         }
@@ -334,7 +334,7 @@ 
      */
     protected function getFromSession($key)
     {
-        if ( ! $this->session)
+        if (!$this->session)
         {
             return null;
         }

src/Understand/UnderstandLaravel5/Exceptions/HandlerException.php

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
      */
     public function __construct($message = '', $code = 0, $previous = null)
     {
-        $message = 'understand-laravel: ' . $message;
+        $message = 'understand-laravel: '.$message;
 
         parent::__construct($message, $code, $previous);
     }

src/Understand/UnderstandLaravel5/UnderstandLaravel5ServiceProvider.php

Indentation +27 added lines, -27 removed lines

@@ -6,20 +6,20 @@ 
 class UnderstandLaravel5ServiceProvider extends ServiceProvider
 {
 
-	/**
-	 * Indicates if loading of the provider is deferred.
-	 *
-	 * @var bool
-	 */
-	protected $defer = false;
-
-	/**
-	 * Bootstrap the application events.
-	 *
-	 * @return void
-	 */
-	public function boot()
-	{
+    /**
+     * Indicates if loading of the provider is deferred.
+     *
+     * @var bool
+     */
+    protected $defer = false;
+
+    /**
+     * Bootstrap the application events.
+     *
+     * @return void
+     */
+    public function boot()
+    {
         $configPath = __DIR__ . '/../../config/understand-laravel.php';
         $this->publishes([$configPath => config_path('understand-laravel.php')], 'config');
 
@@ -32,23 +32,23 @@ 
         {
             $this->listenLaravelEvents();
         }
-	}
-
-	/**
-	 * 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->registerTokenProvider();
         $this->registerLogger();
         $this->registerModelEventListenerProvider();
         $this->registerExceptionEncoder();
         $this->registerExceptionLogger();
-	}
+    }
 
     /**
      * Register config
@@ -72,8 +72,8 @@ 
         {
             $fieldProvider = new FieldProvider();
 
-	    if ($app['config']['session.driver']) 
-	    {
+        if ($app['config']['session.driver']) 
+        {
                 $fieldProvider->setSessionStore($app['session.store']);
             }
             $fieldProvider->setRouter($app['router']);

Doc Comments +3 added lines, -3 removed lines

@@ -218,8 +218,8 @@ 
     /**
      * Detect Laravel version
      * 
-     * @param array $versions
-     * @return type
+     * @param string[] $versions
+     * @return boolean
      */
     protected function detectLaravelVersion(array $versions)
     {
@@ -340,7 +340,7 @@ 
     /**
      * Get the services provided by the provider.
      *
-     * @return array
+     * @return string[]
      */
     public function provides()
     {

Spacing +5 added lines, -5 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');
 
         if ($this->app['config']->get('understand-laravel.log_types.eloquent_log.enabled'))
@@ -60,7 +60,7 @@ 
      */
     protected function registerConfig()
     {
-        $configPath = __DIR__ . '/../../config/understand-laravel.php';
+        $configPath = __DIR__.'/../../config/understand-laravel.php';
         $this->mergeConfigFrom($configPath, 'understand-laravel');
     }
 
@@ -196,7 +196,7 @@ 
             return new Handlers\LaravelQueueHandler($inputToken, $apiUrl, $silent, $sslBundlePath);
         }
 
-        throw new \ErrorException('understand-laravel handler misconfiguration:' . $handlerType);
+        throw new \ErrorException('understand-laravel handler misconfiguration:'.$handlerType);
     }
 
     /**
@@ -282,7 +282,7 @@ 
         }
         else
         {
-            $log = (array)$message;
+            $log = (array) $message;
             $log['tags'] = ['laravel_log'];
         }
 
@@ -319,7 +319,7 @@ 
             {
                 $this->app['events']->listen($listenerName, function($model) use($modelLogger, $eventName)
                 {
-                    $modelLevelEventName = 'eloquent.' . $eventName . ': ' . get_class($model);
+                    $modelLevelEventName = 'eloquent.'.$eventName.': '.get_class($model);
                     
                     $modelLogger->logModelEvent($eventName, $model, $modelLevelEventName);
                 });

Braces +4 added lines, -8 removed lines

@@ -240,8 +240,7 @@ 
             {
                 $this->handleEvent($level, $message, $context);
             });
-        }
-        else
+        } else
         {
             // starting from L5.4 MessageLogged event class was introduced
             // https://github.com/laravel/framework/commit/57c82d095c356a0fe0f9381536afec768cdcc072
@@ -268,8 +267,7 @@ 
         if ($message instanceof Exceptions\HandlerException)
         {
             return;
-        }
-        else if ($message instanceof \Exception)
+        } else if ($message instanceof \Exception)
         {
             $log = $this->app['understand.exception-encoder']->exceptionToArray($message);
             $log['tags'] = ['exception_log'];
@@ -279,8 +277,7 @@ 
         {
             $log['message'] = $message;
             $log['tags'] = ['laravel_log'];
-        }
-        else
+        } else
         {
             $log = (array)$message;
             $log['tags'] = ['laravel_log'];
@@ -323,8 +320,7 @@ 
                     
                     $modelLogger->logModelEvent($eventName, $model, $modelLevelEventName);
                 });
-            }
-            else
+            } else
             {
                 $this->app['events']->listen($listenerName, function($modelLevelEventName, $eventPayload) use($modelLogger, $eventName)
                 {

src/Understand/UnderstandLaravel5/ExceptionEncoder.php

Spacing +7 added lines, -7 removed lines

@@ -89,7 +89,7 @@ 
      */
     protected function stackTraceCallToString(array $trace)
     {
-        if (! isset($trace['type']))
+        if (!isset($trace['type']))
         {
             return 'function';
         }
@@ -115,7 +115,7 @@ 
     {
         $params = [];
 
-        if (! isset($trace['args']))
+        if (!isset($trace['args']))
         {
             return $params;
         }
@@ -124,7 +124,7 @@ 
         {
             if (is_array($arg))
             {
-                $params[] = 'array(' . count($arg) . ')';
+                $params[] = 'array('.count($arg).')';
             }
             else if (is_object($arg))
             {
@@ -132,19 +132,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)
             {

Braces +7 added lines, -14 removed lines

@@ -125,32 +125,25 @@
             if (is_array($arg))
             {
                 $params[] = 'array(' . count($arg) . ')';
-            }
-            else if (is_object($arg))
+            } else if (is_object($arg))
             {
                 $params[] = get_class($arg);
-            }
-            else if (is_string($arg))
+            } else if (is_string($arg))
             {
                 $params[] = 'string(' . $arg . ')';
-            }
-            else if (is_int($arg))
+            } else if (is_int($arg))
             {
                 $params[] = 'int(' . $arg . ')';
-            }
-            else if (is_float($arg))
+            } else if (is_float($arg))
             {
                 $params[] = 'float(' . $arg . ')';
-            }
-            else if (is_bool($arg))
+            } else if (is_bool($arg))
             {
                 $params[] = 'bool(' . ($arg ? 'true' : 'false') . ')';
-            }
-            else if ($arg instanceof \__PHP_Incomplete_Class)
+            } else if ($arg instanceof \__PHP_Incomplete_Class)
             {
                 $params[] = 'object(__PHP_Incomplete_Class)'; 
-            }
-            else
+            } else
             {
                 $params[] = gettype($arg);
             }