@@ -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 | } |