Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:124535
Message-ID: <9f07bda0-48b5-479a-a426-10b64bf657b9@bastelstu.be>
Date: Sun, 21 Jul 2024 16:45:46 +0200
Precedence: bulk
MIME-Version: 1.0
Subject: Re: [PHP-DEV] [RFC] Asymmetric Visibility, v2
To: Larry Garfield <larry@garfieldtech.com>,
 php internals <internals@lists.php.net>
References: <0a6a61cd-f203-4dea-a7f8-97e6b885c52d@app.fastmail.com>
 <b5c0be62-ea39-47fb-a8f8-ab6cd5ee3dab@app.fastmail.com>
Content-Language: en-US
In-Reply-To: <b5c0be62-ea39-47fb-a8f8-ab6cd5ee3dab@app.fastmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
From: tim@bastelstu.be (=?UTF-8?Q?Tim_D=C3=BCsterhus?=)

Hi

On 7/20/24 03:14, Larry Garfield wrote:
> We've made one change since we last discussed it:  Specifically, Ilija realized that __set's behavior is already inconsistent, so supporting it for aviz properties with invisible set would make it even more inconsistent, not less.  For that reason, we've changed the __set behavior such that a non-readonly aviz property will not trigger __set.  Further details are in the RFC, but in short, all of the use cases for that behavior now have better alternatives, such as property types, hooks, and aviz itself.  So there's really no point to falling back to __set in edge cases.
> 
> https://wiki.php.net/rfc/asymmetric-visibility-v2#interaction_with_set_and_unset
> 
> Baring any new developments, we plan to start the vote early next week.

I find it unfortunate that this RFC received a non-trivial change on a 
Saturday after the discussion has been idle for more than a month 
together with the announcement that the vote will begin shortly after 
the weekend. Folks should be given sufficient time to react to the 
newest developments. Personally I plan to give this RFC another read, 
because I already forgot the details after a month of not looking at it, 
but I'm not sure if I'll manage today.

For now:

Is there any implementation of this RFC? The RFC text only references 
the https://github.com/php/php-src/pull/9257 PR hasn't been updated 
since December 2022.

Best regards
Tim Düsterhus