Make nullable parameter types explicit #175
Conversation
| @@ -18,7 +18,7 @@ public function __construct(ResponseInterface $response) | |||
| $this->response = $response; | |||
| } | |||
|
|
|||
| public function then(callable $onFulfilled = null, callable $onRejected = null) | |||
| public function then(?callable $onFulfilled = null, ?callable $onRejected = null) | |||
There was a problem hiding this comment.
can we do this without the interface changing? if not, this bumps the minimum requirement of promise to the version that will contains php-http/promise#34
There was a problem hiding this comment.
can we do this without the interface changing?
Yes, this is a purely syntactical change.
It's not, even if people extended the classes. This change doesn't break anything. |
|
thanks a lot! |
|
Thank you! |
|
Thanks a lot for the work here. Would it be possible to cut a new release with this fix included? 🤓 Thanks again for your work on this package! |
|
sure: https://github.com/php-http/httplug/releases/tag/2.4.1 there are currently merge requests on other php-http repositories to do the same cleanup |
What's in this PR?
This PR turns implicit nullable parameter types to explicit ones.
Why?
PHP allows to implicitly declare a nullable parameter type by setting the parameter's default value to
null. However, in PHP 8.4 this syntax has been deprecated.https://wiki.php.net/rfc/deprecate-implicitly-nullable-types
see also php-http/promise#34