Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:120863
Feedback-ID: i835c46d5:Fastmail
User-Agent: Cyrus-JMAP/3.9.0-alpha0-592-ga9d4a09b4b-fm-defalarms-20230725.001-ga9d4a09b
Mime-Version: 1.0
Message-ID: <19f70187-09d5-4f60-ba01-265f2e97f7cc@app.fastmail.com>
Date: Tue, 25 Jul 2023 19:09:17 +0100
To: =?UTF-8?Q?Bj=C3=B6rn_Larsson?= <internals@lists.php.net>
Content-Type: multipart/alternative;
 boundary=9be6434eb2e346498cae92052a284b82
Subject: New reflection methods for working with attributes
From: php@ollie.codes ("Ollie Read")

--9be6434eb2e346498cae92052a284b82
Content-Type: text/plain

Hello all,

A while back, I wrote a lengthy post about suggested improvements for reflection, but I would like to come back to address additional methods for dealing with attributes. (I have an open git issue here: https://github.com/php/php-src/issues/8489)

I'd like to introduce the following three methods on all reflect classes that have attributes:

hasAttribute(string $name, int $flags = 0): bool

Uses the same filtering as getAttributes(), but returns true if a match is found, false otherwise.

getAttribute(string $name, int $flags = 0): ?ReflectionAttribute

Also uses the same filtering as getAttributes(), except that it will return an instance of ReflectionAttribute if one is found, null if none are found, and will throw an exception of more than 1 is found.

getNumberOfAttributes(?string $name = null, int $flags = 0): int

Again, uses the same filtering as getAttributes(), and is to attributes what getNumberOfParameters() is to parameters. I appreciate that this methods use may not be obvious, but I wanted to include it for consistency.

I'm relatively confident that I have worked out the best method of implementing these, and contributors have agreed to merge a PR, which I will start shortly, should there be no objections.

---
Best Regards,
*Ollie Read*

--9be6434eb2e346498cae92052a284b82--