Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:128229
Precedence: bulk
MIME-Version: 1.0
Date: Fri, 25 Jul 2025 09:33:58 -0700
Message-ID: <CALaXqqTW-r+XoJhcxAp0QEg0c1=azGormzdMfCeOUH=Y2Ga6yA@mail.gmail.com>
Subject: [PHP-DEV] Validating #[\Attribute] targets
To: php internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="000000000000610949063ac384ba"
From: daniel.e.scherzer@gmail.com (Daniel Scherzer)

--000000000000610949063ac384ba
Content-Type: text/plain; charset="UTF-8"

Hi internals,

The #[\Attribute] attribute is used to mark classes as being usable as
attributes. It currently does not trigger any error if added to a trait,
interface, enum, or abstract class. However, if any of those *were* used as
an attribute, calling ReflectionAttribute::newInstance() would fail; if you
are not going to call ReflectionAttribute::newInstance(), then you don't
really need to mark the trait/interface/enum/abstract class as an attribute.

I have proposed https://github.com/php/php-src/pull/19154 to add validation
to the #[Attribute] attribute, and it was suggested that I post here on the
mailing list in case there were any objections. So, are there?

-Daniel

--000000000000610949063ac384ba
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr">Hi internals,<div><br></div><div>The #[\Attribute] attribu=
te is used to mark classes as being usable as attributes. It currently does=
 not trigger any error if added to a trait, interface, enum, or abstract cl=
ass. However, if any of those *were* used as an attribute, calling Reflecti=
onAttribute::newInstance() would fail; if you are not going to call Reflect=
ionAttribute::newInstance(), then you don&#39;t really need to mark the tra=
it/interface/enum/abstract class as an attribute.</div><div><br></div><div>=
I have proposed=C2=A0<a href=3D"https://github.com/php/php-src/pull/19154">=
https://github.com/php/php-src/pull/19154</a> to add validation to the #[At=
tribute] attribute, and it was suggested that I post here on the mailing li=
st in case there were any objections. So, are there?</div><div><br></div><d=
iv>-Daniel</div></div>

--000000000000610949063ac384ba--