Fix: Remove the Debug ignore errors. #7796
Conversation
MGatner left a comment
MGatner left a comment There was a problem hiding this comment.
Thanks again! Some more comments on this one.
system/Log/Logger.php Outdated
| * @var array | ||
| */ | ||
| public $logCache; | ||
| public array $logCache; |
There was a problem hiding this comment.
This is a breaking change for class extensions. Please explain or revert.
There was a problem hiding this comment.
Hi @MGatner , thanks for your reply.
I have now restored this attribute to its original state.
The reason I will make this change is because of the following.
$ignoreErrors[] = [ 'message' => '#^Property CodeIgniter\\\\Log\\\\Logger\\:\\:\\$logCache \\(array\\) on left side of \\?\\? is not nullable\\.$#', 'count' => 1, 'path' => __DIR__ . '/system/Debug/Toolbar/Collectors/Logs.php', ];
There was a problem hiding this comment.
That is not a "type specified" error. We can tackle that in separate PR.
system/Debug/Timer.php Outdated
| * @param string $name The name of the timer | ||
| * @param callable $callable callable to be executed | ||
| * @param string $name The name of the timer | ||
| * @phpstan-param callable():mixed $callable callable to be executed |
There was a problem hiding this comment.
| * @phpstan-param callable():mixed $callable callable to be executed | |
| * @phpstan-param callable(): mixed $callable callable to be executed |
system/Debug/Toolbar.php Outdated
| * | ||
| * @codeCoverageIgnore | ||
| * | ||
| * @return void |
There was a problem hiding this comment.
| * @return void | |
| * @return void | |
| * @phpstan-return never|void |
system/Debug/Exceptions.php Outdated
| * | ||
| * @codeCoverageIgnore | ||
| * | ||
| * @return void |
There was a problem hiding this comment.
| * @return void | |
| * @return void | |
| * @phpstan-return never|void |
system/Debug/Exceptions.php Outdated
| * | ||
| * @codeCoverageIgnore | ||
| * | ||
| * @return void |
There was a problem hiding this comment.
| * @return void | |
| * @return void | |
| * @phpstan-return never|void |
system/Debug/Exceptions.php Outdated
| /** | ||
| * Given an exception and status code will display the error to the client. | ||
| * | ||
| * @return void |
There was a problem hiding this comment.
| * @return void | |
| * @return void | |
| * @phpstan-return never|void |
kai890707 force-pushed the 20230808-debug-phpstan branch from 2122069 to 6dfc38b Compare 4 participants
Description
See #7731
The remaining two sections of the Debug have not been corrected.
The viewer attribute refers to the RendererInterface, but the RendererInterface has undefined method of
getPerformanceData()andgetData()methods.Maybe replace the viewer attribute reference with
CodeIgniter\View\View?After issuing this PR, I tried to change the reference attribute and phpstan does not show Debug section related errors.
I will update the commit to this PR if needed.
before the change viewer attribute refersphpstan-baseline.php:system/Debug/Toolbar/Collectors/Views.php:after the change viewer attribute refersphpstan-baseline.php:system/Debug/Toolbar/Collectors/Views.php:Checklist: