Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:115084
To: "internals@lists.php.net" <internals@lists.php.net>
Thread-Topic: [PHP-DEV] [Vote] make Reflection*#setAccessible() no-op
Thread-Index: AQHXaEHZJXBjURNNnEC8ZpXq9kIrBKsiOhy6
Date: Wed, 23 Jun 2021 23:20:26 +0000
Message-ID:
 <DM6PR07MB66186E32B5138E4253619F55F9089@DM6PR07MB6618.namprd07.prod.outlook.com>
References:
 <CADyq6sLyxKQgK4FwCH2w+uC=goAUK2J6mbz31rZvB=exsY8xzw@mail.gmail.com>
In-Reply-To:
 <CADyq6sLyxKQgK4FwCH2w+uC=goAUK2J6mbz31rZvB=exsY8xzw@mail.gmail.com>
Accept-Language: en-CA, en-US
Content-Language: en-CA
x-ms-exchange-messagesentrepresentingtype: 1
x-tmn: [Yu6yiPu6WfCSb6S8Guf+IKNni+lPLkezz+mxlPtZ3Cmp+2goPfbURX2GgZe+KnF9]
x-ms-publictraffictype: Email
x-incomingheadercount: 44
x-eopattributedmessage: 0
x-ms-office365-filtering-correlation-id: cafe4a6f-32df-44e9-3ae5-08d9369d7bbd
x-ms-traffictypediagnostic: BN8NAM11HT190:
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info:
 3AICspIfcAKysOLgdDIy695XA5quwlE+IP8faeM8y02OShXML0ju3CDhHturEOaCowJZXaB+y0SE/EZm3+NM9qc0SVp6RfBXa+1/aPuUDWp8yrcNWiIgqUbGPQma6EkTl/jGESlx/gCVAP8+R169gYxIQlZt8qNWAGOcjd6cWKlZLGIcZVxEUAXwPeeqJ62fIBbrWcKh8kwTyjuAgNDu6qYZq5Ised9jJ97S6ftzEb2KQ9HC7B5Wr37F74Mu0smaa/6wrZn6QYRBi0g/G1DkcnXDbB1ssM783dIXnFL74ehVMfMoaVDwyUAHetw+g0saRTEUlSzBPX2GDYwX8WVvQOzs1LkNjaIVQbsO6oON2PH2bAdl9tM66otd7qXScZxpJP4IcuaA5HSLQteJ0sXabMmUJPJLfOZo7CuJRjwThl0Gyl71sVmIrdwc2e3RMrXpMY0umzMt3bMh8oba2nQtfw==
x-ms-exchange-antispam-messagedata:
 KsH6xNStZUDo3N9R8ehwRO6lIoehIZX1yOKEO/8qcOv5aiklcaMWpIBq4y2yAw2VYLAa+H+xCLzl9agZ/GFGp2Ci7Tvnqke1+M8daOgC3WLDFhDZ+EypTAxYwv6BCH7JjgJ++Y6mV0p/riADgJlCatmtdPzeJ8Vou3JekD1l4hW1h0WtMUTCTSdWnovJ2vuWIB9s7ADunZWfI5fBHc/FgA==
x-ms-exchange-transport-forked: True
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: hotmail.com
X-MS-Exchange-CrossTenant-AuthAs: Anonymous
X-MS-Exchange-CrossTenant-AuthSource: BN8NAM11FT053.eop-nam11.prod.protection.outlook.com
X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000
X-MS-Exchange-CrossTenant-Network-Message-Id: cafe4a6f-32df-44e9-3ae5-08d9369d7bbd
X-MS-Exchange-CrossTenant-originalarrivaltime: 23 Jun 2021 23:20:26.2490
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Internet
X-MS-Exchange-CrossTenant-id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa
X-MS-Exchange-CrossTenant-rms-persistedconsumerorg: 00000000-0000-0000-0000-000000000000
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN8NAM11HT190
Subject: Re: [PHP-DEV] [Vote] make Reflection*#setAccessible() no-op
From: tysonandre775@hotmail.com (tyson andre)

Mi Marco Pivetta,=0A=
=0A=
> I'm opening the vote for making `Reflection*#setAccessible()`.=0A=
> =0A=
> Voting starts today (2021-06-23) and ends in 14 days (2021-07-07).=0A=
> =0A=
> Vote at https://wiki.php.net/rfc/make-reflection-setaccessible-no-op=0A=
> =0A=
> Discussion: https://marc.info/?l=3Dphp-internals&m=3D162360269505048&w=3D=
2=0A=
> =0A=
> Discussion^2: https://externals.io/message/114841=0A=
=0A=
I'm in favor of this even without adding isAccessible(),=0A=
but just to note:=0A=
=0A=
https://wiki.php.net/rfc/howto mentions:=0A=
=0A=
> Listen to the feedback, and try to answer/resolve all questions.=0A=
> **Update your RFC to document all the issues and discussions.=0A=
> Cover both the positive and negative arguments.** Put the RFC URL into al=
l your replies.=0A=
=0A=
1. This should probably link to the RFC discussions in a References section=
,=0A=
   not everyone who votes reads the mailing list.=0A=
2. https://externals.io/message/114841#114845  is the only thing that resem=
bled an objection for a "Discussion" =0A=
   section or future scope, though=0A=
=0A=
   > I think that isAccessible should be added if any applications actually=
 did depend on ReflectionException=0A=
   > being thrown for correctness - they could throw their own exception if=
 isAccessible was false.=0A=
   > (e.g. for code meant to handle possibly undefined public typed propert=
ies by checking for initialization=0A=
   > then getting the value)=0A=
   >=0A=
   > I can't actually remember needing this (anything other than setAccessi=
ble(true)) personally, though, since `$obj->{$method}(...$args)` could be u=
sed.=0A=
   > I've only used this to access private and protected properties/methods=
.=0A=
=0A=
Thanks,=0A=
Tyson=