[#3575] - [#33413] Fix media manager upload and subfolder creation
- Closed
- 8 May 2014
- Medium
- Build: staging
- # 3575
- Diff
- Bakual:FixMediaManagerUplao
- Foreign ID 33413
- Success The Travis CI build passed Details
User tests: Successful: Unsuccessful:
This is based on a comment from @illovo on #3551 and the forum post linked by him (it helps that I speak german g).
To me, it looks like indeed the correct fix to the issue.
- The JS code was trying to use the index instead of the actual value of the array and thus failed to update the hidden form fields.
- The readonly path field in the subfolder creation part was missing the
update-folderclass and thus the JS code didn't update it with the new path.
Testing
- Try creating a subfolder in the media manager
- Try uploading a file to a subfolder in the media manager
Tracker
http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=33413
test: Works fine for me
But when you create a folder, the name stays as value in the folder-name field. So when you create another folder, the folder-name field isn't empty
| Title |
|
||||||
| Labels |
Added:
?
?
|
||||||
Works fine, I can create subfolders in subfolders and upload file everywhere
RTC
| Status | New | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2014-05-08 15:53:43 |
Not sure what is the best place/procedure to report this, but we can't upload media to subfolders or create nested subfolders on a new (not migrated) Joomla 3.3.6 install, using Media Manager.
MySQL 5.1.56
PHP 5.4.37
Apache (not sure; DreamHost likes to hide this info)
Google Chrome 40.0.2214.111 m
Cleared browser and Joomla caches. This made no difference.
administrator/components/com_media/views/media/tmpl/default.php has the changes from #33413, but curiously, media/media/js/mediamanager.js doesn't. I made the changes manually, but this made no difference.
Also tried from Firefox 35.0.1 with the same result, with the stock mediamanager.js from Joomla 3.3.6 and after changing mediamanager.js as shown here and clearing caches after changes.
What we've got here is bug regression.
test: uploading works with this fix. Nice one!