Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:95380
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmx.de designates 212.227.15.15 as permitted sender)
To: Rowan Collins <rowan.collins@gmail.com>, internals@lists.php.net
References: <CAOWwgp=uAjaLm0gHCb6HFM+g0S=6itiOKZKk_u0Z4Mc_FimJeg@mail.gmail.com>
 <0b8ebd55-976b-f3a9-9f70-de392f2bd8e6@gmail.com>
Message-ID: <aafeeeb5-95d6-eb04-9dd0-f79c270fd05f@gmx.de>
Date: Mon, 22 Aug 2016 17:40:41 +0200
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101
 Thunderbird/45.2.0
MIME-Version: 1.0
In-Reply-To: <0b8ebd55-976b-f3a9-9f70-de392f2bd8e6@gmail.com>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] BC break: ReflectionMethod::invoke() expects parameter1
 to be object, string given
From: cmbecker69@gmx.de ("Christoph M. Becker")

On 22.08.2016 at 16:38, Rowan Collins wrote:

> On 22/08/2016 12:17, Nicolas Grekas wrote:
>> in ReflectionMethod::invoke():
>>
>> the method doesn't accept a string as first argument anymore, see e.g.:
>>
>> https://3v4l.org/pImmv
> 
> To be precise, in prior versions, the first argument was completely
> ignored for static methods: https://3v4l.org/8o4Hm
> 
> $m->invoke('some_other_class_name', 'baz'); // Warns in 7.1.0beta3,
> ignores everywhere else
> $m->invoke(42, 'baz'); // Warns in 7.1.0beta3, ignores everywhere else
> $m->invoke(new DateTime, 'baz'); // OK in all versions
> $m->invoke($m, 'baz'); // OK in all versions
> 
> It is still ignored if it is a valid object, so the current beta's
> behaviour doesn't actually make a lot of sense. 

Well, see <https://3v4l.org/n8cad>.  Only as of 7.1.0beta3 ::invoke()
matches the behavior of ::invokeArgs().

-- 
Christoph M. Becker