Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:112868
MIME-Version: 1.0
References: <CAEPJdu=kopidV44A01Sjv_eLq_fkLsjVuzqXx+j9b9gCahn2Vg@mail.gmail.com>
 <CAOWwgpmwdxnwb+NcHcd59ySwbKKB_1C1nYtPeGkmPXa8F3GXPg@mail.gmail.com>
 <CAESVnVpjyhCH1K5sOHzp8qoa7hUAhRqpfN90JS3CooyFpEOZ2Q@mail.gmail.com> <B802520B-C807-4CA4-A462-EF3E6FB33FF8@stitcher.io>
In-Reply-To: <B802520B-C807-4CA4-A462-EF3E6FB33FF8@stitcher.io>
Date: Wed, 13 Jan 2021 13:43:42 +0100
Message-ID: <CAEPJdun8XD08B+819dex4R1zseMiy==rtDLuPEZXsAeqMM6hQA@mail.gmail.com>
To: Brent Roose <brendt@stitcher.io>
Cc: PHP Internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="000000000000e80e9705b8c77d43"
Subject: Re: [PHP-DEV] #[Deprecated] Attribute
From: kontakt@beberlei.de (Benjamin Eberlei)

--000000000000e80e9705b8c77d43
Content-Type: text/plain; charset="UTF-8"

On Wed, Jan 13, 2021 at 1:05 PM Brent Roose <brendt@stitcher.io> wrote:

> Hi Sara
>
> > On 22 Dec 2020, at 19:54, Sara Golemon <pollita@php.net> wrote:
> >
> > On Tue, Dec 22, 2020 at 12:35 PM Nicolas Grekas <
> > nicolas.grekas+php@gmail.com> wrote:
> >
> >> It would be great to allow adding this attribute on classes. What about
> >> allowing it right now and not bind it to any runtime side-effect? That
> >> would allow static analyzers to do their job. Same for consts and
> >> properties by the way.
> >>
> >> Also, it would be very useful to add named parameters to the attribute,
> >> namely: "package" (the name of the package that declares the
> deprecation)
> >> and "version" (the version of that package that introduced the
> >> deprecation), next to the message.
> >>
> >> This is critical info when building reports of deprecations.
> >>
> >>
> > You could do that now with a polyfill from userspace.  If the annotation
> > need not have an effect, then it's just any other userspace
> implementation.
>
> The difference is that PHP core has the ability to force standarization.
> There's already JetBrains' implementation of #[Deprecated], which Psalm and
> PhpStan also support, but it's not a real standard. Maybe the FIG would one
> day step in to decide these kinds of things, but the reality is that many
> major frameworks don't follow FIG as closely as they used to. I think
> there's value in adding attributes in the core, with the goal only being
> static analysis. It'll allow for consistency and that's a valuable thing.
>

I want to keep #[Deprecated] on other elements than functions / methods out
of this RFC, because they require entirely different implementation
approaches.

We identified in the PR already that this should at some point be
standardized in core, because internal attributes will currently not be
able to support extension through inheritance by userland for
implementation reasons.

My next RFC update will reflect this future scope.

>
> >
> > -Sara
>
>
> Kind regards
> Brent
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: https://www.php.net/unsub.php
>
>

--000000000000e80e9705b8c77d43--