[#28910] - [4.0] Fix media manager file rename
- Fixed in Code Base
- 2 May 2020
- Medium
- Build: 4.0-dev
- # 28910
- Diff
- wilsonge:feature/fix-media-file-rename
- Pending Hound Hound is busy sniffing around... Details
User tests: Successful: Unsuccessful:
Pull Request for Issue #28896 .
Summary of Changes
Fixes the ability to rename a file in media manager. This was broken in #28896 and was shows by the drone tests failing - unfortunately I still merged it anyway :)
Testing Instructions
Before patch check renaming a file with a new name gives a success message but it isn't actually renamed
Check that renaming a file to a different name works (also test that renaming a file and hitting save without saving it also works (i.e. gives a successfully renamed message).
Also check drone now passes
Documentation Changes Required
None
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration com_media |
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/28910.
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/28910.
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/28910.
| Status | Pending | ⇒ | Ready to Commit |
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/28910.
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/28910.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2020-05-02 19:18:34 |
| Closed_By | ⇒ | richard67 | |
| Labels |
Added:
?
|
||
Thanks to all.
I think you need the newAdapter there, otherwise this is fine,
PS ignore me this is fine