? ? Pending

User tests: Successful: Unsuccessful:

avatar mbabker
mbabker
17 May 2018

Summary of Changes

PHP 7.2 got strict about running count() on non-countable things. PHP 7.3 adds a helper function to actually check for countability. This imports the polyfill to be able to use that function.

Testing Instructions

Any place in the code where you need to count something and you're not sure the value is countable, run is_countable($thing);, if true you can safely run count($thing).

avatar mbabker mbabker - open - 17 May 2018
avatar mbabker mbabker - change - 17 May 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 17 May 2018
Category External Library Composer Change Libraries Repository
avatar mbabker
mbabker - comment - 17 May 2018

We might need to put a leading slash on all the lines in CODEOWNERS. You should only get pinged on the root README, this pinged because I (wrongly) committed the upstream README.

avatar mbabker mbabker - change - 17 May 2018
Labels Added: ? ?
avatar zero-24
zero-24 - comment - 17 May 2018

Thanks @mbabker

avatar zero-24 zero-24 - change - 17 May 2018
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-05-17 19:46:57
Closed_By zero-24
avatar zero-24 zero-24 - close - 17 May 2018
avatar zero-24 zero-24 - merge - 17 May 2018

Add a Comment

Login with GitHub to post a comment