Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#14912] - Huge number of queries performed on lists due to fields

No Code Attached Yet J3 Issue
avatar klas
klas
26 Mar 2017

Steps to reproduce the issue

Add 10 fields and save few articles with them. Create a list with 100 items on frontend and display it.

Expected result

Single query for all items&fields.

Actual result

1000 queries.

System information (as much as possible)

Additional comments

avatar klas klas - open - 26 Mar 2017
avatar joomla-cms-bot joomla-cms-bot - change - 26 Mar 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 26 Mar 2017
avatar klas klas - change - 26 Mar 2017
Title
…
[3.7] Huge number of queries performed on lists due to fields
Huge number of queries performed on lists due to fields
avatar klas klas - edited - 26 Mar 2017
avatar klas klas - change - 26 Mar 2017
Title
…
Huge number of queries performed on lists due to fields
[3.7] Huge number of queries performed on lists due to fields
avatar brianteeman brianteeman - change - 26 Mar 2017
Title
…
Huge number of queries performed on lists due to fields
[3.7] Huge number of queries performed on lists due to fields
Status New ⇒ Closed
Closed_Date 0000-00-00 00:00:00 ⇒ 2017-03-26 21:59:56
Closed_By ⇒ brianteeman
avatar brianteeman brianteeman - close - 26 Mar 2017
avatar brianteeman
brianteeman - comment - 26 Mar 2017

See #14558

avatar klas
klas - comment - 26 Mar 2017

After #14558 there will be still 100 queries, which is not acceptable,

avatar mbabker mbabker - change - 27 Mar 2017
Status Closed ⇒ New
Closed_Date 2017-03-26 21:59:56 ⇒
Closed_By brianteeman ⇒
avatar mbabker mbabker - reopen - 27 Mar 2017
avatar mbabker
mbabker - comment - 27 Mar 2017

Reopening. The PR is an improvement but there's still room for better.

avatar laoneo
laoneo - comment - 27 Mar 2017

As @Bakual pointe out here, actually it is not really possible with the current plugin events. Perhaps we can think about to make some list events in J4.

avatar laoneo
laoneo - comment - 27 Mar 2017

I'm not sure if we want to change then tags to work like fields and use the same events there as well as it is the same problem with tags.

avatar csthomas
csthomas - comment - 27 Mar 2017

If you want to get all fields for articles in one query then IMO you have to use some magic:)
Ex:
Instead getFieldValue() use getField() to aggregate all fields and then you have to use magic method __get for $field->value.
This way, when you start using first value you load all fields.

Off course this is more complicated solution and not always acceptable.

avatar franz-wohlkoenig franz-wohlkoenig - change - 30 Mar 2017
Category ⇒ com_content
avatar franz-wohlkoenig franz-wohlkoenig - change - 2 Apr 2017
Category com_content ⇒ com_content SQL
avatar franz-wohlkoenig franz-wohlkoenig - change - 2 Apr 2017
Status New ⇒ Discussion
avatar brianteeman brianteeman - change - 25 Mar 2018
Labels Added: J3 Issue
avatar brianteeman brianteeman - labeled - 25 Mar 2018
avatar franz-wohlkoenig franz-wohlkoenig - change - 19 Apr 2019
Title
…
[3.7] Huge number of queries performed on lists due to fields
Huge number of queries performed on lists due to fields
avatar franz-wohlkoenig franz-wohlkoenig - edited - 19 Apr 2019
avatar brianteeman
brianteeman - comment - 23 Aug 2022

Thank you for raising this issue.

Joomla 3 is now in security only mode with no further bug fixes or new features.

As this issue doesn't relate to Joomla 4 it will now been closed.

If we are mistaken and this does apply to Joomla 4 please open a new issue (and reference this one if you wish) with updated details for testing in Joomla 4.
cc @zero-24

avatar zero-24 zero-24 - change - 23 Aug 2022
Status Discussion ⇒ Closed
Closed_Date 0000-00-00 00:00:00 ⇒ 2022-08-23 13:49:46
Closed_By ⇒ zero-24
Labels Added: No Code Attached Yet
Removed: ?
avatar zero-24 zero-24 - close - 23 Aug 2022

Add a Comment

Login with GitHub to post a comment