joomla / joomla-cms

Home of the Joomla! Content Management System
https://www.joomla.org
GNU General Public License v2.0
4.74k stars 3.65k forks source link

[3.9] JFilterInput' not found when uploading an image through Media manager #21288

Closed jamgit closed 6 years ago

jamgit commented 6 years ago

Steps to reproduce the issue

  1. Going to Content > Media.
  2. Uploading new image, opens new area to browse for new image.
  3. Find your file on computer and click on upload.

Expected result

The image uploads and is added to the library.

Actual result

Error shows up after steps above saying "Class 'Joomla\CMS\Filesystem\JFilterInput' not found"

System information (as much as possible)

Mac: OS High Sierra 2.5 GHz Intel Core i7

Joomla Build is the 3.9 privacy package full dev version.

Additional comments

bayareajenn commented 6 years ago

I can duplicate this issue.


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

ernienet commented 6 years ago

Duplicated in a live server environment with the 3.9 Privacy Package


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

wjlorente commented 6 years ago

Found this as part of testing J 3.9 Privacy Tools Suite, with a complete, fresh install from https://developer.joomla.org/privacy-pack/. Tried uploading an image via Content -> Media -> Upload. After I selected an image and clicked on the "Upload" button, Joomla returned a message "Fatal error: Class 'Joomla\CMS\Filesystem\JFilterInput' not found in /home3/nimda/public_html/dev/20180728_pbf/libraries/src/Filesystem/File.php on line 543" on an otherwise blank page.


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

ggppdk commented 6 years ago

Related PR but for J4 https://github.com/joomla/joomla-cms/pull/21174

Sandra97 commented 6 years ago

I faced the same issue, applied the changes of the PR you're mentioning, now it works as expected.

bayareajenn commented 6 years ago

I applied #21174 in 3.9 and it tested successfully. If there's a separate PR for 3.9, let me know so I can update it. OR, if you want all of us to test #21174 and mark it successful, let us know so everyone else that posted in here can test also.

wojsmol commented 6 years ago

Please test https://github.com/joomla/joomla-cms/pull/21314

bayareajenn commented 6 years ago

Done @wojsmol.

@jamgit , @ernienet and @wjlorente will you please go apply the patch with the patchtester 2.0.1 (stable) from https://github.com/joomla-extensions/patchtester/releases (install it via the extension manager) and apply the patch for #21314. Then test to see if you can add media.

Then go to https://issues.joomla.org/tracker/joomla-cms/21314 and login and report whether it tested successfully or not (click the Test this button to do this).

joomla-cms-bot commented 6 years ago

Set to "closed" on behalf of @franz-wohlkoenig by The JTracker Application at issues.joomla.org/joomla-cms/21288

ghost commented 6 years ago

closed as having Pull Request #21314.