Improve MUC's Destroy element class

- Made jid of type BareJid
- Made it implement TypedCloneable
- Made it implement Serializable
- Made it immutable

Also update its parsing code. And add some convenience methods to
ParserUtils.

smack-core/src/main/java/org/jivesoftware/smack/util/ParserUtils.java

@@ -19,6 +19,7 @@ package org.jivesoftware.smack.util;
 import java.io.IOException;
 import java.util.Locale;
 
+import org.jxmpp.jid.BareJid;
 import org.jxmpp.jid.Jid;
 import org.jxmpp.jid.impl.JidCreate;
 import org.jxmpp.jid.parts.Resourcepart;
@@ -27,6 +28,12 @@ import org.xmlpull.v1.XmlPullParser;
 import org.xmlpull.v1.XmlPullParserException;
 
 public class ParserUtils {
+
+    /**
+     * The constant String "jid".
+     */
+    public static final String JID = "jid";
+
     public static void assertAtStartTag(XmlPullParser parser) throws XmlPullParserException {
         assert(parser.getEventType() == XmlPullParser.START_TAG);
     }
@@ -44,7 +51,7 @@ public class ParserUtils {
     }
 
     public static Jid getJidAttribute(XmlPullParser parser) throws XmppStringprepException {
-        return getJidAttribute(parser, "jid");
+        return getJidAttribute(parser, JID);
     }
 
     public static Jid getJidAttribute(XmlPullParser parser, String name) throws XmppStringprepException {
@@ -55,6 +62,18 @@ public class ParserUtils {
         return JidCreate.from(jidString);
     }
 
+    public static BareJid getBareJidAttribute(XmlPullParser parser) throws XmppStringprepException {
+        return getBareJidAttribute(parser, JID);
+    }
+
+    public static BareJid getBareJidAttribute(XmlPullParser parser, String name) throws XmppStringprepException {
+        final String jidString = parser.getAttributeValue("", name);
+        if (jidString == null) {
+            return null;
+        }
+        return JidCreate.bareFrom(jidString);
+    }
+
     public static Resourcepart getResourcepartAttribute(XmlPullParser parser, String name) throws XmppStringprepException {
         final String resourcepartString = parser.getAttributeValue("", name);
         if (resourcepartString == null) {

smack-extensions/src/main/java/org/jivesoftware/smackx/muc/MultiUserChat.java

@@ -655,15 +655,13 @@ public class MultiUserChat {
      * @throws NotConnectedException 
      * @throws InterruptedException 
      */
-    public void destroy(String reason, String alternateJID) throws NoResponseException, XMPPErrorException, NotConnectedException, InterruptedException {
+    public void destroy(String reason, BareJid alternateJID) throws NoResponseException, XMPPErrorException, NotConnectedException, InterruptedException {
         MUCOwner iq = new MUCOwner();
         iq.setTo(room);
         iq.setType(IQ.Type.set);
 
         // Create the reason for the room destruction
-        Destroy destroy = new Destroy();
+        Destroy destroy = new Destroy(alternateJID, reason);
-        destroy.setReason(reason);
-        destroy.setJid(alternateJID);
         iq.setDestroy(destroy);
 
         connection.createPacketCollectorAndSend(iq).nextResultOrThrow();

smack-extensions/src/main/java/org/jivesoftware/smackx/muc/packet/Destroy.java

@@ -16,8 +16,12 @@
  */
 package org.jivesoftware.smackx.muc.packet;
 
+import java.io.Serializable;
+
 import org.jivesoftware.smack.packet.NamedElement;
+import org.jivesoftware.smack.util.TypedCloneable;
 import org.jivesoftware.smack.util.XmlStringBuilder;
+import org.jxmpp.jid.BareJid;
 
 /**
  * Represents a request to the server to destroy a room. The sender of the request should be the
@@ -26,18 +30,32 @@ import org.jivesoftware.smack.util.XmlStringBuilder;
  * 
  * @author Gaston Dombiak
  */
-public class Destroy implements NamedElement {
+public class Destroy implements NamedElement, TypedCloneable<Destroy>, Serializable {
+    /**
+     *
+     */
+    private static final long serialVersionUID = 1L;
+
     public static final String ELEMENT = "destroy";
 
-    private String reason;
+    private final String reason;
-    private String jid;
+    private final BareJid jid;
+
+    public Destroy(Destroy other) {
+        this(other.jid, other.reason);
+    }
+
+    public Destroy(BareJid alternativeJid, String reason) {
+        this.jid = alternativeJid;
+        this.reason = reason;
+    }
 
     /**
      * Returns the JID of an alternate location since the current room is being destroyed.
      * 
      * @return the JID of an alternate location.
      */
-    public String getJid() {
+    public BareJid getJid() {
         return jid;
     }
 
@@ -50,24 +68,6 @@ public class Destroy implements NamedElement {
         return reason;
     }
 
-    /**
-     * Sets the JID of an alternate location since the current room is being destroyed.
-     * 
-     * @param jid the JID of an alternate location.
-     */
-    public void setJid(String jid) {
-        this.jid = jid;
-    }
-
-    /**
-     * Sets the reason for the room destruction.
-     * 
-     * @param reason the reason for the room destruction.
-     */
-    public void setReason(String reason) {
-        this.reason = reason;
-    }
-
     @Override
     public XmlStringBuilder toXML() {
         XmlStringBuilder xml = new XmlStringBuilder(this);
@@ -83,4 +83,8 @@ public class Destroy implements NamedElement {
         return ELEMENT;
     }
 
+    @Override
+    public Destroy clone() {
+        return new Destroy(this);
+    }
 }

smack-extensions/src/main/java/org/jivesoftware/smackx/muc/provider/MUCParserUtils.java

@@ -23,6 +23,7 @@ import org.jivesoftware.smackx.muc.MUCAffiliation;
 import org.jivesoftware.smackx.muc.MUCRole;
 import org.jivesoftware.smackx.muc.packet.Destroy;
 import org.jivesoftware.smackx.muc.packet.MUCItem;
+import org.jxmpp.jid.BareJid;
 import org.jxmpp.jid.Jid;
 import org.jxmpp.jid.parts.Resourcepart;
 import org.xmlpull.v1.XmlPullParser;
@@ -61,22 +62,27 @@ public class MUCParserUtils {
     }
 
     public static Destroy parseDestroy(XmlPullParser parser) throws XmlPullParserException, IOException {
-        boolean done = false;
+        final int initialDepth = parser.getDepth();
-        Destroy destroy = new Destroy();
+        final BareJid jid = ParserUtils.getBareJidAttribute(parser);
-        destroy.setJid(parser.getAttributeValue("", "jid"));
+        String reason = null;
-        while (!done) {
+        outerloop: while (true) {
             int eventType = parser.next();
-            if (eventType == XmlPullParser.START_TAG) {
+            switch (eventType) {
-                if (parser.getName().equals("reason")) {
+            case XmlPullParser.START_TAG:
-                    destroy.setReason(parser.nextText());
+                final String name = parser.getName();
+                switch (name) {
+                case "reason":
+                    reason = parser.nextText();
+                    break;
                 }
-            }
+                break;
-            else if (eventType == XmlPullParser.END_TAG) {
+            case XmlPullParser.END_TAG:
-                if (parser.getName().equals("destroy")) {
+                if (initialDepth == parser.getDepth()) {
-                    done = true;
+                    break outerloop;
                 }
+                break;
             }
         }
-        return destroy;
+        return new Destroy(jid, reason);
     }
 }