@@ -72,7 +72,6 @@ discard block |
||
| 72 | 72 | /** |
| 73 | 73 | * Create field provider instance and set default providers to provider list |
| 74 | 74 | * |
| 75 | - * @param type $app |
|
| 76 | 75 | * @return void |
| 77 | 76 | */ |
| 78 | 77 | public function __construct() |
@@ -86,7 +85,7 @@ discard block |
||
| 86 | 85 | /** |
| 87 | 86 | * Set session store |
| 88 | 87 | * |
| 89 | - * @param type $service |
|
| 88 | + * @param SessionStore $service |
|
| 90 | 89 | */ |
| 91 | 90 | public function setSessionStore(SessionStore $service) |
| 92 | 91 | { |
@@ -106,7 +105,6 @@ discard block |
||
| 106 | 105 | /** |
| 107 | 106 | * Set request |
| 108 | 107 | * |
| 109 | - * @param Request $server |
|
| 110 | 108 | */ |
| 111 | 109 | public function setRequest(Request $request) |
| 112 | 110 | { |
@@ -127,7 +125,6 @@ discard block |
||
| 127 | 125 | * Register a custom HTML macro. |
| 128 | 126 | * |
| 129 | 127 | * @param string $name |
| 130 | - * @param mixed $macro |
|
| 131 | 128 | * @return void |
| 132 | 129 | */ |
| 133 | 130 | public function extend($name, $provider) |
@@ -138,7 +135,7 @@ discard block |
||
| 138 | 135 | /** |
| 139 | 136 | * Set token provider |
| 140 | 137 | * |
| 141 | - * @param UniqueProcessIdentifier $tokenProvider |
|
| 138 | + * @param TokenProvider $tokenProvider |
|
| 142 | 139 | */ |
| 143 | 140 | public function setTokenProvider(TokenProvider $tokenProvider) |
| 144 | 141 | { |
@@ -226,7 +226,7 @@ |
||
| 226 | 226 | { |
| 227 | 227 | $url = $this->request->path(); |
| 228 | 228 | |
| 229 | - if ( ! starts_with($url, '/')) |
|
| 229 | + if (!starts_with($url, '/')) |
|
| 230 | 230 | { |
| 231 | 231 | $url = '/' . $url; |
| 232 | 232 | } |
@@ -315,8 +315,7 @@ discard block |
||
| 315 | 315 | { |
| 316 | 316 | return $userId; |
| 317 | 317 | } |
| 318 | - } |
|
| 319 | - catch (\Exception $e) |
|
| 318 | + } catch (\Exception $e) |
|
| 320 | 319 | {} |
| 321 | 320 | |
| 322 | 321 | try |
@@ -325,8 +324,7 @@ discard block |
||
| 325 | 324 | { |
| 326 | 325 | return $user->id; |
| 327 | 326 | } |
| 328 | - } |
|
| 329 | - catch (\Exception $e) |
|
| 327 | + } catch (\Exception $e) |
|
| 330 | 328 | {} |
| 331 | 329 | |
| 332 | 330 | try |
@@ -335,8 +333,7 @@ discard block |
||
| 335 | 333 | { |
| 336 | 334 | return $user->id; |
| 337 | 335 | } |
| 338 | - } |
|
| 339 | - catch (\Exception $e) |
|
| 336 | + } catch (\Exception $e) |
|
| 340 | 337 | {} |
| 341 | 338 | } |
| 342 | 339 | |
@@ -104,7 +104,7 @@ |
||
| 104 | 104 | /** |
| 105 | 105 | * Parse respnse into array |
| 106 | 106 | * |
| 107 | - * @param type $response |
|
| 107 | + * @param string $response |
|
| 108 | 108 | * @param string $requestData |
| 109 | 109 | * @return array |
| 110 | 110 | */ |
@@ -30,7 +30,7 @@ |
||
| 30 | 30 | * Serialize data and send to storage |
| 31 | 31 | * |
| 32 | 32 | * @param array $requestData |
| 33 | - * @return array |
|
| 33 | + * @return type|null |
|
| 34 | 34 | */ |
| 35 | 35 | public function handle(array $requestData) |
| 36 | 36 | { |
@@ -19,7 +19,7 @@ |
||
| 19 | 19 | } |
| 20 | 20 | catch (\Exception $ex) |
| 21 | 21 | { |
| 22 | - if ( ! $this->silent) |
|
| 22 | + if (!$this->silent) |
|
| 23 | 23 | { |
| 24 | 24 | throw new \Understand\UnderstandLaravel5\Exceptions\HandlerException($ex->getMessage(), $ex->getCode(), $ex); |
| 25 | 25 | } |
@@ -16,8 +16,7 @@ |
||
| 16 | 16 | \Queue::push('Understand\UnderstandLaravel5\Handlers\LaravelQueueListener@listen', [ |
| 17 | 17 | 'requestData' => $requestData |
| 18 | 18 | ]); |
| 19 | - } |
|
| 20 | - catch (\Exception $ex) |
|
| 19 | + } catch (\Exception $ex) |
|
| 21 | 20 | { |
| 22 | 21 | if ( ! $this->silent) |
| 23 | 22 | { |
@@ -28,8 +28,8 @@ discard block |
||
| 28 | 28 | protected $silent = true; |
| 29 | 29 | |
| 30 | 30 | /** |
| 31 | - * @param \Understand\UnderstandLaravel\FieldProvider $fieldProvider |
|
| 32 | - * @param \Understand\UnderstandLaravel\Handlers\BaseHandler $handler |
|
| 31 | + * @param FieldProvider $fieldProvider |
|
| 32 | + * @param BaseHandler $handler |
|
| 33 | 33 | * @param bool $silent |
| 34 | 34 | */ |
| 35 | 35 | public function __construct(FieldProvider $fieldProvider, BaseHandler $handler, $silent = true) |
@@ -56,7 +56,6 @@ discard block |
||
| 56 | 56 | /** |
| 57 | 57 | * Send multiple events |
| 58 | 58 | * |
| 59 | - * @param array $data |
|
| 60 | 59 | * @return array |
| 61 | 60 | */ |
| 62 | 61 | public function bulkLog(array $events, array $additional = []) |
@@ -107,7 +106,7 @@ discard block |
||
| 107 | 106 | /** |
| 108 | 107 | * Set field provider |
| 109 | 108 | * |
| 110 | - * @param \Understand\UnderstandLaravel\FieldProvider $fieldProvider |
|
| 109 | + * @param FieldProvider $fieldProvider |
|
| 111 | 110 | */ |
| 112 | 111 | public function setFieldProvider(FieldProvider $fieldProvider) |
| 113 | 112 | { |
@@ -117,7 +116,6 @@ discard block |
||
| 117 | 116 | /** |
| 118 | 117 | * Send data to storage |
| 119 | 118 | * |
| 120 | - * @param array $requestData |
|
| 121 | 119 | * @return mixed |
| 122 | 120 | */ |
| 123 | 121 | protected function send(array $event) |
@@ -128,7 +128,7 @@ |
||
| 128 | 128 | } |
| 129 | 129 | catch (\Exception $ex) |
| 130 | 130 | { |
| 131 | - if (! $this->silent) |
|
| 131 | + if (!$this->silent) |
|
| 132 | 132 | { |
| 133 | 133 | throw new $ex; |
| 134 | 134 | } |
@@ -125,8 +125,7 @@ |
||
| 125 | 125 | try |
| 126 | 126 | { |
| 127 | 127 | return $this->handler->handle($event); |
| 128 | - } |
|
| 129 | - catch (\Exception $ex) |
|
| 128 | + } catch (\Exception $ex) |
|
| 130 | 129 | { |
| 131 | 130 | if (! $this->silent) |
| 132 | 131 | { |
@@ -21,9 +21,9 @@ |
||
| 21 | 21 | protected $additionalFields; |
| 22 | 22 | |
| 23 | 23 | /** |
| 24 | - * @param Understand\UnderstandLaravel\Logger $logger |
|
| 24 | + * @param Logger $logger |
|
| 25 | 25 | * |
| 26 | - * @param \Understand\UnderstandLaravel\Logger $logger |
|
| 26 | + * @param Logger $logger |
|
| 27 | 27 | * @param array $additionalFields |
| 28 | 28 | */ |
| 29 | 29 | public function __construct(Logger $logger, array $additionalFields) |
@@ -277,7 +277,7 @@ |
||
| 277 | 277 | /** |
| 278 | 278 | * Get the services provided by the provider. |
| 279 | 279 | * |
| 280 | - * @return array |
|
| 280 | + * @return string[] |
|
| 281 | 281 | */ |
| 282 | 282 | public function provides() |
| 283 | 283 | { |
@@ -6,20 +6,20 @@ discard block |
||
| 6 | 6 | class UnderstandLaravel5ServiceProvider extends ServiceProvider |
| 7 | 7 | { |
| 8 | 8 | |
| 9 | - /** |
|
| 10 | - * Indicates if loading of the provider is deferred. |
|
| 11 | - * |
|
| 12 | - * @var bool |
|
| 13 | - */ |
|
| 14 | - protected $defer = false; |
|
| 15 | - |
|
| 16 | - /** |
|
| 17 | - * Bootstrap the application events. |
|
| 18 | - * |
|
| 19 | - * @return void |
|
| 20 | - */ |
|
| 21 | - public function boot() |
|
| 22 | - { |
|
| 9 | + /** |
|
| 10 | + * Indicates if loading of the provider is deferred. |
|
| 11 | + * |
|
| 12 | + * @var bool |
|
| 13 | + */ |
|
| 14 | + protected $defer = false; |
|
| 15 | + |
|
| 16 | + /** |
|
| 17 | + * Bootstrap the application events. |
|
| 18 | + * |
|
| 19 | + * @return void |
|
| 20 | + */ |
|
| 21 | + public function boot() |
|
| 22 | + { |
|
| 23 | 23 | $configPath = __DIR__ . '/../../config/understand-laravel.php'; |
| 24 | 24 | $this->publishes([$configPath => config_path('understand-laravel.php')], 'config'); |
| 25 | 25 | |
@@ -32,23 +32,23 @@ discard block |
||
| 32 | 32 | { |
| 33 | 33 | $this->listenLaravelEvents(); |
| 34 | 34 | } |
| 35 | - } |
|
| 36 | - |
|
| 37 | - /** |
|
| 38 | - * Register the service provider. |
|
| 39 | - * |
|
| 40 | - * @return void |
|
| 41 | - */ |
|
| 42 | - public function register() |
|
| 43 | - { |
|
| 44 | - $this->registerConfig(); |
|
| 35 | + } |
|
| 36 | + |
|
| 37 | + /** |
|
| 38 | + * Register the service provider. |
|
| 39 | + * |
|
| 40 | + * @return void |
|
| 41 | + */ |
|
| 42 | + public function register() |
|
| 43 | + { |
|
| 44 | + $this->registerConfig(); |
|
| 45 | 45 | $this->registerFieldProvider(); |
| 46 | 46 | $this->registerTokenProvider(); |
| 47 | 47 | $this->registerLogger(); |
| 48 | 48 | $this->registerModelEventListenerProvider(); |
| 49 | 49 | $this->registerExceptionEncoder(); |
| 50 | 50 | $this->registerExceptionLogger(); |
| 51 | - } |
|
| 51 | + } |
|
| 52 | 52 | |
| 53 | 53 | /** |
| 54 | 54 | * Register config |
@@ -220,18 +220,15 @@ |
||
| 220 | 220 | if ($message instanceof Exceptions\HandlerException) |
| 221 | 221 | { |
| 222 | 222 | return; |
| 223 | - } |
|
| 224 | - else if ($message instanceof \Exception) |
|
| 223 | + } else if ($message instanceof \Exception) |
|
| 225 | 224 | { |
| 226 | 225 | $log = $this->app['understand.exception-encoder']->exceptionToArray($message); |
| 227 | 226 | $log['tags'] = ['exception_log']; |
| 228 | - } |
|
| 229 | - else if (is_string($message)) |
|
| 227 | + } else if (is_string($message)) |
|
| 230 | 228 | { |
| 231 | 229 | $log['message'] = $message; |
| 232 | 230 | $log['tags'] = ['laravel_log']; |
| 233 | - } |
|
| 234 | - else |
|
| 231 | + } else |
|
| 235 | 232 | { |
| 236 | 233 | $log = $message; |
| 237 | 234 | $log['tags'] = ['laravel_log']; |
@@ -89,7 +89,7 @@ discard block |
||
| 89 | 89 | */ |
| 90 | 90 | protected function stackTraceCallToString(array $trace) |
| 91 | 91 | { |
| 92 | - if (! isset($trace['type'])) |
|
| 92 | + if (!isset($trace['type'])) |
|
| 93 | 93 | { |
| 94 | 94 | return 'function'; |
| 95 | 95 | } |
@@ -115,7 +115,7 @@ discard block |
||
| 115 | 115 | { |
| 116 | 116 | $params = []; |
| 117 | 117 | |
| 118 | - if (! isset($trace['args'])) |
|
| 118 | + if (!isset($trace['args'])) |
|
| 119 | 119 | { |
| 120 | 120 | return $params; |
| 121 | 121 | } |
@@ -125,28 +125,22 @@ |
||
| 125 | 125 | if (is_array($arg)) |
| 126 | 126 | { |
| 127 | 127 | $params[] = 'array(' . count($arg) . ')'; |
| 128 | - } |
|
| 129 | - else if (is_object($arg)) |
|
| 128 | + } else if (is_object($arg)) |
|
| 130 | 129 | { |
| 131 | 130 | $params[] = get_class($arg); |
| 132 | - } |
|
| 133 | - else if (is_string($arg)) |
|
| 131 | + } else if (is_string($arg)) |
|
| 134 | 132 | { |
| 135 | 133 | $params[] = 'string(' . $arg . ')'; |
| 136 | - } |
|
| 137 | - else if (is_int($arg)) |
|
| 134 | + } else if (is_int($arg)) |
|
| 138 | 135 | { |
| 139 | 136 | $params[] = 'int(' . $arg . ')'; |
| 140 | - } |
|
| 141 | - else if (is_float($arg)) |
|
| 137 | + } else if (is_float($arg)) |
|
| 142 | 138 | { |
| 143 | 139 | $params[] = 'float(' . $arg . ')'; |
| 144 | - } |
|
| 145 | - else if (is_bool($arg)) |
|
| 140 | + } else if (is_bool($arg)) |
|
| 146 | 141 | { |
| 147 | 142 | $params[] = 'bool(' . ($arg ? 'true' : 'false') . ')'; |
| 148 | - } |
|
| 149 | - else |
|
| 143 | + } else |
|
| 150 | 144 | { |
| 151 | 145 | $params[] = (string) $arg; |
| 152 | 146 | } |
@@ -57,7 +57,7 @@ |
||
| 57 | 57 | return; |
| 58 | 58 | } |
| 59 | 59 | |
| 60 | - if ( ! $this->config->get('understand-laravel.log_types.exception_log.enabled')) |
|
| 60 | + if (!$this->config->get('understand-laravel.log_types.exception_log.enabled')) |
|
| 61 | 61 | { |
| 62 | 62 | return; |
| 63 | 63 | } |