Re: BC break: ReflectionMethod::invoke() expects parameter1to be object, string given
| From: | Lauri Kenttä | Date: | Tue, 23 Aug 2016 09:16:53 +0000 |
| Subject: | Re: BC break: ReflectionMethod::invoke() expects parameter1to be object, string given | ||
| References: | 1 2 3 4 5 6 7 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-95399@lists.php.net to get a copy of this message | ||
On 2016-08-23 11:30, Christoph M. Becker wrote:
On 23.08.2016 at 00:25, Levi Morrison wrote:Would there be different checks for static and non-static? - For non-static, accept object or null. (Or only a non-null object?) - For static, accept only null (deprecate also objects). -- Lauri KenttäOn Mon, Aug 22, 2016 at 3:40 PM, Rowan Collins <rowan.collins@gmail.com> wrote:That appears to be the most reasonable compromise presented yet. Thanks Levi! I suggest to wait for Julian, though, who wrote: "I'll prepare a patch exposing my ideas soon."In my opinion, the best "fix", if something needs to change, would be to reject anything other than null; that anything else works appears to just be an oversight.Anyone oppose to emitting E_DEPRECATED for a parameter other than an object or null? This opens up the possibility to use it for something no earlier than 8.0.