aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorHolger Weiss <holger@zedat.fu-berlin.de>2021-03-16 17:26:47 +0100
committerHolger Weiss <holger@zedat.fu-berlin.de>2021-03-16 17:26:47 +0100
commit7008ae231cc4970a2d2b5cdd39752b84b3c61071 (patch)
treedc1bd9969aa1986d4c16773810ebd7c21f9d2aed /src
parentGenerate enabled_backends in ejabberd.app instead of configure (#3549) (diff)
Don't fail on PEP unsubscribe
Don't crash if a PEP node is explicitly unsubscribed. This fixes a regression introduced by 45eb49125ba46cd692a9fb430c567c0c2cf4a84d. Thanks to Melvin Keskin for reporting the bug.
Diffstat (limited to 'src')
-rw-r--r--src/node_pep.erl2
-rw-r--r--src/node_pep_sql.erl2
2 files changed, 2 insertions, 2 deletions
diff --git a/src/node_pep.erl b/src/node_pep.erl
index 8428be970..58c3050a0 100644
--- a/src/node_pep.erl
+++ b/src/node_pep.erl
@@ -128,7 +128,7 @@ subscribe_node(Nidx, Sender, Subscriber, AccessModel,
unsubscribe_node(Nidx, Sender, Subscriber, SubId) ->
case node_flat:unsubscribe_node(Nidx, Sender, Subscriber, SubId) of
{error, Error} -> {error, Error};
- {result, _} -> {result, []}
+ {result, _} -> {result, default}
end.
publish_item(Nidx, Publisher, Model, MaxItems, ItemId, Payload, PubOpts) ->
diff --git a/src/node_pep_sql.erl b/src/node_pep_sql.erl
index b62ef098a..7b21aa901 100644
--- a/src/node_pep_sql.erl
+++ b/src/node_pep_sql.erl
@@ -85,7 +85,7 @@ subscribe_node(Nidx, Sender, Subscriber, AccessModel,
unsubscribe_node(Nidx, Sender, Subscriber, SubId) ->
case node_flat_sql:unsubscribe_node(Nidx, Sender, Subscriber, SubId) of
{error, Error} -> {error, Error};
- {result, _} -> {result, []}
+ {result, _} -> {result, default}
end.
publish_item(Nidx, Publisher, Model, MaxItems, ItemId, Payload, PubOpts) ->