From b51f1a42248f81370df2d2cbb29da059eadf33d0 Mon Sep 17 00:00:00 2001 From: Kim Alvefur Date: Tue, 26 Jun 2018 02:40:06 +0200 Subject: Backed out changeset d41f8ce67c8e Turns out this argument wasn?t only in trunk, but has been like this since 81406277279e --- plugins/muc/subject.lib.lua | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) (limited to 'plugins/muc') diff --git a/plugins/muc/subject.lib.lua b/plugins/muc/subject.lib.lua index b751bc49..56d8d174 100644 --- a/plugins/muc/subject.lib.lua +++ b/plugins/muc/subject.lib.lua @@ -13,7 +13,7 @@ local dt = require "util.datetime"; local muc_util = module:require "muc/util"; local valid_roles = muc_util.valid_roles; -local function create_subject_message(subject, from) +local function create_subject_message(from, subject) return st.message({from = from; type = "groupchat"}) :tag("subject"):text(subject or ""):up(); end @@ -54,7 +54,7 @@ end); local function get_subject(room) -- a stanza from the room JID (or from the occupant JID of the entity that set the subject) - return room._data.subject, room._data.subject_from or room.jid; + return room._data.subject_from or room.jid, room._data.subject; end local function send_subject(room, to, time) @@ -70,14 +70,14 @@ local function send_subject(room, to, time) room:route_stanza(msg); end -local function set_subject(room, subject, from) +local function set_subject(room, from, subject) if subject == "" then subject = nil; end - local old_subject, old_from = get_subject(room); + local old_from, old_subject = get_subject(room); if old_subject == subject and old_from == from then return false; end room._data.subject_from = from; room._data.subject = subject; room._data.subject_time = os.time(); - local msg = create_subject_message(subject, from); + local msg = create_subject_message(from, subject); room:broadcast_message(msg); return true; end @@ -99,7 +99,7 @@ module:hook("muc-occupant-groupchat", function(event) local role_rank = valid_roles[occupant and occupant.role or "none"]; if role_rank >= valid_roles.moderator or ( role_rank >= valid_roles.participant and get_changesubject(room) ) then -- and participant - set_subject(room, subject:get_text(), occupant.nick); + set_subject(room, occupant.nick, subject:get_text()); room:save(); return true; else -- cgit v1.2.3