\SoapServer stub is missing old style constructor with php 7

[sakarikl]

Bug report

phpstan stub for \SoapServer seems to reflect php 8 structure for it. With php 7 will give false positives for a call to a old style cosntructor

https://phpstan.org/r/c3acec73-226d-40f4-8a6c-356ae666a50f

Expected output

expected no errors

[ondrejmirtes]

Send a PR to https://github.com/JetBrains/phpstorm-stubs/blob/master/soap/soap.php.

[sakarikl]

It seems to be correct in there.

this worked fine before php 8 support in phpstan

[ondrejmirtes]

It's not correct in there. Compare SoapClient and SoapServer. One has the old style constructor and the other doesn't.

[sakarikl]

Just updated from 0.12.48 to 0.12.50 and this error came up.

[ondrejmirtes]

Because of this change: JetBrains/phpstorm-stubs@f23297f#diff-642148d22d9633b96b4f1007d0831e7eea0148d7fd4fe1ad18bb6086bfd5e2cbR463-R731

The constructors should have been added, not renamed.

[ondrejmirtes]

Fixed phpstan/phpstan-src@4a886db

[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.