1 | <?php |
||
2 | /** |
||
3 | * Class DatabaseLog |
||
4 | * |
||
5 | * @filesource DatabaseLog.php |
||
6 | * @created 04.01.2018 |
||
7 | * @package chillerlan\Logger\Output |
||
8 | * @author Smiley <[email protected]> |
||
9 | * @copyright 2018 Smiley |
||
10 | * @license MIT |
||
11 | */ |
||
12 | |||
13 | namespace chillerlan\Logger\Output; |
||
14 | |||
15 | use chillerlan\Database\Database; |
||
0 ignored issues
–
show
|
|||
16 | use chillerlan\Settings\SettingsContainerInterface; |
||
17 | |||
18 | class DatabaseLog extends LogOutputAbstract{ |
||
19 | |||
20 | /** |
||
21 | * @var \chillerlan\Database\Database |
||
22 | */ |
||
23 | protected $db; |
||
24 | |||
25 | /** @noinspection PhpMissingParentConstructorInspection */ |
||
26 | /** |
||
27 | * DatabaseLog constructor. |
||
28 | * |
||
29 | * @param \chillerlan\Settings\SettingsContainerInterface|null $options |
||
30 | * @param \chillerlan\Database\Database $db |
||
31 | */ |
||
32 | public function __construct(SettingsContainerInterface $options, Database $db){ |
||
33 | $this->options = $options; |
||
0 ignored issues
–
show
$options is of type chillerlan\Settings\SettingsContainerInterface , but the property $options was declared to be of type chillerlan\Logger\LogOptions . Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?
Our type inference engine has found a suspicous assignment of a value to a property. This check raises an issue when a value that can be of a given class or a super-class is assigned to a property that is type hinted more strictly. Either this assignment is in error or an instanceof check should be added for that assignment. class Alien {}
class Dalek extends Alien {}
class Plot
{
/** @var Dalek */
public $villain;
}
$alien = new Alien();
$plot = new Plot();
if ($alien instanceof Dalek) {
$plot->villain = $alien;
}
![]() |
|||
34 | $this->db = $db; |
||
35 | |||
36 | $this->db->connect(); |
||
37 | } |
||
38 | |||
39 | /** |
||
40 | * @inheritdoc |
||
41 | */ |
||
42 | protected function __log(string $level, string $message, array $context = []){ |
||
43 | |||
44 | // TODO: $context, tests |
||
45 | $this->db->insert |
||
46 | ->into($this->options->dbLogTable) |
||
47 | ->values([ |
||
48 | 'level' => $level, |
||
49 | 'message' => $message, |
||
50 | 'context' => json_encode($context), |
||
51 | 'time' => time(), |
||
52 | ]) |
||
53 | ->query() |
||
54 | ; |
||
55 | } |
||
56 | |||
57 | /** |
||
58 | * @inheritdoc |
||
59 | */ |
||
60 | public function close():LogOutputInterface{ |
||
61 | $this->db->disconnect(); |
||
62 | |||
63 | return $this; |
||
64 | } |
||
65 | |||
66 | } |
||
67 |
The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g.
excluded_paths: ["lib/*"]
, you can move it to the dependency path list as follows:For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths