Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:120790
Date: Wed, 12 Jul 2023 05:25:46 +0200
To: internals@lists.php.net
Message-ID: <906b41e4-dcbd-4a03-9d10-33723ce1c708@Spark>
In-Reply-To: <29da91e9-d649-49c5-a415-46a84d22d531@Spark>
References: <29da91e9-d649-49c5-a415-46a84d22d531@Spark>
MIME-Version: 1.0
Content-Type: multipart/alternative; boundary="64ae1d3f_4ccc316d_24c1"
Subject: Reflection*::hasAttribute()
From: robin.chalas@gmail.com (Robin Chalas)

--64ae1d3f_4ccc316d_24c1
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
Content-Disposition: inline

Hi internals,

It is common attributes to be used as markers. Just like a marker interfa=
ce with no methods, such attributes don=E2=80=99t have any parameter.

Examples from Symfony:
- =23=5BIgnore=5D indicating that a property should be skipped in a seria=
lization contextl
- =23=5BExclude=5D telling the dependency injection container that a give=
n class should be skipped from automatic service registration


=46rom Doctrine ORM:
- =23=5BId=5D to tell which property of en entity class should be mapped =
as primary key

Those are meant to opt in/out some behavior through detection only. But w=
ith the current API, such code needs to call =60getAttributes()=60 while =
it doesn't care about the returned ReflectionAttribute instances.
Hence I=E2=80=99m wondering if we could add a =60hasAttribute()=60 method=
 to the reflectors that are targetable by attributes.

Does that make sense=3F
If it does, do I need to open an R=46C for that feature or is a pull requ=
est be enough=3F

Anyway, I=E2=80=99d be happy to work on the implementation as a first con=
tribution to PHP.
Looking forward to your feedback.

Thanks.
Robin


Robin Chalas
Principal Engineer =40Les-Tilleuls.coop
03 66 72 43 94
robin=40les-tilleuls.coop
82 Rue Winston Churchill - 59160 Lomme

--64ae1d3f_4ccc316d_24c1--