From da034b309056827f7901b938d4be59ff20ccca6d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jean-S=C3=A9bastien=20P=C3=A9dron?= Date: Mon, 29 Sep 2008 11:30:25 +0000 Subject: [PATCH] Fix a bug in #xmlel construction: children must be a list. SVN Revision: 1583 --- ChangeLog | 3 +++ src/mod_roster.erl | 2 +- src/mod_roster_odbc.erl | 2 +- 3 files changed, 5 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index 52db9ee99..dcf564881 100644 --- a/ChangeLog +++ b/ChangeLog @@ -19,6 +19,9 @@ * src/mod_shared_roster.erl: Convert to exmpp (not tested yet). + * src/mod_roster.erl (push_item), src/mod_roster_odbc.erl (push_item): + Fix a bug in #xmlel construction: children must be a list. + 2008-09-25 Jean-Sébastien Pédron * src/jlib.erl (timestamp_to_xml): Create an #xmlel element, not an diff --git a/src/mod_roster.erl b/src/mod_roster.erl index b97171a5e..ebaebbbc0 100644 --- a/src/mod_roster.erl +++ b/src/mod_roster.erl @@ -319,7 +319,7 @@ push_item(User, Server, From, Item) -> % TODO: don't push to those who didn't load roster push_item(User, Server, Resource, From, Item) -> Request = #xmlel{ns = ?NS_ROSTER, name = 'query', - children = item_to_xml(Item)}, + children = [item_to_xml(Item)]}, ResIQ = exmpp_iq:set(?NS_JABBER_CLIENT, Request, "push"), ejabberd_router:route( From, diff --git a/src/mod_roster_odbc.erl b/src/mod_roster_odbc.erl index aabe62e65..3b7d84eee 100644 --- a/src/mod_roster_odbc.erl +++ b/src/mod_roster_odbc.erl @@ -354,7 +354,7 @@ push_item(User, Server, From, Item) -> % TODO: don't push to those who not load roster push_item(User, Server, Resource, From, Item) -> Request = #xmlel{ns = ?NS_ROSTER, name = 'query', - children = item_to_xml(Item)}, + children = [item_to_xml(Item)]}, ResIQ = exmpp_iq:set(?NS_JABBER_CLIENT, Request, "push"), ejabberd_router:route( From,