diff options
| author | Badlop <badlop@process-one.net> | 2013-04-23 10:27:04 +0200 |
|---|---|---|
| committer | Badlop <badlop@process-one.net> | 2013-04-23 13:55:40 +0200 |
| commit | 066c47ca96e09411b21df0c8e590a7de32a1e6a5 (patch) | |
| tree | 242a4ed5b9a1ad312753c70d1950cced2fbb13eb /src | |
| parent | SCRAM optional parameter parsing bugfix (diff) | |
Include actor element in MUC stanzas when kicking (thanks to Purvesh Sahoo)(EJAB-1206)
Diffstat (limited to 'src')
| -rw-r--r-- | src/mod_muc/mod_muc_room.erl | 36 |
1 files changed, 26 insertions, 10 deletions
diff --git a/src/mod_muc/mod_muc_room.erl b/src/mod_muc/mod_muc_room.erl index d5414395..8e358536 100644 --- a/src/mod_muc/mod_muc_room.erl +++ b/src/mod_muc/mod_muc_room.erl @@ -2593,7 +2593,7 @@ process_admin_items_set(UJID, Items, Lang, StateData) -> SD; {JID, role, none, Reason} -> catch - send_kickban_presence(JID, + send_kickban_presence(UJID, JID, Reason, <<"307">>, SD), @@ -2605,7 +2605,7 @@ process_admin_items_set(UJID, Items, Lang, StateData) -> of true -> catch - send_kickban_presence(JID, + send_kickban_presence(UJID, JID, Reason, <<"321">>, none, @@ -2628,7 +2628,7 @@ process_admin_items_set(UJID, Items, Lang, StateData) -> {JID, affiliation, outcast, Reason} -> catch - send_kickban_presence(JID, + send_kickban_presence(UJID, JID, Reason, <<"301">>, outcast, @@ -2979,12 +2979,12 @@ can_change_ra(_FAffiliation, _FRole, _TAffiliation, _TRole, role, _Value, _ServiceAf) -> false. -send_kickban_presence(JID, Reason, Code, StateData) -> +send_kickban_presence(UJID, JID, Reason, Code, StateData) -> NewAffiliation = get_affiliation(JID, StateData), - send_kickban_presence(JID, Reason, Code, NewAffiliation, + send_kickban_presence(UJID, JID, Reason, Code, NewAffiliation, StateData). -send_kickban_presence(JID, Reason, Code, NewAffiliation, +send_kickban_presence(UJID, JID, Reason, Code, NewAffiliation, StateData) -> LJID = jlib:jid_tolower(JID), LJIDs = case LJID of @@ -3007,18 +3007,27 @@ send_kickban_presence(JID, Reason, Code, NewAffiliation, StateData#state.users), add_to_log(kickban, {Nick, Reason, Code}, StateData), tab_remove_online_user(J, StateData), - send_kickban_presence1(J, Reason, Code, + send_kickban_presence1(UJID, J, Reason, Code, NewAffiliation, StateData) end, LJIDs). -send_kickban_presence1(UJID, Reason, Code, Affiliation, +send_kickban_presence1(MJID, UJID, Reason, Code, Affiliation, StateData) -> {ok, #user{jid = RealJID, nick = Nick}} = (?DICT):find(jlib:jid_tolower(UJID), StateData#state.users), SAffiliation = affiliation_to_list(Affiliation), BannedJIDString = jlib:jid_to_string(RealJID), + case MJID /= <<"">> of + true -> + {ok, #user{jid = RealMJID}} = + (?DICT):find(jlib:jid_tolower(MJID), + StateData#state.users), + ActorJIDString = jlib:jid_to_string(RealMJID); + false -> + ActorJIDString = <<"">> + end, lists:foreach(fun ({_LJID, Info}) -> JidAttrList = case Info#user.role == moderator orelse (StateData#state.config)#config.anonymous @@ -3039,6 +3048,12 @@ send_kickban_presence1(UJID, Reason, Code, Affiliation, children = [{xmlcdata, Reason}]}] end, + ItemElsActor = case MJID of + <<"">> -> []; + _ -> [#xmlel{name = <<"actor">>, + attrs = + [{<<"jid">>, ActorJIDString}]}] + end, Packet = #xmlel{name = <<"presence">>, attrs = [{<<"type">>, <<"unavailable">>}], @@ -3053,7 +3068,7 @@ send_kickban_presence1(UJID, Reason, Code, Affiliation, attrs = ItemAttrs, children = - ItemEls}, + ItemElsActor ++ ItemEls}, #xmlel{name = <<"status">>, attrs = @@ -3067,6 +3082,7 @@ send_kickban_presence1(UJID, Reason, Code, Affiliation, end, (?DICT):to_list(StateData#state.users)). + %%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%% % Owner stuff @@ -3751,7 +3767,7 @@ remove_nonmembers(StateData) -> Affiliation = get_affiliation(JID, SD), case Affiliation of none -> - catch send_kickban_presence(JID, <<"">>, + catch send_kickban_presence(<<"">>, JID, <<"">>, <<"322">>, SD), set_role(JID, none, SD); _ -> SD |