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 |