From e584413813f3ee6f5b47a6227d335f1be16fd9e8 Mon Sep 17 00:00:00 2001
From: Christian Weiske <cweiske@cweiske.de>
Date: Sat, 29 May 2010 09:58:21 +0200
Subject: [PATCH] fix wrong xml type when getting roster items added (as per
 RFC3921, 7.4. "Adding a Roster Item". Thanks to neustradamus@jabber.org for
 notifying me about that

---
 XMPPHP/XMPP.php | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/XMPPHP/XMPP.php b/XMPPHP/XMPP.php
index ffd9340..97793bc 100644
--- a/XMPPHP/XMPP.php
+++ b/XMPPHP/XMPP.php
@@ -341,6 +341,8 @@ class XMPPHP_XMPP extends XMPPHP_XMLStream {
 	* Roster iq handler
 	* Gets all packets matching XPath "iq/{jabber:iq:roster}query'
 	*
+	* Implements RFC3921, 7.4. "Adding a Roster Item"
+	*
 	* @param string $xml
 	*/
 	protected function roster_iq_handler($xml) {
@@ -368,7 +370,7 @@ class XMPPHP_XMPP extends XMPPHP_XMLStream {
 			}
 		}
 		if ($xml->attrs['type'] == 'set') {
-			$this->send("<iq type=\"reply\" id=\"{$xml->attrs['id']}\" to=\"{$xml->attrs['from']}\" />");
+			$this->send("<iq type=\"result\" id=\"{$xml->attrs['id']}\" to=\"{$xml->attrs['from']}\" />");
 		}
 	}