Mark Scherer
Mark Scherer
I just did above, with static example non static the same: https://phpstan.org/r/3917f1a6-2bcd-4d49-b79d-c403e25f785d (Enable "strict-rules" in sandbox for this to see) Maybe your sandbox is not running on PHP 8.2? Since...
Or maybe this somehow gets activated by default for PHP 8.2? Would be my guess then - explains the different behavior here. Shouldnt be the case IMO, though. And it...
I stand corrected, it also happens in other PHP envs it seems. But still doesnt explain why "strict-rules" are being applied on a config that is not using it.
``` git clone [email protected]:cakephp/cakephp.git git checkout 5.x composer install composer stan-setup composer phpstan ``` creates those 7 issues mentioned above My setup: PHP 8.2.1 (cli) (built: Jan 13 2023 10:42:44)...
I sth like this viable? https://github.com/cakephp/chronos/compare/2.x...2.x-interval
Downside is that the types on methods wouldnt work, if something expects a native one.
I agree, the docblock here is wrong and should read /** @var string|null */ Especially if there is no constructor making it for sure assigned a not null value.
Isset() does include null check. Is there any way it would not in php?
We should fix this before I tag a first 0.1 release.
@ravage84 I started the Cake5 upgrade are you still interested in this plugin and should be combine forces getting it production ready?