Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:122967
Date: Fri, 05 Apr 2024 15:24:37 +0200
To: internals@lists.php.net
Subject: =?US-ASCII?Q?Re=3A_=5BPHP-DEV=5D_Proposal=3A_retrieve_line=2C_filena?=
 =?US-ASCII?Q?me_and_if_user_defined_for_ReflectionAttribute?=
User-Agent: K-9 Mail for Android
In-Reply-To: <CAF-fPAk0JxToi=dEupJHOEhsb3QiA-M1x_9znAFXfDSbCxvNzQ@mail.gmail.com>
References: <CAF-fPAk0JxToi=dEupJHOEhsb3QiA-M1x_9znAFXfDSbCxvNzQ@mail.gmail.com>
Message-ID: <E631D76A-7A87-4A37-9978-681D5EDD0411@php.net>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain;
 charset=utf-8
Content-Transfer-Encoding: quoted-printable
From: derick@php.net (Derick Rethans)

On 5 April 2024 15:09:42 CEST, Joel Wurtz <jwurtz@jolicode=2Ecom> wrote:
>Hello everyone,
>
>Like a lot of libraries, we offer the possibility to configure behaviors
>with Attributes=2E However in some cases it's wrongly configured by the u=
ser
>and this wrong configuration cannot be detected on the attribute
>constructor but afterwards=2E
>
>In this case we may want to pinpoint which attribute (in which file and a=
t
>which line) cause this bad configuration=2E Since there was no method to
>retrieve those information in the ReflectionAttribute I proposed a PR
>https://github=2Ecom/php/php-src/pull/13889 to add those informations=2E
>
>I do believe this will allow better DX for end user when correctly used,

Would it make sense to not only add this for ReflectionAttribute, but also=
 Function and/or others?

Functions have a range of lines associate with them=2E Could Attributes ha=
ve that too?

cheers
Derick