Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:129548
Precedence: list
MIME-Version: 1.0
References: <e906b61e-b951-4eb4-aa0f-700e76dae6a9@app.fastmail.com> <CAOWwgp=L=YaZDdLmFr+paOVEotU1Lt6S4vAuU+HFTNAo0S0hSQ@mail.gmail.com>
In-Reply-To: <CAOWwgp=L=YaZDdLmFr+paOVEotU1Lt6S4vAuU+HFTNAo0S0hSQ@mail.gmail.com>
Date: Thu, 4 Dec 2025 15:18:15 +0100
Message-ID: <CAOWwgpktbtX_vH=xxN943TQQY6+a5PfoEU+h0DdeBkZfbm2AQw@mail.gmail.com>
Subject: Re: [PHP-DEV] [RFC] isReadable/isWriteable property reflection
To: Larry Garfield <larry@garfieldtech.com>
Cc: php internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="000000000000a25e7c06452100f0"
From: nicolas.grekas+php@gmail.com (Nicolas Grekas)

--000000000000a25e7c06452100f0
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Sorry, typo:

I also think the auto-scope is a *not* good idea

Le jeu. 4 d=C3=A9c. 2025 =C3=A0 15:05, Nicolas Grekas <nicolas.grekas+php@g=
mail.com>
a =C3=A9crit :

>
> Le jeu. 6 nov. 2025 =C3=A0 00:28, Larry Garfield <larry@garfieldtech.com>=
 a
> =C3=A9crit :
>
>> In other news, Ilija and I said a year ago that we'd take a swing at
>> adding isReadable/isWriteable methods to ReflectionProperty.  Took a whi=
le,
>> but here we are.  A strangely small RFC from us:
>>
>> https://wiki.php.net/rfc/isreadable-iswriteable
>>
>>
>
> Thanks for this.
> I also think the auto-scope is a good idea. You state that 90% of use
> cases will need this but my experience doesn't back this claim. The only
> cases where I had to check for read/writeable were out of the local scope=
,
> so I'd say 100% of my experience goes against that 90% number ;) Joke
> aside, it'd be just fine to let ppl be explicit. That's better than "oops=
 I
> forgot to give the correct scope" bugs.
>
> About magic methods, one unsets a property only to have __get/__set
> called. Existing code works with this assumption. This means we have to
> return true IMHO. Magic methods are just generic hooks also. Which means
> they should behave the same.
>
> Nicolas
>

--000000000000a25e7c06452100f0
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr">Sorry, typo:<div><br></div><div>I also think the auto-scop=
e is a *not* good idea</div></div><br><div class=3D"gmail_quote gmail_quote=
_container"><div dir=3D"ltr" class=3D"gmail_attr">Le=C2=A0jeu. 4 d=C3=A9c. =
2025 =C3=A0=C2=A015:05, Nicolas Grekas &lt;<a href=3D"mailto:nicolas.grekas=
%2Bphp@gmail.com">nicolas.grekas+php@gmail.com</a>&gt; a =C3=A9crit=C2=A0:<=
br></div><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8e=
x;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir=3D"ltr"=
><div dir=3D"ltr"><br></div><div class=3D"gmail_quote"><div dir=3D"ltr" cla=
ss=3D"gmail_attr">Le=C2=A0jeu. 6 nov. 2025 =C3=A0=C2=A000:28, Larry Garfiel=
d &lt;<a href=3D"mailto:larry@garfieldtech.com" target=3D"_blank">larry@gar=
fieldtech.com</a>&gt; a =C3=A9crit=C2=A0:<br></div><blockquote class=3D"gma=
il_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,2=
04,204);padding-left:1ex">In other news, Ilija and I said a year ago that w=
e&#39;d take a swing at adding isReadable/isWriteable methods to Reflection=
Property.=C2=A0 Took a while, but here we are.=C2=A0 A strangely small RFC =
from us:<br>
<br>
<a href=3D"https://wiki.php.net/rfc/isreadable-iswriteable" rel=3D"noreferr=
er" target=3D"_blank">https://wiki.php.net/rfc/isreadable-iswriteable</a><b=
r>
<br></blockquote><div><br></div><div><br></div>Thanks for this.<br><div>I a=
lso think the auto-scope is a good idea. You state that 90% of use cases wi=
ll need this but my experience doesn&#39;t back this claim. The only cases =
where I had to check for read/writeable=C2=A0were out of the local scope, s=
o I&#39;d say 100% of my experience goes against that 90% number ;) Joke as=
ide, it&#39;d be just fine to let ppl be explicit. That&#39;s better than &=
quot;oops I forgot to give the correct scope&quot; bugs.</div><div><br></di=
v><div>About magic methods, one unsets a property=C2=A0only to have __get/_=
_set called. Existing code works with this assumption. This means we have t=
o return true IMHO. Magic methods are just generic hooks also. Which means =
they should behave the same.</div><div><br></div><div>Nicolas</div></div></=
div>
</blockquote></div>

--000000000000a25e7c06452100f0--