Page MenuHomePhabricator

Only variables should be assigned by reference in backend/ConfirmAccount.class.php
Closed, ResolvedPublic

Description

I got this PHP notice with ConfirmAccount 0d79eb3 + MW 1.29alpha + PHP 7.0:

Only variables should be assigned by reference in backend/ConfirmAccount.class.php on line 217

The faulty syntax is:

$magic =& MimeMagic::singleton();

When searching in MediaWiki codebase elsewhere for MimeMagic, the usual usage is $magic = MimeMagic::singleton();.

Event Timeline

Change 324515 had a related patch set uploaded (by Seb35):
Fix PHP notice 'Only variables should be assigned by reference'

https://gerrit.wikimedia.org/r/324515

Seb35 triaged this task as Lowest priority.Nov 30 2016, 6:16 PM

Change 324515 merged by jenkins-bot:
Fix PHP notice 'Only variables should be assigned by reference'

https://gerrit.wikimedia.org/r/324515

Seb35 claimed this task.