Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:46403
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.219.226 as permitted sender)
DomainKey-Status: bad
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=message-id:date:from:user-agent:mime-version:to:cc:subject
         :references:in-reply-to:content-type:content-transfer-encoding;
        b=rPS9wLCDGL3fCKGGcYNq3Uh2aFO8NuegdKWQjxwRYmmJ32SeWy2bZqrQj1hQF6Jo1p
         O7cB9AbB6duXN8urEXlXoQz/H5mVQYK0dA82D9e+sb01jYu1obsxcoMO31EPMYSl7URv
         mSMNMELPAIdzqJKyD4Tw0kvJG+2pHwyeBhTnU=
Message-ID: <4B23CE1C.4000407@gmail.com>
Date: Sat, 12 Dec 2009 18:08:44 +0100
User-Agent: Thunderbird 2.0.0.21 (Windows/20090302)
MIME-Version: 1.0
To: Tim Starling <tstarling@wikimedia.org>
CC: PHP Developers Mailing List <internals@lists.php.net>, 
 ilia@prohost.org
References: <4B2027FF.3050704@wikimedia.org>
In-Reply-To: <4B2027FF.3050704@wikimedia.org>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] PHP 5.3.1 reference parameter regression
From: keisial@gmail.com (Keisial)

Tim Starling wrote:
> Since 5.3.1 it's impossible to have a reference parameter passed to a
> __call() function. 

> This change causes bugs in MediaWiki, where an attempt by a proxy object
> to pass a call through to a function that accepts reference parameters
> will always fail, due to the exacerbating change made to reference
> argument semantics in PHP 5.3.0.
>
> If we want MediaWiki to support PHP 5.3.1, we'll have to do some major
> changes, like abandoning __call entirely, or rewriting all the relevant
> interfaces to not use reference parameters. I'm not too keen about doing
> this without getting some indication from the PHP dev team as to whether
> this change is permanent or whether it will be fixed in an imminent PHP
> 5.3.2 release. If it's going to be fixed soon, we can just blacklist PHP
> 5.3.1 in our installer.
>
> I've tried emailing stas but I didn't get a reply.
>
> -- Tim Starling
It's also broken in PHP 5.2.11 and 5.2.12 RC4. Works in 5.2.10 and
5.2.11RC1.
Whatever change broke it, was backported and got into 5.2.11RC2.

PS: Did mediawiki opt-out for release candidate announcements? It didn't
receive emails about RCs
for 5.2.12 branch.