[#45631] - [5.4] Media Delete Confirmation message
- Fixed in Code Base
- 24 Jun 2025
- Medium
- Build: 5.4-dev
- # 45631
- Diff
- brianteeman:media_delete
User tests: Successful: Unsuccessful:
Pull Request for Issue #29564 .
Summary of Changes
This PR introduces a dedicated delete confirmation message for com_media instead of the global delete message.
The reason is to further clarify/warn that deleting a folder will delete all the contents of that folder. Unlike the generic message which just talks about deleting the selected item
Ideally I wanted to have a separate confirmation message for items and folders but this is not possible without significant rewrite and as no one has shown interest in that since the issue was first raised #29564 this PR is a small improvement.
Testing Instructions
To test you will need to either use a prebuilt package from this PR or apply the PR and then run npm i
Actual result BEFORE applying this Pull Request
Expected result AFTER applying this Pull Request
Link to documentations
Please select:
-
Documentation link for docs.joomla.org:
-
No documentation changes for docs.joomla.org needed
-
Pull Request link for manual.joomla.org:
-
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration com_media NPM Change Language & Strings |
I have tested this item ✅ successfully on 15864b1
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45631.
| Status | Pending | ⇒ | Ready to Commit |
| Labels |
Added:
NPM Resource Changed
Language Change
PR-5.4-dev
|
||
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45631.
| Labels |
Added:
RTC
|
||
@brianteeman Is there a reason why we have different texts for frontend and backend?
Yes. My error. The admin language is the correct one. Will fix the other when I can get to a pc
Yes. My error. The admin language is the correct one. Will fix the other when I can get to a pc
@brianteeman Thanks.
@richard67 fixed it
I think it does not need new tests as I have just tested it successfully with the last 2 changes in administrator and site.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2025-06-24 21:31:46 |
| Closed_By | ⇒ | richard67 |
Thanks all (author and testers).
Thanks @richard67 for spotting my errors
I have tested this item ✅ successfully on 15864b1
NPM Package
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45631.