Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:127892
Precedence: bulk
MIME-Version: 1.0
References: <CALaXqqRKy4_y3HkO6dVhVcOjoHOp=HYdsU7v3SVZQ8kr1EfgLA@mail.gmail.com>
In-Reply-To: <CALaXqqRKy4_y3HkO6dVhVcOjoHOp=HYdsU7v3SVZQ8kr1EfgLA@mail.gmail.com>
Date: Fri, 4 Jul 2025 15:49:07 -0700
Message-ID: <CALaXqqSyPt-Dt_RFVrERrD+MV=yVspNfGL9apA+6L6KRidx+Hw@mail.gmail.com>
Subject: [PHP-DEV] Re: [RFC] [Discussion] #[\DelayedTargetValidation] attribute
To: php internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="00000000000016decd0639224f8c"
From: daniel.e.scherzer@gmail.com (Daniel Scherzer)

--00000000000016decd0639224f8c
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Tue, Jun 17, 2025 at 4:26=E2=80=AFPM Daniel Scherzer <daniel.e.scherzer@=
gmail.com>
wrote:

> Hi internals,
>
> I'd like to start the discussion for a new RFC about adding a
> `#[\DelayedTargetValidation]` attribute.
>
> * RFC: https://wiki.php.net/rfc/delayedtargetvalidation_attribute
> * Implementation: https://github.com/php/php-src/pull/18817
>
> --Daniel
>


This discussion has been ongoing for more than 2 weeks, and from what I can
see there have only been 2 concerns/suggestions raised:

* Going in the opposite direction and validating userland attributes at
compile time too - this is counter to the design of attributes, which are
meant to be easy to apply and can even be used when the specified attribute
does not exist, with errors only raised when attributes are parsed.
* Avoid the need for the `#[\DelayedTargetValidation]` attribute by
changing the default behavior - I (and others) have responded about why
this would be problematic, and I have updated the RFC to include changing
the default behavior as a rejected feature with an explanation of the
issues.

If there is no further feedback, I intend to start a vote in a few days.

-Daniel

--00000000000016decd0639224f8c
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div dir=3D"ltr">On Tue, Jun 17, 2025 at 4:26=E2=80=AFPM D=
aniel Scherzer &lt;<a href=3D"mailto:daniel.e.scherzer@gmail.com">daniel.e.=
scherzer@gmail.com</a>&gt; wrote:</div><div class=3D"gmail_quote gmail_quot=
e_container"><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px =
0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir=3D"=
ltr">Hi internals,<div><br></div><div>I&#39;d like to start the discussion =
for a new RFC about adding a `#[\DelayedTargetValidation]` attribute.</div>=
<div><br></div><div>* RFC:=C2=A0<a href=3D"https://wiki.php.net/rfc/delayed=
targetvalidation_attribute" target=3D"_blank">https://wiki.php.net/rfc/dela=
yedtargetvalidation_attribute</a></div><div>* Implementation:=C2=A0<a href=
=3D"https://github.com/php/php-src/pull/18817" target=3D"_blank">https://gi=
thub.com/php/php-src/pull/18817</a></div><div><br></div><div>--Daniel</div>=
</div></blockquote><div><br></div><div><br></div><div>This discussion has b=
een ongoing for more than 2 weeks, and from what I can see there have only =
been 2 concerns/suggestions raised:</div><div><br></div><div>* Going in the=
 opposite direction and validating userland attributes at compile time too =
- this is counter to the design of attributes, which are meant to be easy t=
o apply and can even be used when the specified attribute does not exist, w=
ith errors only raised when attributes are parsed.</div><div>* Avoid the ne=
ed for the `#[\DelayedTargetValidation]` attribute by changing the default =
behavior - I (and others) have responded about why this would be problemati=
c, and I have updated the RFC to include changing the default behavior as a=
 rejected feature with an explanation of the issues.</div><div><br></div><d=
iv>If there is no further feedback, I intend to start a vote in a few days.=
</div><div><br></div><div>-Daniel</div></div></div>

--00000000000016decd0639224f8c--