Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#10660] - PHP 7.0.7 backend menu notice

?
avatar Denitz
Denitz
29 May 2016

Steps to reproduce the issue

Install PHP 7.0.7
Turn Error Reporting to Development
Load backend

Expected result

No errors

Actual result

Notice: Only variables should be passed by reference in \administrator\modules\mod_menu\tmpl\default_enabled.php on line 23

System information (as much as possible)

Additional comments

Fix: remove $ reference in JAdminCssMenu::addChild() arguments.

avatar Denitz Denitz - open - 29 May 2016
avatar Denitz Denitz - change - 29 May 2016
Title
…
PHP 7.0.7
PHP 7.0.7 backend menu notice
avatar Denitz Denitz - change - 29 May 2016
Title
…
PHP 7.0.7
PHP 7.0.7 backend menu notice
avatar brianteeman
brianteeman - comment - 29 May 2016

Pretty certain this has already been reported and a pr submitted. On my
phone right now so it is hard to check

avatar SharkyKZ
SharkyKZ - comment - 29 May 2016

#10452. This can be closed as duplicate.

avatar brianteeman brianteeman - change - 29 May 2016
Status New ⇒ Duplicate Report
Closed_Date 0000-00-00 00:00:00 ⇒ 2016-05-29 12:18:19
Closed_By ⇒ brianteeman
avatar brianteeman brianteeman - close - 29 May 2016
avatar brianteeman brianteeman - close - 29 May 2016
avatar sovainfo
sovainfo - comment - 29 May 2016

Also, the fix proposed is wrong. The proper fix is to remove the &, not the $ !.
Fortunately, that is what is implemented with #10452.

avatar Denitz
Denitz - comment - 31 May 2016

Sure &, sorry, I was in hurry :)

Add a Comment

Login with GitHub to post a comment