Success

User tests: Successful: Unsuccessful:

avatar Bakual
Bakual
14 Jul 2013

Fixing articles and article model query. It was failing if there are multiple contacts of different languages assigned to the same user, resulting in duplicated content shown.

There is also a huge performance loss if there are many articles due to the GROUP BY statement. According to the discussion on Google Groups:
https://groups.google.com/forum/?fromgroups=#!topic/joomla-dev-general/sVFw9reqGGw

avatar Bakual Bakual - open - 14 Jul 2013
avatar infograf768
infograf768 - comment - 15 Jul 2013

concerning the GROUP BY part, see comments on tracker.

avatar Bakual
Bakual - comment - 15 Jul 2013

This PR will only remove the GROUP BY on the main query. There is still one GROUP BY in a subquery which searchs the parent categories for unpublished or trashed categories. I think that one is still needed.

avatar AmyStephen
AmyStephen - comment - 15 Jul 2013

Correct @Bakual - Let's hold on this for now. We'll need to adapt this a bit for the default contact approach and then see what folks think.

avatar AmyStephen
AmyStephen - comment - 15 Jul 2013

I did not realize a release was pending. Definitely do not want to hold this performance and duplication fix back - it must go in for certain! We'll work on a fuller solution in next release. That'll give us a little time to play with your integration ideas, ;-) @Bakual

avatar infograf768 infograf768 - close - 17 Jul 2013
avatar Bakual Bakual - head_ref_deleted - 17 Jul 2013
avatar Bakual Bakual - reference | - 17 Jul 13
avatar garyamort garyamort - reference | - 2 Dec 13

Add a Comment

Login with GitHub to post a comment