Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:117338
Sender: Miguel Rosales <m.rosales.sueiro@gmail.com>
To: internals@lists.php.net
References: <4724184.31r3eYUQgx@come-prox15amd>
 <37421f61-4e2e-53d3-56d0-022a3c21ed2e@gmx.de>
Message-ID: <4f8bad56-267a-9279-1e57-66fd10b8874f@gmail.com>
Date: Tue, 15 Mar 2022 20:07:52 +0100
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.16; rv:52.0)
 Gecko/20100101 PostboxApp/7.0.54
MIME-Version: 1.0
In-Reply-To: <37421f61-4e2e-53d3-56d0-022a3c21ed2e@gmx.de>
Content-Type: multipart/alternative;
 boundary="------------D78288F0D73220823C92E03C"
Content-Language: en-US
Subject: Re: [PHP-DEV] [VOTE] Undefined Variable Error Promotion
From: mrosalessueiro@gmail.com (Miguel Rosales)

--------------D78288F0D73220823C92E03C
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 8bit

Christoph M. Becker wrote on 15/3/22 19:17:
> On 15.03.2022 at 17:54, Côme Chilliet wrote:
>
>> Le lundi 14 mars 2022, 18:18:55 CET Mark Randall a écrit :
>>
>>> I have started the vote for promoting undefined variable access to throw
>>> an Error exception.
>>>
>>> The vote will run for 2 weeks until March 28th 2022.
>>>
>>> https://wiki.php.net/rfc/undefined_variable_error_promotion
>> This does not mention properties, are they concerned by the change? What about array keys?
> The RFC states:
>
> | If the code does not currently emit a “Warning: Undefined variable
> | $varname” 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.

--------------D78288F0D73220823C92E03C--