Software / code / prosody
Changeset
13681:8f43b954bdac 13.0
mod_admin_shell: Fix result handling of user addrole/delrole commands
| author | Matthew Wild <mwild1@gmail.com> |
|---|---|
| date | Thu, 13 Feb 2025 15:54:39 +0000 |
| parents | 13680:22f1444f08f9 |
| children | 13682:0055c177a54c |
| files | plugins/mod_admin_shell.lua |
| diffstat | 1 files changed, 10 insertions(+), 2 deletions(-) [+] |
line wrap: on
line diff
--- a/plugins/mod_admin_shell.lua Thu Feb 13 15:31:37 2025 +0000 +++ b/plugins/mod_admin_shell.lua Thu Feb 13 15:54:39 2025 +0000 @@ -1855,7 +1855,11 @@ elseif userhost ~= host then return nil, "Can't add roles outside users own host" end - return um.add_user_secondary_role(username, host, new_role); + local role, err = um.add_user_secondary_role(username, host, new_role); + if not role then + return nil, err; + end + return true, "Role added"; end describe_command [[user:delrole(jid, host, role) - Remove a secondary role from a user]] @@ -1869,7 +1873,11 @@ elseif userhost ~= host then return nil, "Can't remove roles outside users own host" end - return um.remove_user_secondary_role(username, host, role_name); + local ok, err = um.remove_user_secondary_role(username, host, role_name); + if not ok then + return nil, err; + end + return true, "Role removed"; end describe_command [[user:list(hostname, pattern) - List users on the specified host, optionally filtering with a pattern]]
