Changeset

8946:3a095233e178

mod_pubsub: Handle optional item (thanks jonasw) The ‘item’ argument is optional, so it needs to handle it not being included, or util.stanza will throw an error in add_child
author Kim Alvefur <zash@zash.se>
date Thu, 28 Jun 2018 01:46:28 +0200
parents 8945:cbeb1a045eb6
children 8948:3be8799263f3
files plugins/mod_pubsub/mod_pubsub.lua
diffstat 1 files changed, 5 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/plugins/mod_pubsub/mod_pubsub.lua	Wed Jun 27 20:06:13 2018 +0200
+++ b/plugins/mod_pubsub/mod_pubsub.lua	Thu Jun 28 01:46:28 2018 +0200
@@ -55,10 +55,13 @@
 	local message = st.message({ from = module.host, type = msg_type, id = id })
 		:tag("event", { xmlns = xmlns_pubsub_event })
 			:tag(kind, { node = node })
-				:add_child(item);
+
+	if item then
+		message:add_child(item);
+	end
 
 	-- Compose a sensible textual representation of at least Atom payloads
-	if node_obj and node_obj.config.include_body and item.tags[1] then
+	if node_obj and item and node_obj.config.include_body and item.tags[1] then
 		local payload = item.tags[1];
 		if payload.attr.xmlns == "http://www.w3.org/2005/Atom" then
 			message:reset();