? Pending

User tests: Successful: Unsuccessful:

avatar SharkyKZ
SharkyKZ
5 Oct 2019

Summary of Changes

Replaces JOIN and GROUP clauses in query with a subquery for improved performance.

Testing Instructions

Set up a multilingual site.
Enable language associations.
Create some menu items with and without associations
View menu item list in backend.

Expected result

Works like before.

Documentation Changes Required

No.

avatar SharkyKZ SharkyKZ - open - 5 Oct 2019
avatar SharkyKZ SharkyKZ - change - 5 Oct 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 5 Oct 2019
Category Administration com_menus
avatar richard67 richard67 - test_item - 5 Oct 2019 - Tested successfully
avatar richard67
richard67 - comment - 5 Oct 2019

I have tested this item successfully on 426111e

Tested with MySQL 5.7 and PostgreSQL 10.10.

About performance I can't say anything because my test data set is too small.

But I can confirm everything works as well as before, carefully watching PHP error log with PHP error reporting set to maximum and watching database log.


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

avatar alikon alikon - test_item - 6 Oct 2019 - Tested successfully
avatar alikon
alikon - comment - 6 Oct 2019

I have tested this item successfully on 426111e


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

avatar alikon alikon - change - 6 Oct 2019
Status Pending Ready to Commit
avatar alikon
alikon - comment - 6 Oct 2019

RTC


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

avatar wilsonge wilsonge - change - 6 Oct 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-10-06 10:19:49
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 6 Oct 2019
avatar wilsonge wilsonge - merge - 6 Oct 2019
avatar wilsonge
wilsonge - comment - 6 Oct 2019

Thanks!

avatar Stuartemk
Stuartemk - comment - 6 Oct 2019

Tested and implemented in joomla 3.9.12, php 7.2, Mysql 5.7
Tested on Articles list, Newsfeeds list, Contact list, Menu list.
NOT tested on articles filtered by Tags, due to my little knowledge to implement it in Joomla 3.9.12

Everything works as usual, I have not encountered problems, the performance has improved a bit, unfortunately I do not have the necessary knowledge to measure the percentage of improvement.

However, the Menu Component continues to perform poorly since this improvement would only improve performance in the multilanguage area, but individually the Menu Component continues to create a TTFB in sites with several menus.

I make available a site with real load for case study in case anyone is interested in finding the definitive solution to the poor performance of the Menu Component in Joomla, which from my personal point of view, is the only thing that limits joomla to be The best CMS.

Thank you all for your great work!

avatar infograf768
infograf768 - comment - 9 Oct 2019

@Stuartemk

Tested and implemented in joomla 3.9.12

This PR was for 4.0

Add a Comment

Login with GitHub to post a comment