User tests: Successful: Unsuccessful:
Fix the error message returned when deleting a menu type fails because menu items or modules are checked out.
Both failure paths in MenuType::delete() now use JLIB_DATABASE_ERROR_MENUTYPE directly instead of JLIB_DATABASE_ERROR_DELETE_FAILED.
git diff --check passes.MenuType was found in the current test suite.| Status | New | ⇒ | Pending |
| Category | ⇒ | Libraries |
There was a reason you were given a template to complete when you submitted the PR. As it is right now it is completely useless.
Please update your post using the pull request template which you can find here https://github.com/joomla/joomla-cms/blob/5.4-dev/.github/PULL_REQUEST_TEMPLATE.md
Otherwise it will have to be closed.
Correct. The AI policy has to be confirmed, issue #48255 should be referred in the description, and real testing instructions for end users have to be provided.
P.S.: Our contribution guidelines require that PR authors have tested their PR themselves before submitting. Telling to check a git diff as testing instruction clearly show that the author obviously has not done that, and it is even questionable if he has ever used Joomla.
@richard67 i was trying to be polite in my reply which is why i didnt comment on diff being a good way to test
@richard67 i was trying to be polite in my reply which is why i didnt comment on diff being a good way to test
Well, maybe I was a bit too harsh. Sorry for that.
@jiteshkhatri11 Thank you for your contribution. The fix works, but this is an improvement rather than a bug fix and should therefore target the 6.3-dev branch.
Please also consider the comments above: use the PR template, link the related issue and provide testing instructions.
For these reasons, we will close this PR. You're welcome to open a new PR against 6.3-dev with these points addressed.
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2026-08-25 09:37:01 |
| Closed_By | ⇒ | muhme | |
| Labels |
Added:
PR-5.4-dev
|
||
There was a reason you were given a template to complete when you submitted the PR. As it is right now it is completely useless.
Please update your post using the pull request template which you can find here https://github.com/joomla/joomla-cms/blob/5.4-dev/.github/PULL_REQUEST_TEMPLATE.md
Otherwise it will have to be closed.