Software /
code /
prosody
Comparison
plugins/mod_admin_adhoc.lua @ 10543:46b456ed12bd
mod_admin_adhoc: Remove unused JID resource variables [luacheck]
author | Kim Alvefur <zash@zash.se> |
---|---|
date | Mon, 23 Dec 2019 21:37:16 +0100 |
parent | 10398:fa50789d6ec7 |
child | 11632:21a1b9fb08a1 |
comparison
equal
deleted
inserted
replaced
10542:f1886a48a6d4 | 10543:46b456ed12bd |
---|---|
57 | 57 |
58 local add_user_command_handler = adhoc_simple(add_user_layout, function(fields, err) | 58 local add_user_command_handler = adhoc_simple(add_user_layout, function(fields, err) |
59 if err then | 59 if err then |
60 return generate_error_message(err); | 60 return generate_error_message(err); |
61 end | 61 end |
62 local username, host, resource = jid.split(fields.accountjid); | 62 local username, host = jid.split(fields.accountjid); |
63 if module_host ~= host then | 63 if module_host ~= host then |
64 return { status = "completed", error = { message = "Trying to add a user on " .. host .. " but command was sent to " .. module_host}}; | 64 return { status = "completed", error = { message = "Trying to add a user on " .. host .. " but command was sent to " .. module_host}}; |
65 end | 65 end |
66 if (fields["password"] == fields["password-verify"]) and username and host then | 66 if (fields["password"] == fields["password-verify"]) and username and host then |
67 if usermanager_user_exists(username, host) then | 67 if usermanager_user_exists(username, host) then |
92 | 92 |
93 local change_user_password_command_handler = adhoc_simple(change_user_password_layout, function(fields, err) | 93 local change_user_password_command_handler = adhoc_simple(change_user_password_layout, function(fields, err) |
94 if err then | 94 if err then |
95 return generate_error_message(err); | 95 return generate_error_message(err); |
96 end | 96 end |
97 local username, host, resource = jid.split(fields.accountjid); | 97 local username, host = jid.split(fields.accountjid); |
98 if module_host ~= host then | 98 if module_host ~= host then |
99 return { | 99 return { |
100 status = "completed", | 100 status = "completed", |
101 error = { | 101 error = { |
102 message = "Trying to change the password of a user on " .. host .. " but command was sent to " .. module_host | 102 message = "Trying to change the password of a user on " .. host .. " but command was sent to " .. module_host |
134 return generate_error_message(err); | 134 return generate_error_message(err); |
135 end | 135 end |
136 local failed = {}; | 136 local failed = {}; |
137 local succeeded = {}; | 137 local succeeded = {}; |
138 for _, aJID in ipairs(fields.accountjids) do | 138 for _, aJID in ipairs(fields.accountjids) do |
139 local username, host, resource = jid.split(aJID); | 139 local username, host = jid.split(aJID); |
140 if (host == module_host) and usermanager_user_exists(username, host) and usermanager_delete_user(username, host) then | 140 if (host == module_host) and usermanager_user_exists(username, host) and usermanager_delete_user(username, host) then |
141 module:log("debug", "User %s has been deleted", aJID); | 141 module:log("debug", "User %s has been deleted", aJID); |
142 succeeded[#succeeded+1] = aJID; | 142 succeeded[#succeeded+1] = aJID; |
143 else | 143 else |
144 module:log("debug", "Tried to delete non-existant user %s", aJID); | 144 module:log("debug", "Tried to delete non-existant user %s", aJID); |
178 return generate_error_message(err); | 178 return generate_error_message(err); |
179 end | 179 end |
180 local failed = {}; | 180 local failed = {}; |
181 local succeeded = {}; | 181 local succeeded = {}; |
182 for _, aJID in ipairs(fields.accountjids) do | 182 for _, aJID in ipairs(fields.accountjids) do |
183 local username, host, resource = jid.split(aJID); | 183 local username, host = jid.split(aJID); |
184 if (host == module_host) and usermanager_user_exists(username, host) and disconnect_user(aJID) then | 184 if (host == module_host) and usermanager_user_exists(username, host) and disconnect_user(aJID) then |
185 succeeded[#succeeded+1] = aJID; | 185 succeeded[#succeeded+1] = aJID; |
186 else | 186 else |
187 failed[#failed+1] = aJID; | 187 failed[#failed+1] = aJID; |
188 end | 188 end |
210 | 210 |
211 local get_user_password_handler = adhoc_simple(get_user_password_layout, function(fields, err) | 211 local get_user_password_handler = adhoc_simple(get_user_password_layout, function(fields, err) |
212 if err then | 212 if err then |
213 return generate_error_message(err); | 213 return generate_error_message(err); |
214 end | 214 end |
215 local user, host, resource = jid.split(fields.accountjid); | 215 local user, host = jid.split(fields.accountjid); |
216 local accountjid; | 216 local accountjid; |
217 local password; | 217 local password; |
218 if host ~= module_host then | 218 if host ~= module_host then |
219 return { status = "completed", error = { message = "Tried to get password for a user on " .. host .. " but command was sent to " .. module_host } }; | 219 return { status = "completed", error = { message = "Tried to get password for a user on " .. host .. " but command was sent to " .. module_host } }; |
220 elseif usermanager_user_exists(user, host) then | 220 elseif usermanager_user_exists(user, host) then |
241 local get_user_roster_handler = adhoc_simple(get_user_roster_layout, function(fields, err) | 241 local get_user_roster_handler = adhoc_simple(get_user_roster_layout, function(fields, err) |
242 if err then | 242 if err then |
243 return generate_error_message(err); | 243 return generate_error_message(err); |
244 end | 244 end |
245 | 245 |
246 local user, host, resource = jid.split(fields.accountjid); | 246 local user, host = jid.split(fields.accountjid); |
247 if host ~= module_host then | 247 if host ~= module_host then |
248 return { status = "completed", error = { message = "Tried to get roster for a user on " .. host .. " but command was sent to " .. module_host } }; | 248 return { status = "completed", error = { message = "Tried to get roster for a user on " .. host .. " but command was sent to " .. module_host } }; |
249 elseif not usermanager_user_exists(user, host) then | 249 elseif not usermanager_user_exists(user, host) then |
250 return { status = "completed", error = { message = "User does not exist" } }; | 250 return { status = "completed", error = { message = "User does not exist" } }; |
251 end | 251 end |