Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:128109
Message-ID: <00f8137b-2760-4caa-97ca-1bc7c1195a0e@bastelstu.be>
Date: Fri, 18 Jul 2025 15:04:46 +0200
Precedence: bulk
MIME-Version: 1.0
Subject: Re: [PHP-DEV] [RFC] Readonly property hooks
To: Larry Garfield <larry@garfieldtech.com>,
 php internals <internals@lists.php.net>
References: <1e8634d7-ac1a-4025-b4e2-1948aabf5251@app.fastmail.com>
 <CAMzRQTdDNsF_LPpNKUbVROTWjDbN2uDw-j2PrL3xp7qASA62GA@mail.gmail.com>
 <be533915-2b0d-460d-a5ea-76fe98fc4080@app.fastmail.com>
 <f4e4057b19a5ed9de7ad92215e78d58c@bastelstu.be>
 <cedce076-82de-4391-8a03-409690added9@app.fastmail.com>
 <CAOWwgp=o0PpE2b8TD9reKvO053UAu5RxTaFA7g+Huu7GozxrhQ@mail.gmail.com>
 <6acab95a554fe5e188364840ea36d2b7@bastelstu.be>
 <cd86dc08-9aa8-4820-b367-fbd823db92e7@app.fastmail.com>
Content-Language: en-US
In-Reply-To: <cd86dc08-9aa8-4820-b367-fbd823db92e7@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/10/25 17:34, Larry Garfield wrote:
> Nick previously suggested having the get-hook's first return value cached; it would still be subsequently called, so any side effects would still happen (though I don't know why you'd want side effects), but only the first returned value would ever get returned.  Would anyone find that acceptable?  (In the typical case, it would be the same as the current $this->foo ??= compute() pattern, just with an extra cache entry.)

I'm seeing this proposal has already been dropped, but to spell it out 
explicitly:

No, I would not find it acceptable for side effects to happen once 
again, but the return value ignored.

And when dropping the "side effects run once again" part, you arrive at 
an 'init' hook, which I would be in favor of, since it would provide 
semantics that are sound with regard to user expectations.

"Cached get" is just init with extra confusion.

Best regards
Tim Düsterhus