Bug #72119 [Asn->Nab]: Interface declaration compatibility regression

Edit report at https://bugs.php.net/bug.php?id=72119&edit=1 ID: 72119 Updated by: bwoebi@php.net Reported by: ben dot davies at gmail dot com Summary: Interface declaration compatibility regression -Status: Assigned +Status: Not a bug Type: Bug Package: PHP Language Specification PHP Version: 7.0.6 -Assigned To: levim +Assigned To: Block user comment: N Private report: N New Comment: This is not a bug; your code is a LSP violation which had been fixed in bug #71428. In the concrete example, the Foo::bar() function can be passed null, but Hello::bar() cannot. [This is something we try to fix with nullable parameters/null unions in 7.1.] Previous Comments: ------------------------------------------------------------------------ [2016-04-28 15:18:48] levim@php.net This was changed in commit e9d65160 by Dmitry. It appears to be a fix for a different bug (71978). ------------------------------------------------------------------------ [2016-04-28 11:33:37] ben dot davies at gmail dot com Description: ------------ I wanted to raise this against 7.0.6, but it was not available in the list above, despite being released? There seems to be a regression in interface method declaration compatibility checking. The below test script only fails on 7.0.6. It also only fails if the $baz is type hinted as array. Test script: --------------- <?php interface Foo { public function bar(array $baz = null); } class Hello implements Foo { public function bar(array $baz = []) { } } Expected result: ---------------- N/A Actual result: -------------- Fatal error: Declaration of Hello::bar(array $baz = Array) must be compatible with Foo::bar(array $baz = NULL) in /in/VhEtd on line 7 Process exited with code 255. ------------------------------------------------------------------------ -- Edit this bug report at https://bugs.php.net/bug.php?id=72119&edit=1