summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLukas Fleischer <lfleischer@archlinux.org>2016-03-01 18:30:48 +0100
committerLukas Fleischer <lfleischer@archlinux.org>2016-03-01 18:30:48 +0100
commit42f5c405c0280b6216028d0513e29c306e388426 (patch)
tree4097971ce440f365df2a0052042082400d20dd47
parent4c15f02ecb01e5d31ee575ead72c5b33db731032 (diff)
downloadaur-42f5c405c0280b6216028d0513e29c306e388426.tar.gz
aur-42f5c405c0280b6216028d0513e29c306e388426.tar.xz
aurjson: Remove stray GROUP BY clause
The IDs of packages are unique, so there is no need to group search results by package ID. Note that the GROUP BY statement in question was introduced in commit 3447dfc (Support versioned RPC queries, 2014-04-28) for no apparent reason and could even lead to errors in various DBMS. Signed-off-by: Lukas Fleischer <lfleischer@archlinux.org>
-rw-r--r--web/lib/aurjson.class.php1
1 files changed, 0 insertions, 1 deletions
diff --git a/web/lib/aurjson.class.php b/web/lib/aurjson.class.php
index 7f9b5f2e..3bd9179c 100644
--- a/web/lib/aurjson.class.php
+++ b/web/lib/aurjson.class.php
@@ -266,7 +266,6 @@ class AurJSON {
"ON Licenses.ID = PackageLicenses.LicenseID " .
"WHERE ${where_condition} " .
"AND PackageBases.PackagerUID IS NOT NULL " .
- "GROUP BY Packages.ID " .
"LIMIT $max_results";
} elseif ($this->version >= 2) {
if ($this->version == 2 || $this->version == 3) {