Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:76676
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.214.181 as permitted sender)
MIME-Version: 1.0
Sender: thegreatall@gmail.com
In-Reply-To: <8DC84B14-42BB-466D-B07A-03ADB68EFA34@ajf.me>
References: <235C617B-25E1-4185-A784-38EF1D7BFFDC@ajf.me>
	<CAFMT4NpziftUPjPQY9n9Qv5uA6-qV4_EkdTOk8=ZnK5JO19zoQ@mail.gmail.com>
	<38FC0B11-5E15-43E9-8467-347359970D46@ajf.me>
	<53F1372B.9080207@sugarcrm.com>
	<CAFMT4Npbq_tvanY3qmZrp4FxPotD7c_M9ixJ1k_teL2ey3=yow@mail.gmail.com>
	<8DC84B14-42BB-466D-B07A-03ADB68EFA34@ajf.me>
Date: Mon, 18 Aug 2014 21:21:29 -0500
Message-ID: <CAMVzVHPAh-q74zbEU-PYNszhDtE1-_kkt_HCCXnEEBDSUHARgw@mail.gmail.com>
To: Andrea Faulds <ajf@ajf.me>
Cc: Levi Morrison <levim@php.net>, Stas Malyshev <smalyshev@sugarcrm.com>, 
	PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=089e01160bf8375f2c0500f22501
Subject: Re: [PHP-DEV] [VOTE][RFC] Closure::call()
From: nathan.bruer@starin.biz (Nathan)

--089e01160bf8375f2c0500f22501
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

I will +1 this as I have had to deal with some of these struggles, as a
suggestion though:

"Like the bind(To) methods, a static class cannot be bound (using =E2=86=92=
call
will fail)"

I would like to see this function work like
Closure::bind()/Closure::bindTo() and give the option for the $newScope
option.

> why would you need a closure that is not static,
> but is not bound to any object?

I think I can answer this, I wrote a project which allows for "hooks" to be
installed in areas of the code. I made it so each [static] method has a
"before" and "after" which may be tied to it. This allows for things like:
"beforeUpdate()" or "afterUpdate" or "afterCreate" exc... The framework
handled the installation and execution of all hooks that a dev may have
wished to install. If a method like this was available it would use much
less resources and be faster in theory because the PHP engine would not
need to create an entirely new object/closure every time I wished to
execute the method.

Simple example of what I spoke of above:
https://gist.github.com/allada/b7e880104e3a7eb13a63

Notice how you don't need to create an entirely new object just to call a
method inside a scope. This allows a dev to make their closure have access
to $this just as if it was installed inside the function it's hooking into.


On Mon, Aug 18, 2014 at 7:04 PM, Andrea Faulds <ajf@ajf.me> wrote:

>
> On 18 Aug 2014, at 00:40, Levi Morrison <levim@php.net> wrote:
>
> > On Sun, Aug 17, 2014 at 5:13 PM, Stas Malyshev <smalyshev@sugarcrm.com>
> wrote:
> >> Hi!
> >>
> >>> It does, but maybe you missed it. Under the Proposal section:
> >>>
> >>>> To solve this, we relax the current invariant of scoped closures
> >>>> having to be bound, and add a new parameter to bind(To) to produce
> >>>> an unbound, scoped closure, like so:
> >>
> >> Indeed, I missed it - I was looking for bindTo. I think it should be
> >> made more prominent, and I'm not sure what closure that is not static,
> >> but does not have a bound object, means. What happens if you call it? =
It
> >> sounds a bit strange concept - why would you need a closure that is no=
t
> >> static, but is not bound to any object?
> >
> > Hence the vote was cancelled while it is reworked.
>
> To clarify: The RFC has been reworked now, I actually changed the RFC and
> patch just before I cancelled the vote. Review appreciated before vote is
> reopened.
>
> --
> Andrea Faulds
> http://ajf.me/
>
>
>
>
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>

--089e01160bf8375f2c0500f22501--