Mark ob_start() callback parameter nullable

[haszi]

Mark the $callback callable parameter of ob_start() as nullable. The parameter is already nullable and is needed for using the default output handler with user-defined chunk_size and/or flags.

[haszi]

@kocsismate For your review as this is strictly a stub file change.

[kocsismate]

Nice catch, thank you!

[VincentLanglet]

Hi @kocsismate,

Stubs of PHP 8.0, 8.1, 8.2 and 8.3 branches are currently wrong and some tools like PHPStan are using them.

I know some of these version are EOL but since it's only a stub update would it be ok to backport this PR ?
I can make the PR if it's ok.

[kocsismate]

@VincentLanglet I can backport it for PHP 8.3 for sure, but we have never done similar changes in EOL or security fix only branches.

AFAIK PHPStan has its own sources of type info besides the official PHP stubs, so they could fix the type info somewhere else.

[kocsismate]

@VincentLanglet Fix for PHP 8.3: 701f3a1

[VincentLanglet]

Phpstan rely a lot on PHP stub for php 8+ and the override is not always easy.

But it's ok, a fix was made on phpstan to automatically update the phpdoc to add "|null" when the default value is null.

[kocsismate]

Phpstan rely a lot on PHP stub for php 8+ and the override is not always easy.

Thanks for the info! When I had a discussion with Ondrej a few years ago, he told me that multiple type sources are merged together. But maybe it was related to other symbols, or it's also possible that I didn't exactly understood what he mentioned.

[VincentLanglet]

Phpstan rely a lot on PHP stub for php 8+ and the override is not always easy.

Thanks for the info! When I had a discussion with Ondrej a few years ago, he told me that multiple type sources are merged together. But maybe it was related to other symbols, or it's also possible that I didn't exactly understood what he mentioned.

Yes, PHPStan rely on multiple source

• PHPStorm stubs
• Some internals stubs
• PHP stubs https://github.com/phpstan/php-8-stubs is generated from php-src