# HG changeset patch
# User Kim Alvefur <zash@zash.se>
# Date 1533567851 -7200
# Node ID db47db7882959bae94fc1ce3f4c71c2ff46b9f4b
# Parent  4803d2feeecbde27760a4169b7f53dbe8557656f
util.pubsub: Re-check all subscriptions on access_model change, unsubscribing those no longer allowed

diff -r 4803d2feeecb -r db47db788295 util/pubsub.lua
--- a/util/pubsub.lua	Mon Aug 06 17:16:42 2018 +0200
+++ b/util/pubsub.lua	Mon Aug 06 17:04:11 2018 +0200
@@ -598,6 +598,18 @@
 		end
 	end
 
+	if old_config["access_model"] ~= node_obj.config["access_model"] then
+		for subscriber in pairs(node_obj.subscribers) do
+			if not self:may(node, subscriber, "be_subscribed") then
+				local ok, err = self:remove_subscription(node, true, subscriber);
+				if not ok then
+					node_obj.config = old_config;
+					return ok, err;
+				end
+			end
+		end
+	end
+
 	if old_config["persist_items"] ~= node_obj.config["persist_items"] then
 		self.data[node] = self.config.itemstore(self.nodes[node].config, node);
 	elseif old_config["max_items"] ~= node_obj.config["max_items"] then