? Success

User tests: Successful: Unsuccessful:

avatar Bakual
Bakual
8 Oct 2014

Same fix as #4330 but for queries which containg GROUP BY and/or HAVING.

Issue

Pagination doesn't show at all.

Testing

Issue has been raised in #4472 with the component JDownloads (http://www.jdownloads.com/index.php?option=com_jdownloads&Itemid=133&view=viewdownload&catid=43&cid=337). The issue doesn't show itself in core to my knowledge. So to test, best is to use JDownloads or any other 3rd party extension which uses a query with GROUP BY or HAVING clauses.

When testing especially make sure nothing else breaks.

avatar Bakual Bakual - open - 8 Oct 2014
avatar jissues-bot jissues-bot - change - 8 Oct 2014
Labels Added: ?
avatar ColinM2
ColinM2 - comment - 8 Oct 2014

Excellent. Have tested all various combinations in front end where issue was manifesting itself - all now AOK. Also for completeness tested in backend which had and still has no problem
Will advise others in jDownloads test team

Looks a good fix to me. Thank you.

avatar ColinM2
ColinM2 - comment - 8 Oct 2014

Tested AOK on 3.3.6 as expected

avatar Bakual Bakual - alter_testresult - 8 Oct 2014 - ColinM2: Tested successfully
avatar RCheesley
RCheesley - comment - 17 Oct 2014

Anyone got a sample data set to test with? I've installed JDownloads and made 6 categories, and six downloads for example. In a front end view with list limit set to 5, I see the pagination and it works (before applying the patch).

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

avatar RCheesley RCheesley - test_item - 17 Oct 2014 - Tested unsuccessfully
avatar Bakual
Bakual - comment - 17 Oct 2014

I think JDownloads fixed it in their latest release. You would have to use an older one.

avatar nicksavov nicksavov - change - 17 Oct 2014
The description was changed
Labels Added: ?
avatar brianteeman brianteeman - change - 17 Oct 2014
Category Libraries
avatar ColinM2
ColinM2 - comment - 30 Oct 2014

Sorry about delayed response but have been away. Yes jDownloads code was changed to avoid problem. I do have a previous jDownloads zip if you would like it. Approx 24Mb.

avatar Bakual
Bakual - comment - 30 Oct 2014

Can you link that here? I'm sure that would help.

avatar Bakual
Bakual - comment - 8 Nov 2014

Updated the PR so it also clears the offset. That doesn't change anything for all databases except PostgreSQL since the others already clear the offset together with the limit.

@ColinM2 If you can link that file, then it sure would help getting this tested.

avatar waader
waader - comment - 8 Nov 2014

@test I added this patch to the current staging branch and installed the test data to a mysql and postgresql installation. As there is no easy test case I manipulated the _getListCount function.

First I commented out the "fast part" - the first if block - and tested the "article category list" with the "inefficent part" - what comes after this block. That did work.

Then I added $query->group('a.id'); before the "inefficent part" and tested with the mysql installation. That did work too.

Are there other ideas how to test this?

avatar Bakual
Bakual - comment - 8 Nov 2014

Are there other ideas how to test this?

I think that's about the best you can do it.

avatar Bakual Bakual - close - 3 Dec 2014
avatar Bakual Bakual - change - 3 Dec 2014
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2014-12-03 08:05:59

Add a Comment

Login with GitHub to post a comment