?

User tests: Successful: Unsuccessful:

avatar ribeirobreno
ribeirobreno
17 Feb 2014

This patch improves performance for high volume websites by loading the menu data to the cache if it is enabled.

avatar ribeirobreno ribeirobreno - open - 17 Feb 2014
avatar Bakual
Bakual - comment - 17 Feb 2014

There are a lot of codestyle issues with that PR and also usage of JError which is deprecated.
Please update the PR to follow our coding standards and remove the (new) usage of JError.

avatar Bakual
Bakual - comment - 17 Feb 2014

Ah sorry, didn't catch that this one is for J! 2.5.
So JError will be fine here of course.
It would still be nice to follow our Codestyle standards, but it is not a must.

avatar brianteeman
brianteeman - comment - 17 Feb 2014

Excuse my ignorance but isnt this already handled by the menu module

avatar ribeirobreno
ribeirobreno - comment - 17 Feb 2014

@Bakual Thanks for pointing out the problems.

@brianteeman Monitoring a MySQL server itself has shown this query and some of the session queries are being run despite the settings in configuration.php. I don't know about 3.x but for 2.5 you can see it just by enabling debug in a Joomla website and looking at "Database Queries".

Before this patch, the query in this file is in every request.

avatar goyat
goyat - comment - 24 Mar 2014

I got this error.
The file marked for modification does not exist: includes/menu.php

avatar ribeirobreno
ribeirobreno - comment - 30 Jun 2014

@goyat have you tried to merge this with a branch higher than 2.5? In 3.x this is handled somewhere else. :)

avatar nicksavov nicksavov - change - 21 Aug 2014
Labels Removed: ?
avatar brianteeman brianteeman - change - 21 Aug 2014
Status New Pending
avatar brianteeman brianteeman - change - 2 Sep 2014
Category Cache
avatar brianteeman brianteeman - change - 1 Jan 2015
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2015-01-01 13:03:22
Closed_By brianteeman
avatar brianteeman brianteeman - close - 1 Jan 2015
avatar brianteeman
brianteeman - comment - 1 Jan 2015

Thanks for working on this. Unfortunately this did not make it into the final release of Joomla 2.5, or it was handled elsewhere, so this is being closed. If you feel this is still a valid issue in Joomla 3 please create a new issue.


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

Add a Comment

Login with GitHub to post a comment