PR-5.4-dev Pending

User tests: Successful: Unsuccessful:

avatar jiteshkhatri11
jiteshkhatri11
24 Aug 2026

Summary

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.

Testing

  • git diff --check passes.
  • Verified the final diff contains only the two intended changes.
  • No existing unit test for MenuType was found in the current test suite.
avatar jiteshkhatri11 jiteshkhatri11 - open - 24 Aug 2026
avatar jiteshkhatri11 jiteshkhatri11 - change - 24 Aug 2026
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 24 Aug 2026
Category Libraries
avatar brianteeman
brianteeman - comment - 24 Aug 2026

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.

avatar richard67
richard67 - comment - 24 Aug 2026

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.

avatar richard67
richard67 - comment - 24 Aug 2026

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.

avatar brianteeman
brianteeman - comment - 24 Aug 2026

@richard67 i was trying to be polite in my reply which is why i didnt comment on diff being a good way to test

avatar richard67
richard67 - comment - 24 Aug 2026

@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.

avatar brianteeman
brianteeman - comment - 25 Aug 2026

This is what your pull request should have looked like, with clear instructions how to replicate the reported problem and before and after results #48303

avatar muhme muhme - close - 25 Aug 2026
avatar muhme
muhme - comment - 25 Aug 2026

@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.

avatar muhme muhme - change - 25 Aug 2026
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

Add a Comment

Login with GitHub to post a comment