Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:25835
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain cschneid.com from 195.226.6.42 cause and error)
Message-ID: <451911E0.3010609@cschneid.com>
Date: Tue, 26 Sep 2006 13:41:20 +0200
User-Agent: Thunderbird 1.5.0.7 (X11/20060911)
MIME-Version: 1.0
To: Marcus Boerger <helly@php.net>
CC: PHP Developers Mailing List <internals@lists.php.net>
References: <4511925F.5060602@cschneid.com> <783278347.20060921074613@marcus-boerger.de>
In-Reply-To: <783278347.20060921074613@marcus-boerger.de>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] Parameter checking in 5.2
From: cschneid@cschneid.com (Christian Schneider)

Marcus Boerger wrote:
>   the short form is, use interfaces. And the long form is read the upgrade
>   file and find out to use interfaces :-)

I don't understand how interfaces are connected to my proposal, please
explain.

Personally I don't need the strict OO checks for matching parameter
lists at all, but as long as they are E_STRICT (or the like) I have no
problem with them being done. Nevertheless I would like to relax them
just a tiny  little bit for signature changes not having an impact of
object compatibility.

That's why I posted two use cases where the checks IMHO go too far and
added a patch to change that.

Copy of my message:

>> In the discussion about parameter checking in 5.2 I proposed to relax
>> the checks a tiny little bit: Don't test static functions (e.g. useful
>> for factory methods) and allow adding default values to functions (the
>> object of the inherited class still accepts the same parameters as the
>> base class). A patch is attached.
> 
>> Example:
>> class Base
>> {
>> static function factory($x) { }
>> function foo($x) { }
>> }
> 
>> class Spezialized extends Base
>> {
>> static function factory() { }   # Static method, e.g. factory method
>> function foo($x = null) { }     # Default values for specialized class
>> }

- Chris