Error with latest patch

[dereuromark]

With latest patch the master branch has phpstan false positives on errors:

Method Cake\Http\ServerRequest::input() invoked with 0 parameters, at
least 1 required.

Code snippet that reproduces the problem

$data = $this->input();

and

public function input(?callable $callback = null, ...$args)
{
}

Expected output

No error

[dereuromark]

https://phpstan.org/r/d76d2fe8-f093-443b-825c-1f73bc8aac06

[ondrejmirtes]

Fixed: ondrejmirtes/BetterReflection@a155b31 + phpstan/phpstan-src@3af7eaf

[dereuromark]

You might want to allow also issues @ https://github.com/ondrejmirtes/BetterReflection
Since none of us can use that original library anyway due to 7.4+, we all will have to open tickets sooner or later in your fork.

[ondrejmirtes]

@dereuromark Hi, I don't plan to support the ondrejmirtes/BetterReflection fork as a separate project, I'm only interested in bugs that manifest in PHPStan.

And I plan to base the fork on the latest Roave/BetterReflection with a script that will transform 7.4 sources to 7.1, similarly to how I already do it in phpstan-src.

[holtkamp]

@ondrejmirtes, regarding to #3409 (comment), I just tried 0.12.28 and I still have this problem in case an interface is involved:

Parameter #2 $upperCaseFirst of method Translator::translate() is required but parameter #2 $upperCaseFirst of method TranslatorInterface::translate() is optional.

https://phpstan.org/r/b8c4683b-148d-4979-91e7-1e9dfce33310

This might be a different issue, not sure...

[ondrejmirtes]

@holtkamp Please follow #3443.

[github-actions]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.