? ? Pending

User tests: Successful: Unsuccessful:

avatar SharkyKZ
SharkyKZ
4 Oct 2019

Pull Request for Issue #25770.

Summary of Changes

Replaces JOIN and GROUP clauses in query with a subquery.

Testing Instructions

Set up a multilingual site.
Enable associations.
Create some articles and add associations.
View article list in backend.

Expected result

Works like before.

Documentation Changes Required

No.

10687ec 4 Oct 2019 avatar SharkyKZ CS
avatar SharkyKZ SharkyKZ - open - 4 Oct 2019
avatar SharkyKZ SharkyKZ - change - 4 Oct 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 4 Oct 2019
Category Administration com_content
avatar richard67 richard67 - test_item - 4 Oct 2019 - Tested successfully
avatar richard67
richard67 - comment - 4 Oct 2019

I have tested this item successfully on 10687ec

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.

There is a PHP notice "Undefined index: link in /layouts/joomla/content/associations.php on line 18" at least when using PostgreSQL before and after having applied this PR, so not related to this PR. This just as info for other testers.


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

avatar alikon
alikon - comment - 4 Oct 2019

What language have you used in the installation step ?
i've tryied persian , french always got this trying to add multilanguage content sample data
Screenshot from 2019-10-04 19-09-51

avatar richard67
richard67 - comment - 4 Oct 2019

Sometimes content languages are not added after language install. I've helped myself then with uninstalling languages and installing again.

avatar alikon
alikon - comment - 4 Oct 2019

seems that German - DE allows me to run sample multilanguage data (just for note on postgresql now)

avatar richard67
richard67 - comment - 4 Oct 2019

I'm using German (DE), French and for RTC sometimes Persian.

avatar SharkyKZ SharkyKZ - change - 5 Oct 2019
Labels Added: ?
avatar SharkyKZ
SharkyKZ - comment - 5 Oct 2019

@alikon you may have to create a Content Language manually.

avatar richard67 richard67 - test_item - 5 Oct 2019 - Tested successfully
avatar richard67
richard67 - comment - 5 Oct 2019

I have tested this item successfully on 89e6b32

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/26465.

avatar alikon
alikon - comment - 5 Oct 2019

@SharkyKZ ok maybe another issue to open if it will be still there when we will have languages pack ready for 4, anyway PR seems fine & all works as before , despite i don't have a good big dataset to do some real case performance check... i cannot trust my EXPLAIN with my short data , but it looks fine and i cannot guess how it can works worst than now...

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

I have tested this item successfully on 89e6b32


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

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

RTC


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

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 20:49:53
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!

Add a Comment

Login with GitHub to post a comment