summaryrefslogtreecommitdiffstats
path: root/web/lib
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 /web/lib
parent4c15f02ecb01e5d31ee575ead72c5b33db731032 (diff)
downloadaurweb-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>
Diffstat (limited to 'web/lib')
-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 7f9b5f2..3bd9179 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) {