Remove extra logic for handling unsubscribed in mod_shared_roster
This breaks some state changes, and it's original purpose (removing pending flag from roster item) works correctly without this it.
This commit is contained in:
@@ -336,13 +336,6 @@ in_subscription(Acc, #presence{to = To, from = JID, type = Type}) ->
|
||||
process_subscription(in, User, Server, JID, Type, Acc).
|
||||
|
||||
-spec out_subscription(presence()) -> boolean().
|
||||
out_subscription(#presence{from = From, to = To, type = unsubscribed} = Pres) ->
|
||||
#jid{user = User, server = Server} = From,
|
||||
mod_roster:out_subscription(Pres#presence{type = unsubscribe}),
|
||||
mod_roster:in_subscription(false, xmpp:set_from_to(
|
||||
Pres#presence{type = unsubscribe},
|
||||
To, From)),
|
||||
process_subscription(out, User, Server, To, unsubscribed, false);
|
||||
out_subscription(#presence{from = From, to = To, type = Type}) ->
|
||||
#jid{user = User, server = Server} = From,
|
||||
process_subscription(out, User, Server, To, Type, false).
|
||||
|
||||
Reference in New Issue
Block a user