User tests: Successful: Unsuccessful:
Same fix as #4330 but for queries which containg GROUP BY and/or HAVING.
Pagination doesn't show at all.
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.
Labels |
Added:
?
|
Tested AOK on 3.3.6 as expected
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.
I think JDownloads fixed it in their latest release. You would have to use an older one.
Labels |
Added:
?
|
Category | ⇒ | Libraries |
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.
Can you link that here? I'm sure that would help.
@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?
Are there other ideas how to test this?
I think that's about the best you can do it.
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2014-12-03 08:05:59 |
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.