J3 Issue No Code Attached Yet
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: No Code Attached Yet
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

Add a Comment

Login with GitHub to post a comment