Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:86928
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmx.de designates 212.227.17.21 as permitted sender)
Message-ID: <55904980.7020404@gmx.de>
Date: Sun, 28 Jun 2015 21:22:40 +0200
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0
MIME-Version: 1.0
To: Marc Bennewitz <dev@mabe.berlin>, internals@lists.php.net
CC: Aaron Piotrowski <aaron@icicle.io>, 
 Nikita Popov <nikita.ppv@gmail.com>
References: <CAJW__o0D_DadKJCyZbMhDiEVpO-pzpWwKV5Ht=ukK1MOBCLCKg@mail.gmail.com> <558C51CA.9080905@mabe.berlin> <558C5705.9040604@gmx.de> <558C58E9.9010008@mabe.berlin> <8078246D-54BB-4D83-ADFC-47EA17929B25@icicle.io> <559044BD.2090905@mabe.berlin>
In-Reply-To: <559044BD.2090905@mabe.berlin>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 8bit
Subject: Re: [PHP-DEV] Headsup: PHP7 feature freeze
From: cmbecker69@gmx.de (Christoph Becker)

Marc Bennewitz wrote:

> On 06/25/2015 09:48 PM, Aaron Piotrowski wrote:
>>
>>> On Jun 25, 2015, at 2:39 PM, Marc Bennewitz <dev@mabe.berlin
>>> <mailto:dev@mabe.berlin>> wrote:
>>>
>>> Nice to see this - didn't noted it in the last month :)
>>>
>>> But there is one edge case that is not handled by PHP-7 at current
>>> behavior;
>>> http://3v4l.org/HkRQ7
>>>
>>> class Foo {
>>>    public static function __callStatic($m, $args) {
>>>        var_dump($m);
>>>    }
>>>    public function __call($m, $args) {
>>>        var_dump($m);
>>>    }
>>> }
>>>
>>> $callable = [new Foo, ''];
>>> $callable(); // string(0) ""
>>>
>>> $callable = 'Foo::';
>>> $callable(); // Fatal error: Uncaught Error: Call to undefined
>>> function Foo::()
>>>
>>> This behavior is inconsistent!
>>>
>>> Thanks
>>> Marc
>>>
>>
>> Interesting, I didn’t consider that an empty method name should invoke
>> __callStatic(). I’ll look into fixing this sometime today or tomorrow.
> 
> Is this edge case addressed now?
> My PR has been closed now where this edge case was addressed, too.

That is supposed to be addressed by
<http://git.php.net/?p=php-src.git;a=commit;h=ba67fc221890aaa395912aefebb96155afe671c1>.
 Can you please confirm with a recent master?  (The Windows snapshots
currently in build progress are likely to have this commit included, too.)

-- 
Christoph M. Becker