Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:74231
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 74.125.82.48 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <53750338.5070004@marc-bennewitz.de>
References: <53750338.5070004@marc-bennewitz.de>
Date: Thu, 15 May 2014 20:17:09 +0200
Message-ID: <CALtuQzAaTKpUbxELqPksgyCwe25z6ZxuZv+65fFdP46J6xY5-A@mail.gmail.com>
To: Marc Bennewitz <php@marc-bennewitz.de>
Cc: PHP Internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=001a11c38e4687172204f97450ef
Subject: Re: [PHP-DEV] PR 659 / #60687: directly call a callable
From: krebs.seb@gmail.com (Sebastian Krebs)

--001a11c38e4687172204f97450ef
Content-Type: text/plain; charset=UTF-8

2014-05-15 20:11 GMT+02:00 Marc Bennewitz <php@marc-bennewitz.de>:

> Hi list
>
> Since the hype-hint "callable" is out it doesn't catch if the callable
> argument is directly callable - so currently using "call_user_func[_array]"
> is a requirement.
>
> @see http://3v4l.org/OYAgk
>
> class MyClass {
>     public static function test() {
>         var_dump(__METHOD__);
>     }
> }
>
> function call(callable $f) {
>     $f();
> }
>
> call('MyClass::test');
>
> I have created a PR to make this one possible and would very like to see
> this one fixed from PHP-5.6 down to PHP-5.4 were the callable type-hint was
> added.
>
> Is there something I have to do ?
> (I'll be not available after the next week for 3 month)



Hi

I've fixed it :)

http://3v4l.org/et7MX

<?php

class MyClass {
    public static function test() {
        var_dump(__METHOD__);
    }
}

function call(callable $f) {
    $f();
}

call(['MyClass','test']);


Regards,
Sebastian


>
>
> Marc
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>


-- 
github.com/KingCrunch

--001a11c38e4687172204f97450ef--