[5.2] $this: Cleanup of typehints for phpstan
Summary of Changes
When typehinting $this in our layouts, we use both the full qualified name and a shortened version with an import. phpstan unfortunately does not currently understand the version with the import and thus this unifies this to the long version.
Testing Instructions
Codereview
Link to documentations
Please select:
-
[ ] Documentation link for docs.joomla.org:
-
[X] No documentation changes for docs.joomla.org needed
-
[ ] Pull Request link for manual.joomla.org:
-
[X] No documentation changes for manual.joomla.org needed
Then we should disable the rule in phpstan, is there an issue to check at phpstan for this feature?
Then we should disable the rule in phpstan, is there an issue to check at phpstan for this feature?
exactly
https://github.com/phpstan/phpstan/issues/10936
Issue deferred to PHPStan, change not required in Joomla. Thanks Hannes.
Is this excluded in our PHPStan setup?
no, it isn't.