PR-staging

Success

User tests: Successful: Unsuccessful:

avatar Hackwar
Hackwar
14 Nov 2014

To calculate the total number of items that a query returns, we try to use a faster method by replacing the whole SELECT part with a COUNT(). When we are using a UNION or UNIONALL in our query, this however fails, since the number of fields of the 2 different queries when using COUNT() might differ. This PR excludes such queries from being treated in that special way.

Again, not easily testable, needs a maintainer to do a code review.

I will provide another PR soon, that merges the update and discover view of com_installer and which uses this feature.

avatar Hackwar Hackwar - open - 14 Nov 2014
avatar jissues-bot jissues-bot - change - 14 Nov 2014
Labels Added: PR-staging
avatar chrisdavenport
chrisdavenport - comment - 14 Nov 2014

Makes sense. Good catch.

avatar brianteeman brianteeman - change - 15 Nov 2014
Category SQL
avatar Bakual
Bakual - comment - 15 Nov 2014

Makes sense! :+1:

avatar Bakual Bakual - change - 15 Nov 2014
Labels Added: ?
avatar Bakual
Bakual - comment - 15 Nov 2014

Set ton RTC

avatar phproberto phproberto - close - 20 Nov 2014
avatar zero-24 zero-24 - close - 20 Nov 2014
avatar phproberto phproberto - change - 20 Nov 2014
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2014-11-20 09:45:03
avatar zero-24 zero-24 - change - 14 Oct 2015
Labels Removed: ?

Add a Comment

Login with GitHub to post a comment