Feeling Lucky
[#45164] - Fix error handling rename item
- Pending
- Medium
- Build: 5.2-dev
- # 45164
- Diff
- 3idey:fix-error-handling-renameItem
- Failure continuous-integration/appveyor/pr AppVeyor build failed Details
- Success Download Prebuilt packages are available for download. Details
- Success continuous-integration/drone/pr Build is passing Details
User tests: Successful: 0 Unsuccessful: 0
Summary
This PR improves error handling in renameItem() in the Joomla Media Manager. It replaces raw error throws with Joomla notifications, ensuring better user feedback and using Joomla's Notification System Instead of Throwing Errors
Changes Made
- Updated renameItem() to use notifications.error() instead of throw new Error(error);
- Ensured that errors are properly logged in Joomla’s notification system.
- Improve File Exists Handling (Duplicate Files).
Testing Instructions
- Open Joomla Media Manager.
- Try renaming a file to an invalid name (e.g., same name as an existing file).
- You should see an error notification instead of the app crashing.
Checklist
- Code follows Joomla's coding standards.
- Fix does not introduce new bugs.
- Properly tested within Joomla.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | JavaScript Administration com_media NPM Change |
The description was changed
The description was changed
The description was changed
The description was changed
@richard67 @laoneo can you tell me is there anything that needs to be fixed.