Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#21319] - Fixed the double import of InputFilter

? Pending

User tests: Successful: Unsuccessful:

avatar roland-d
roland-d
31 Jul 2018

Pull Request for Issue #21174

Summary of Changes

This removes the double import statement of use Joomla\CMS\Filter\InputFilter; causing a fatal error.

In addition, the import statements are now alphabetically ordered, should help prevent these kind of issues :)

Testing Instructions

  1. Try to login to Joomla in backend, you get a fatal error
  2. Apply patch
  3. You get login screen

Expected result

Login screen in backend

Actual result

Fatal error: Cannot use Joomla\CMS\Filter\InputFilter as InputFilter because the name is already in use in /usr/local/var/www/joomlacms/libraries/src/Filesystem/File.php on line 22

Documentation Changes Required

None

@laoneo You merged #21174 so calling you here.

avatar roland-d roland-d - open - 31 Jul 2018
avatar roland-d roland-d - change - 31 Jul 2018
Status New ⇒ Pending
avatar joomla-cms-bot joomla-cms-bot - change - 31 Jul 2018
Category ⇒ Libraries
avatar roland-d roland-d - change - 31 Jul 2018
The description was changed
avatar roland-d roland-d - edited - 31 Jul 2018
avatar SharkyKZ SharkyKZ - test_item - 31 Jul 2018 - Tested successfully
avatar SharkyKZ
SharkyKZ - comment - 31 Jul 2018

I have tested this item ✅ successfully on 401b89d

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

avatar laoneo laoneo - change - 31 Jul 2018
Status Pending ⇒ Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 ⇒ 2018-07-31 07:22:42
Closed_By ⇒ laoneo
Labels Added: ?
avatar laoneo laoneo - close - 31 Jul 2018
avatar laoneo laoneo - merge - 31 Jul 2018
avatar laoneo
laoneo - comment - 31 Jul 2018

Thanks for the fast reaction!

Add a Comment

Login with GitHub to post a comment