PhpStan fixes #364
PhpStan fixes #364
Conversation
| if ($this->errors) { | ||
| throw new Exception('SQL translate error: ' . trim(reset($this->errors), '*'), 0, $sql); | ||
| if ($this->errors !== []) { | ||
| throw new Exception('SQL translate error: ' . trim((string) reset($this->errors), '*'), 0, $sql); |
There was a problem hiding this comment.
it changes behavior, can be null or empty array. And original code is correct.
There was a problem hiding this comment.
I reverted original condition but kept (string) for PhpStan because function reset() can return mixed value.
There was a problem hiding this comment.
For nonempty string[]? I think it must be always string.
| if (in_array($code, [1216, 1217, 1451, 1452, 1701], true)) { | ||
| return new Dibi\ForeignKeyConstraintViolationException($message, $code, $sql); | ||
| if (is_int($code) === true) { | ||
| if (in_array($code, [1216, 1217, 1451, 1452, 1701], true)) { |
There was a problem hiding this comment.
there is no reason for this change, in_array checks type
There was a problem hiding this comment.
Parameter $code can be int or string. In the case of string the if statements can be skipped (better performance and code readability).
What do you think?
There was a problem hiding this comment.
I think in mysql driver it is always int.
| @@ -41,7 +41,8 @@ public function __construct(\mysqli_result $resultSet, bool $buffered) | |||
| */ | |||
| public function __destruct() | |||
| { | |||
| if ($this->autoFree && $this->getResultResource()) { | |||
| if ($this->autoFree) { | |||
| $this->getResultResource(); | |||
There was a problem hiding this comment.
I tested all combinations and I think not. The result of getResultResource() is always true, so it should be called in if statement.
dg
force-pushed
the
master
branch
8 times, most recently
from
138139d to
81a66da
Compare
dg
force-pushed
the
master
branch
3 times, most recently
from
8db0534 to
cc3b09e
Compare
dg
force-pushed
the
master
branch
5 times, most recently
from
17866e0 to
82c45c3
Compare
dg
force-pushed
the
master
branch
3 times, most recently
from
dd46ea7 to
9e71132
Compare
dg
force-pushed
the
master
branch
2 times, most recently
from
f739394 to
86a71dd
Compare
dg
force-pushed
the
master
branch
2 times, most recently
from
ebee17f to
9908f6f
Compare
dg
force-pushed
the
master
branch
2 times, most recently
from
b6a770f to
ab68077
Compare
dg
force-pushed
the
master
branch
14 times, most recently
from
4947fa5 to
04f006a
Compare
I fixed some PhpStan errors.
Resend #363.