Mercurial > prosody-modules
changeset 2356:49a01b78b45f
mod_storage_muc_log: Address luacheck warnings
author | Kim Alvefur <zash@zash.se> |
---|---|
date | Wed, 09 Nov 2016 13:37:58 +0100 |
parents | 513332854531 |
children | 093b70378fa5 |
files | mod_storage_muc_log/mod_storage_muc_log.lua |
diffstat | 1 files changed, 6 insertions(+), 5 deletions(-) [+] |
line wrap: on
line diff
--- a/mod_storage_muc_log/mod_storage_muc_log.lua Tue Nov 08 00:16:39 2016 +0100 +++ b/mod_storage_muc_log/mod_storage_muc_log.lua Wed Nov 09 13:37:58 2016 +0100 @@ -1,3 +1,4 @@ +-- luacheck: ignore 212/self 431/err local datamanager = require"core.storagemanager".olddm; local xml_parse = require"util.xml".parse; @@ -48,7 +49,9 @@ return with and tag.name .. "<" .. with or tag.name; end -function driver:append(node, key, stanza, when, with) +function driver:append(node, key, stanza, when, with) -- luacheck: ignore 212/key + -- luacheck: ignore 311/with + -- 'with' doesn't exist in the original mod_muc_log, so gets derived here if type(when) ~= "number" then when, with, stanza = stanza, when, with; end @@ -83,8 +86,6 @@ table.sort(dates); return coroutine.wrap(function () - local query = query; - local dates = dates; local start_date = query and query.start and os_date(datef, query.start) or dates[1]; local end_date = query and query["end"] and os_date(datef, query["end"]) or dates[#dates]; local start_time = query and query.start and os_date(timef, query.start) or dates[1]; @@ -120,8 +121,8 @@ inner_start = inner_start + (query and query.reverse and -1 or 1); seek_once = nil; end - for i = inner_start, inner_stop, inner_step do - item, err = data[i]; + for i = inner_start, inner_stop, inner_step do -- luacheck: ignore 423/i + item, err = data[i], nil; if item then item, err = xml_parse(item); end