From bcab930f28d96d464d36da3eaa41cdc61f960328 Mon Sep 17 00:00:00 2001
From: Gaston Dombiak <gaston@jivesoftware.com>
Date: Tue, 1 Nov 2005 18:31:52 +0000
Subject: [PATCH] Do not fire invitation listeners with messages of type error.
 SMACK-102

git-svn-id: http://svn.igniterealtime.org/svn/repos/smack/trunk@3015 b35dd754-fafc-0310-a699-88a17e54d16e
---
 .../smackx/muc/MultiUserChat.java             | 29 ++++++++++---------
 1 file changed, 15 insertions(+), 14 deletions(-)

diff --git a/source/org/jivesoftware/smackx/muc/MultiUserChat.java b/source/org/jivesoftware/smackx/muc/MultiUserChat.java
index b71ed0b1c..2bfbcbcdd 100644
--- a/source/org/jivesoftware/smackx/muc/MultiUserChat.java
+++ b/source/org/jivesoftware/smackx/muc/MultiUserChat.java
@@ -35,10 +35,10 @@ import org.jivesoftware.smackx.packet.*;
 /**
  * A MultiUserChat is a conversation that takes place among many users in a virtual
  * room. A room could have many occupants with different affiliation and roles.
- * Possible affiliatons are "owner", "admin", "member", and "outcast". Possible roles 
+ * Possible affiliatons are "owner", "admin", "member", and "outcast". Possible roles
  * are "moderator", "participant", and "visitor". Each role and affiliation guarantees
  * different privileges (e.g. Send messages to all occupants, Kick participants and visitors,
- * Grant voice, Edit member list, etc.). 
+ * Grant voice, Edit member list, etc.).
  *
  * @author Gaston Dombiak
  */
@@ -74,12 +74,12 @@ public class MultiUserChat {
     static {
         XMPPConnection.addConnectionListener(new ConnectionEstablishedListener() {
             public void connectionEstablished(final XMPPConnection connection) {
-                // Set on every established connection that this client supports the Multi-User 
-                // Chat protocol. This information will be used when another client tries to 
+                // Set on every established connection that this client supports the Multi-User
+                // Chat protocol. This information will be used when another client tries to
                 // discover whether this client supports MUC or not.
                 ServiceDiscoveryManager.getInstanceFor(connection).addFeature(discoNamespace);
                 // Set the NodeInformationProvider that will provide information about the
-                // joined rooms whenever a disco request is received 
+                // joined rooms whenever a disco request is received
                 ServiceDiscoveryManager.getInstanceFor(connection).setNodeInformationProvider(
                     discoNode,
                     new NodeInformationProvider() {
@@ -89,7 +89,7 @@ public class MultiUserChat {
                             while (rooms.hasNext()) {
                                 answer.add(new DiscoverItems.Item((String)rooms.next()));
                             }
-                            return answer.iterator(); 
+                            return answer.iterator();
                         }
                     });
             }
@@ -138,9 +138,9 @@ public class MultiUserChat {
 
     /**
      * Returns an Iterator on the rooms where the user has joined using a given connection.
-     * The Iterator will contain Strings where each String represents a room 
+     * The Iterator will contain Strings where each String represents a room
      * (e.g. room@muc.jabber.org).
-     * 
+     *
      * @param connection the connection used to join the rooms.
      * @return an Iterator on the rooms where the user has joined using a given connection.
      */
@@ -149,14 +149,14 @@ public class MultiUserChat {
         if (rooms != null) {
             return rooms.iterator();
         }
-        // Return an iterator on an empty collection (i.e. the user never joined a room) 
+        // Return an iterator on an empty collection (i.e. the user never joined a room)
         return new ArrayList().iterator();
     }
-    
+
     /**
-     * Returns an Iterator on the rooms where the requested user has joined. The Iterator will 
+     * Returns an Iterator on the rooms where the requested user has joined. The Iterator will
      * contain Strings where each String represents a room (e.g. room@muc.jabber.org).
-     * 
+     *
      * @param connection the connection to use to perform the service discovery.
      * @param user the user to check. A fully qualified xmpp ID, e.g. jdoe@example.com.
      * @return an Iterator on the rooms where the requested user has joined.
@@ -175,7 +175,7 @@ public class MultiUserChat {
         }
         catch (XMPPException e) {
             e.printStackTrace();
-            // Return an iterator on an empty collection 
+            // Return an iterator on an empty collection
             return new ArrayList().iterator();
         }
     }
@@ -2554,7 +2554,8 @@ public class MultiUserChat {
                     MUCUser mucUser = 
                         (MUCUser) packet.getExtension("x", "http://jabber.org/protocol/muc#user");
                     // Check if the MUCUser extension includes an invitation
-                    if (mucUser.getInvite() != null) {
+                    if (mucUser.getInvite() != null &&
+                            ((Message) packet).getType() != Message.Type.ERROR) {
                         // Fire event for invitation listeners
                         fireInvitationListeners(packet.getFrom(), mucUser.getInvite().getFrom(),
                                 mucUser.getInvite().getReason(), mucUser.getPassword(), (Message) packet);