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