Mercurial > prosody-modules
view mod_muc_hats_api/mod_muc_hats_api.lua @ 6029:cc665f343690
mod_firewall: SUBSCRIBED: Flip subscription check to match documentation
The documentation claims that this condition checks whether the recipient is
subscribed to the sender.
However, it was using the wrong method, and actually checking whether the
sender was subscribed to the recipient.
A quick poll of folk suggested that the documentation's approach is the right
one, so this should fix the code to match the documentation.
This should also fix the bundled anti-spam rules from blocking presence from
JIDs that you subscribe do (but don't have a mutual subscription with).
| author | Matthew Wild <mwild1@gmail.com> |
|---|---|
| date | Fri, 22 Nov 2024 13:50:48 +0000 |
| parents | 1f90e333b1d8 |
| children |
line wrap: on
line source
local mod_muc = module:depends("muc"); function add_user_hat(user_jid, room_jid, hat_id, attachment) local room = mod_muc.get_room_from_jid(room_jid); if not room then return nil, "item-not-found", "no such room"; end local user_aff = room:get_affiliation(user_jid); if not user_aff then return nil, "item-not-found", "user not affiliated with room"; end local aff_data = room:get_affiliation_data(user_jid) or {}; local hats = aff_data.hats; if not hats then hats = {}; aff_data.hats = hats; end hats[hat_id] = { active = attachment.active; required = attachment.required; title = attachment.title; }; local ok, err = room:set_affiliation(true, user_jid, user_aff, nil, aff_data); if not ok then return nil, err; end return true; end function remove_user_hat(user_jid, room_jid, hat_id) local room = mod_muc.get_room_from_jid(room_jid); if not room then return nil, "item-not-found", "no such room"; end local user_aff = room:get_affiliation(user_jid); if not user_aff then return nil, "item-not-found", "user not affiliated with room"; end local aff_data = room:get_affiliation_data(user_jid); local hats = aff_data and aff_data.hats; if not hats then return true; end hats[hat_id] = nil; local ok, err = room:set_affiliation(true, user_jid, user_aff, nil, aff_data); if not ok then return nil, err; end return true; end function set_user_hats(user_jid, room_jid, new_hats) local room = mod_muc.get_room_from_jid(room_jid); if not room then return nil, "item-not-found", "no such room"; end local user_aff = room:get_affiliation(user_jid); if not user_aff then return nil, "item-not-found", "user not affiliated with room"; end local aff_data = room:get_affiliation_data(user_jid) or {}; aff_data.hats = new_hats; local ok, err = room:set_affiliation(true, user_jid, user_aff, nil, aff_data); if not ok then return nil, err; end return true; end
