Changeset

11908:624c14b77bb4

MUC: Switch to event.allowed signaling to block event, matching muc-pre-set-role ...and fixing the logic bug that broke everything in the previous commit.
author Matthew Wild <mwild1@gmail.com>
date Tue, 16 Nov 2021 11:52:36 +0000
parents 11907:0dc2c3530d64
children 11909:b33d71c43ac0
files plugins/muc/muc.lib.lua
diffstat 1 files changed, 3 insertions(+), 1 deletions(-) [+]
line wrap: on
line diff
--- a/plugins/muc/muc.lib.lua	Tue Nov 16 11:41:08 2021 +0000
+++ b/plugins/muc/muc.lib.lua	Tue Nov 16 11:52:36 2021 +0000
@@ -1432,7 +1432,9 @@
 		previous_affiliation = target_affiliation;
 		data = data and data or nil; -- coerce false to nil
 	};
-	if not module:fire_event("muc-pre-set-affiliation", event_data) then
+
+	module:fire_event("muc-pre-set-affiliation", event_data);
+	if event.allowed == false then
 		local err = event_data.error or { type = "cancel", condition = "not-allowed" };
 		return nil, err.type, err.condition;
 	end