1 | <?php |
||
13 | class Recorder extends Component { |
||
14 | |||
15 | /** |
||
16 | * @var mixed |
||
17 | */ |
||
18 | protected $config = []; |
||
19 | |||
20 | /** |
||
21 | * The constructor |
||
22 | */ |
||
23 | 42 | public function __construct() { |
|
26 | |||
27 | /** |
||
28 | * Records an Exception to the database |
||
29 | * @param Exception $e The exception you want to record |
||
30 | * @return false|ExceptionModel |
||
31 | * @throws RecorderFailedException |
||
32 | */ |
||
33 | 9 | public function record(Exception $e) |
|
34 | { |
||
35 | 9 | if ($this->shouldntHandle($e)) { |
|
36 | 3 | return false; |
|
37 | } |
||
38 | |||
39 | $opts = [ |
||
40 | 6 | 'class' => get_class($e), |
|
41 | 6 | 'file' => $e->getFile(), |
|
42 | 6 | 'line' => $e->getLine(), |
|
43 | 6 | 'code' => (is_int($e->getCode()) ? $e->getCode() : 0), |
|
44 | 6 | 'message' => $e->getMessage(), |
|
45 | 6 | 'trace' => $e->getTraceAsString(), |
|
46 | ]; |
||
47 | |||
48 | 6 | $configDependant = array_keys($this->config['collect']); |
|
49 | |||
50 | try { |
||
51 | 6 | foreach ($configDependant as $key) { |
|
52 | 6 | if ($this->canCollect($key)) { |
|
53 | 3 | $value = $this->collect($key, $e); |
|
54 | 3 | if ($value !== null) { |
|
55 | 6 | $opts[$key] = $value; |
|
56 | } |
||
57 | } |
||
58 | } |
||
59 | |||
60 | 6 | return ExceptionModel::create($opts); |
|
61 | } catch (Exception $e) { |
||
62 | $code = (is_int($e->getCode()) ? $e->getCode() : 0); |
||
63 | throw new RecorderFailedException($e->getMessage(), $code, $e); |
||
64 | } |
||
65 | } |
||
66 | |||
67 | /** |
||
68 | * Checks the config to see if you can collect certain information |
||
69 | * @param string $type the config value you want to check |
||
70 | * @return boolean |
||
71 | */ |
||
72 | 6 | private function canCollect($type) { |
|
78 | |||
79 | /** |
||
80 | * @param string $key |
||
81 | * @param Exception $e |
||
82 | * @return array|int|null|string |
||
83 | * @throws Exception |
||
84 | */ |
||
85 | 3 | protected function collect($key, Exception $e = null) { |
|
106 | |||
107 | /** |
||
108 | * Gets the ID of the User that is logged in |
||
109 | * @return integer|null The ID of the User or Null if not logged in |
||
110 | */ |
||
111 | 3 | protected function getUserId() { |
|
119 | |||
120 | /** |
||
121 | * Gets the Method of the Request |
||
122 | * @return string|null Possible values are null or GET, POST, DELETE, PUT, etc... |
||
123 | */ |
||
124 | 3 | protected function getMethod() { |
|
132 | |||
133 | /** |
||
134 | * Gets the input data of the Request |
||
135 | * @return array|null The Input data or null |
||
136 | */ |
||
137 | 6 | protected function getData() { |
|
145 | |||
146 | /** |
||
147 | * Gets the URL of the Request |
||
148 | * @return string|null Returns a URL string or null |
||
149 | */ |
||
150 | 3 | protected function getUrl() { |
|
158 | |||
159 | /** |
||
160 | * Returns the IP from the request |
||
161 | * |
||
162 | * @return string |
||
163 | */ |
||
164 | protected function getIp() { |
||
167 | |||
168 | /** |
||
169 | * Gets the status code of the Exception |
||
170 | * @param Exception $e The Exception to check |
||
171 | * @return string|integer The status code value |
||
172 | */ |
||
173 | 3 | protected function getStatusCode(Exception $e) { |
|
180 | |||
181 | /** |
||
182 | * This function will remove all keys from an array recursively as defined in the config file |
||
183 | * @param array $data The array to remove keys from |
||
184 | * @return void |
||
185 | */ |
||
186 | 12 | protected function excludeKeys(array $data) { |
|
198 | } |
||
199 |
If you access a property on an interface, you most likely code against a concrete implementation of the interface.
Available Fixes
Adding an additional type check:
Changing the type hint: