Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:120799
MIME-Version: 1.0
References: <29da91e9-d649-49c5-a415-46a84d22d531@Spark> <906b41e4-dcbd-4a03-9d10-33723ce1c708@Spark>
 <3d9f4cac-7840-42c9-be9c-e9fc3bcc030a@app.fastmail.com>
In-Reply-To: <3d9f4cac-7840-42c9-be9c-e9fc3bcc030a@app.fastmail.com>
Date: Thu, 13 Jul 2023 00:05:27 +0200
Message-ID: <CAG9XoMQ1DTGFi82xV87zw5qSX1Yr5jYXq5FK7vAx0Qz0FRRYLA@mail.gmail.com>
To: Larry Garfield <larry@garfieldtech.com>
Cc: php internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="0000000000006e4b460600516a5d"
Subject: Re: [PHP-DEV] Reflection*::hasAttribute()
From: benjamin.morel@gmail.com (Benjamin Morel)

--0000000000006e4b460600516a5d
Content-Type: text/plain; charset="UTF-8"

>
> I think that would require an RFC.
>
> To clarify, you're proposing to replace this:
>
> count((new ReflectionClass(Beep::class))->getAttributes(Ignore::class)) ==
> true
>
> with
>
> (new ReflectionClass(Beep::class))->hasAttribute(Ignore::class)
>
> Right?  That doesn't seem like a huge improvement.
>
> --Larry Garfield



Actually,

if ((new ReflectionClass(Beep::class))->getAttributes(Ignore::class))

would work as well, so basically:

$rc->hasAttribute(Ignore::class) === (bool)
$rc->getAttributes(Ignore::class);

- Benjamin

>

--0000000000006e4b460600516a5d--