Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#47601] - [6.1] Fix: Debug plugin crash with Query Explain on AJAX requests

bug PR-6.1-dev Pending

User tests: Successful: Unsuccessful:

avatar hiteshm0
hiteshm0
15 Apr 2026

Pull Request resolves #47526

  • I read the Generative AI policy and my contribution is either not created with the help of AI or is compatible with the policy and GNU/GPL 2 or later.

Summary of Changes

The debug plugin could assume that $this->queryMonitor was always available and crash with Call to a member function getLogs() on null on some backend AJAX requests when $this->queryMonitor is null. This change makes query explain handling run only when $this->queryMonitor is not null preventing the error.

Testing Instructions

  1. Enable in global config: System > Debug System = Yes
  2. In plugin debug > set Query Explains = Show
  3. Return to Joomla Home Dashboard
  4. Change the menutype of a menu item in its edit form )

Actual result BEFORE applying this Pull Request

Some AJAX requests fail with error 500

example:

  1. Notification module in the dashboard
image
  1. Fetching Parent Items in menu item edit form ( when i menutype is changed )
image

Expected result AFTER applying this Pull Request

No errors

Link to documentations

Please select:

  • Documentation link for guide.joomla.org:

  • No documentation changes for guide.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar hiteshm0 hiteshm0 - open - 15 Apr 2026
avatar hiteshm0 hiteshm0 - change - 15 Apr 2026
Status New ⇒ Pending
avatar joomla-cms-bot joomla-cms-bot - change - 15 Apr 2026
Category ⇒ Front End Plugins
avatar hiteshm0 hiteshm0 - change - 15 Apr 2026
The description was changed
avatar hiteshm0 hiteshm0 - edited - 15 Apr 2026
avatar hiteshm0 hiteshm0 - change - 15 Apr 2026
The description was changed
avatar hiteshm0 hiteshm0 - edited - 15 Apr 2026
avatar Fedik
Fedik - comment - 15 Apr 2026

Please revert all unnecessary changes, and just check the $this->queryMonitor exists

if ($this->params->get('query_explains') 
  && $this->queryMonitor 
  && \in_array($db->getServerType(), ['mysql', 'postgresql'], true)
) {
....
}
avatar hiteshm0 hiteshm0 - change - 15 Apr 2026
Labels Added: bug PR-6.1-dev
avatar hiteshm0 hiteshm0 - change - 15 Apr 2026
The description was changed
avatar hiteshm0 hiteshm0 - edited - 15 Apr 2026
avatar hiteshm0
hiteshm0 - comment - 15 Apr 2026

@Fedik done!

avatar Fedik
Fedik - comment - 15 Apr 2026

Thanks. It can be tested now.

avatar alikon alikon - test_item - 17 Apr 2026 - Tested successfully
avatar alikon
alikon - comment - 17 Apr 2026

I have tested this item ✅ successfully on 6abc860

This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47601.

avatar tecpromotion tecpromotion - test_item - 19 Apr 2026 - Tested successfully
avatar tecpromotion
tecpromotion - comment - 19 Apr 2026

I have tested this item ✅ successfully on 6abc860

This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47601.

avatar tecpromotion tecpromotion - change - 19 Apr 2026
Status Pending ⇒ Ready to Commit
avatar tecpromotion
tecpromotion - comment - 19 Apr 2026

RTC

This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47601.

Add a Comment

Login with GitHub to post a comment