Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:117339
MIME-Version: 1.0
References: <4724184.31r3eYUQgx@come-prox15amd> <37421f61-4e2e-53d3-56d0-022a3c21ed2e@gmx.de>
 <4f8bad56-267a-9279-1e57-66fd10b8874f@gmail.com>
In-Reply-To: <4f8bad56-267a-9279-1e57-66fd10b8874f@gmail.com>
Date: Tue, 15 Mar 2022 14:32:49 -0500
Message-ID: <CAESVnVrKfUNTqo8W8MLc4gUPsXtYTpSbmV53VTZQ67J-r6bOTA@mail.gmail.com>
To: Miguel Rosales <mrosalessueiro@gmail.com>
Cc: PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="00000000000060efdf05da46ddd2"
Subject: Re: [PHP-DEV] [VOTE] Undefined Variable Error Promotion
From: pollita@php.net (Sara Golemon)

--00000000000060efdf05da46ddd2
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Tue, Mar 15, 2022 at 2:08 PM Miguel Rosales <mrosalessueiro@gmail.com>
wrote:
> > The RFC states:
> >
> > | If the code does not currently emit a =E2=80=9CWarning: Undefined var=
iable
> > | $varname=E2=80=9D then it is out of scope for this RFC. This RFC does=
 NOT
> > | apply to array indexes.
> >
> > That explicitly excludes undefined array indexes, and apparently also
> > excludes undefined properties (which raise "Warning: Undefined
property:")
> >
> AFAIU error promotion for undefined properties has already been covered
> by the Deprecate Dynamic Properties RFC
> <https://wiki.php.net/rfc/deprecate_dynamic_properties>, which will
> start throwing errors in 9.0.
>

What I'm hearing is that we also need an RFC for promoting 'Undefined
index...' warnings to Errors as well.

-Sara

--00000000000060efdf05da46ddd2--