? Pending

User tests: Successful: Unsuccessful:

avatar joomdonation
joomdonation
28 Dec 2017

Pull Request for Issue # .

Summary of Changes

When method JHtmlMenu::treerecurse is used on PHP 7.2, there are some warnings like this displayed:

Warning: count(): Parameter must be an array or an object that implements Countable in D:\www\joomla\libraries\cms\html\menu.php on line 382

This PR just fixes it.

Testing Instructions

  1. You need to run your Joomla website on PHP 7.2
  2. Open the file templates/protostar/index.php, add this line of code:
echo JHtml::_('select.genericlist', JHtmlMenu::linkOptions(), 'link_option');
  1. Access to frontend of the site, you will see many warning messages like this:

Warning: count(): Parameter must be an array or an object that implements Countable in D:\www\joomla\libraries\cms\html\menu.php on line 382

  1. Apply patch, the warnings are gone. You should see a dropdown contains menu items
avatar joomdonation joomdonation - open - 28 Dec 2017
avatar joomdonation joomdonation - change - 28 Dec 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 28 Dec 2017
Category Libraries
avatar 810 810 - test_item - 28 Dec 2017 - Tested successfully
avatar 810
810 - comment - 28 Dec 2017

I have tested this item successfully on abd88cd


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19193.

avatar Quy Quy - test_item - 28 Dec 2017 - Tested successfully
avatar Quy
Quy - comment - 28 Dec 2017

I have tested this item successfully on abd88cd


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19193.

avatar Quy Quy - change - 28 Dec 2017
Status Pending Ready to Commit
avatar Quy
Quy - comment - 28 Dec 2017

RTC

avatar mbabker mbabker - change - 3 Jan 2018
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-01-03 00:09:58
Closed_By mbabker
Labels Added: ?
avatar mbabker mbabker - close - 3 Jan 2018
avatar mbabker mbabker - merge - 3 Jan 2018

Add a Comment

Login with GitHub to post a comment