Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:78296
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain seld.be designates 74.125.82.42 as permitted sender)
Message-ID: <544A1BF5.3090708@seld.be>
Date: Fri, 24 Oct 2014 10:29:25 +0100
User-Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:24.0) Gecko/20100101 Thunderbird/24.6.0
MIME-Version: 1.0
To: internals@lists.php.net
References: <6E04B9BE-854E-4112-8C74-7D90BD8BFE95@ajf.me>
In-Reply-To: <6E04B9BE-854E-4112-8C74-7D90BD8BFE95@ajf.me>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
Subject: Re: [PHP-DEV] [RFC] Readonly Properties
From: j.boggiano@seld.be (Jordi Boggiano)

On 24/10/2014 00:36, Andrea Faulds wrote:
> Here’s another RFC: https://wiki.php.net/rfc/readonly_properties
>
> It proposes, with a working implementation, a modifier for properties
> that makes them readable and writeable from different scopes.
>
> Since I am a big proponent of including specification patches in new
> RFCs, I have decided to put my money (or rather, time) where my mouth
> is and I have actually written a specification patch before writing
> an RFC. I would love to see this become the new standard for RFCs
> affecting the language.

Thanks for the work (again). It's an interesting small idea but I'd much 
prefer revisiting the original getter/setter RFC [1] which had a 
majority but just fell short of the 66% mark.

This RFC only implements parts of the getter/setter functionality 
(readonly properties) but it does not address the fact that sometimes 
you want to add logic to a setter or a getter and if you don't have 
language level getters/setters you end up having to change the interface 
of the object from a public property to a getFoo/setFoo pair.

This leads to everyone having getFoo/setFoo by default just to avoid 
interface changes or mixed interfaces of public properties and setFoo.

[1] https://wiki.php.net/rfc/propertygetsetsyntax-v1.2

Cheers

-- 
Jordi Boggiano
@seldaek - http://nelm.io/jordi