? Success

User tests: Successful: Unsuccessful:

avatar Denitz
Denitz
23 Jun 2017

Summary of Changes

No need to count message queue elements.

Testing Instructions

Use PHP 7.2 with development error reporting.

Expected result

Nice Joomla

Actual result

PHP warning:

Warning: count(): Parameter must be an array or an object that implements Countable in \libraries\cms\application\cms.php on line 470

8a39948 23 Jun 2017 avatar Denitz fix
avatar Denitz Denitz - open - 23 Jun 2017
avatar Denitz Denitz - change - 23 Jun 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 23 Jun 2017
Category Libraries
avatar Denitz Denitz - change - 23 Jun 2017
The description was changed
avatar Denitz Denitz - edited - 23 Jun 2017
avatar mbabker
mbabker - comment - 25 Jun 2017

Based on the result from https://travis-ci.org/joomla/joomla-cms/jobs/236743260 which enabled a Travis build against PHP 7.2, this is a good fix.

avatar zero-24
zero-24 - comment - 1 Jul 2017

@Denitz just to be 100% sure can you update to the last staging branch? So travis run against PHP 7.2 and we can check the results?

avatar mbabker mbabker - change - 6 Aug 2017
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-08-06 16:25:01
Closed_By mbabker
avatar mbabker mbabker - close - 6 Aug 2017
avatar mbabker mbabker - merge - 6 Aug 2017

Add a Comment

Login with GitHub to post a comment