Software / code / prosody
Changeset
12599:5aafcf6a0730 0.12
mod_storage_sql: Fix summary API with Postgres (fixes #1766)
The ORDER BY and LIMIT clauses are not needed and don't even make much
sense. This part was most likely a leftover from the :find method.
Tested with sqlite and postgres 14
| author | Kim Alvefur <zash@zash.se> |
|---|---|
| date | Fri, 22 Jul 2022 19:09:50 +0200 |
| parents | 12598:a2624315d30e |
| children | 12600:3d3a0c4e2662 12637:2200f0c6b3f1 |
| files | plugins/mod_storage_sql.lua |
| diffstat | 1 files changed, 2 insertions(+), 4 deletions(-) [+] |
line wrap: on
line diff
--- a/plugins/mod_storage_sql.lua Fri Jul 22 18:32:45 2022 +0200 +++ b/plugins/mod_storage_sql.lua Fri Jul 22 19:09:50 2022 +0200 @@ -544,8 +544,7 @@ SELECT DISTINCT "with", COUNT(*), MIN("when"), MAX("when") FROM "prosodyarchive" WHERE %s - GROUP BY "with" - ORDER BY "sort_id" %s%s; + GROUP BY "with"; ]]; local args = { host, user or "", store, }; local where = { "\"host\" = ?", "\"user\" = ?", "\"store\" = ?", }; @@ -558,8 +557,7 @@ args[#args+1] = query.limit; end - sql_query = sql_query:format(t_concat(where, " AND "), query.reverse - and "DESC" or "ASC", query.limit and " LIMIT ?" or ""); + sql_query = sql_query:format(t_concat(where, " AND ")); return engine:select(sql_query, unpack(args)); end); if not ok then return ok, result end
