? ? Pending

User tests: Successful: Unsuccessful:

avatar Quy
Quy
1 Apr 2018

Pull Request for Issue #19852.

Summary of Changes

Fix Warning: count(): Parameter must be an array or an object that implements Countable in \libraries\src\Installer\Adapter\FileAdapter.php on line 451

Testing Instructions

  • Install PHP 7.2
  • Install Akeeba Backup
  • Admin -> Extensions -> Manage -> search for akeeba
  • Select all
  • Click uninstall

Documentation Changes Required

no

avatar Quy Quy - open - 1 Apr 2018
avatar Quy Quy - change - 1 Apr 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 1 Apr 2018
Category Libraries
avatar laoneo
laoneo - comment - 1 Apr 2018

Thanks. This is already fixed in upstream joomla-projects/media-manager-improvement#522. Will sync it next week in here.

avatar brianteeman
brianteeman - comment - 1 Apr 2018

@laoneo unless I am very mistaken your fix is for a completely different issue

avatar laoneo
laoneo - comment - 1 Apr 2018

Yes you are right.

avatar brianteeman
brianteeman - comment - 7 Apr 2018

I have tested this item successfully on 1290e5d


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

avatar brianteeman brianteeman - test_item - 7 Apr 2018 - Tested successfully
avatar Anu1601CS
Anu1601CS - comment - 15 Apr 2018

I have tested this item successfully on 1290e5d


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

avatar Anu1601CS Anu1601CS - test_item - 15 Apr 2018 - Tested successfully
avatar franz-wohlkoenig franz-wohlkoenig - change - 15 Apr 2018
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 15 Apr 2018

Ready to Commit after two successful tests.

avatar mbabker mbabker - change - 22 Apr 2018
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-04-22 15:41:45
Closed_By mbabker
Labels Added: ? ?
avatar mbabker mbabker - close - 22 Apr 2018
avatar mbabker mbabker - merge - 22 Apr 2018

Add a Comment

Login with GitHub to post a comment