diff --git a/smack-core/src/main/java/org/jivesoftware/smack/packet/Message.java b/smack-core/src/main/java/org/jivesoftware/smack/packet/Message.java index 045532211..aafcbeb5e 100644 --- a/smack-core/src/main/java/org/jivesoftware/smack/packet/Message.java +++ b/smack-core/src/main/java/org/jivesoftware/smack/packet/Message.java @@ -24,6 +24,7 @@ import java.util.List; import java.util.Locale; import java.util.Set; +import org.jivesoftware.smack.util.TypedCloneable; import org.jivesoftware.smack.util.XmlStringBuilder; import org.jxmpp.jid.Jid; @@ -51,7 +52,7 @@ import org.jxmpp.jid.Jid; * * @author Matt Tucker */ -public final class Message extends Stanza implements Cloneable { +public final class Message extends Stanza implements TypedCloneable { public static final String ELEMENT = "message"; public static final String BODY = "body"; diff --git a/smack-core/src/main/java/org/jivesoftware/smack/packet/Presence.java b/smack-core/src/main/java/org/jivesoftware/smack/packet/Presence.java index 7f347dd51..c50e35f74 100644 --- a/smack-core/src/main/java/org/jivesoftware/smack/packet/Presence.java +++ b/smack-core/src/main/java/org/jivesoftware/smack/packet/Presence.java @@ -20,6 +20,7 @@ package org.jivesoftware.smack.packet; import java.util.Locale; import org.jivesoftware.smack.util.Objects; +import org.jivesoftware.smack.util.TypedCloneable; import org.jivesoftware.smack.util.XmlStringBuilder; /** @@ -55,7 +56,7 @@ import org.jivesoftware.smack.util.XmlStringBuilder; * * @author Matt Tucker */ -public final class Presence extends Stanza implements Cloneable { +public final class Presence extends Stanza implements TypedCloneable { public static final String ELEMENT = "presence"; diff --git a/smack-core/src/main/java/org/jivesoftware/smack/util/TypedCloneable.java b/smack-core/src/main/java/org/jivesoftware/smack/util/TypedCloneable.java new file mode 100644 index 000000000..226b5e3d8 --- /dev/null +++ b/smack-core/src/main/java/org/jivesoftware/smack/util/TypedCloneable.java @@ -0,0 +1,34 @@ +/** + * + * Copyright 2015 Florian Schmaus + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +package org.jivesoftware.smack.util; + +/** + * An extended version of {@link java.lang.Cloneable}, which defines a generic {@link #clone()} + * method. + * + * @param the type returned by {@link #clone()}. + */ +public interface TypedCloneable extends Cloneable { + + /** + * Clone this instance. + * + * @return a cloned version of this instance. + */ + public T clone(); + +} diff --git a/smack-extensions/src/main/java/org/jivesoftware/smackx/caps/EntityCapsManager.java b/smack-extensions/src/main/java/org/jivesoftware/smackx/caps/EntityCapsManager.java index 20df9cf67..5467aa108 100644 --- a/smack-extensions/src/main/java/org/jivesoftware/smackx/caps/EntityCapsManager.java +++ b/smack-extensions/src/main/java/org/jivesoftware/smackx/caps/EntityCapsManager.java @@ -201,7 +201,7 @@ public class EntityCapsManager extends Manager { DiscoverInfo info = CAPS_CACHE.get(nodeVer); // If it was not in CAPS_CACHE, try to retrieve the information from persistentCache - if (info == null) { + if (info == null && persistentCache != null) { info = persistentCache.lookup(nodeVer); // Promote the information to CAPS_CACHE if one was found if (info != null) { diff --git a/smack-extensions/src/main/java/org/jivesoftware/smackx/disco/packet/DiscoverInfo.java b/smack-extensions/src/main/java/org/jivesoftware/smackx/disco/packet/DiscoverInfo.java index e4f5ca0f6..774259868 100644 --- a/smack-extensions/src/main/java/org/jivesoftware/smackx/disco/packet/DiscoverInfo.java +++ b/smack-extensions/src/main/java/org/jivesoftware/smackx/disco/packet/DiscoverInfo.java @@ -18,6 +18,7 @@ package org.jivesoftware.smackx.disco.packet; import org.jivesoftware.smack.packet.IQ; import org.jivesoftware.smack.util.StringUtils; +import org.jivesoftware.smack.util.TypedCloneable; import org.jivesoftware.smack.util.XmlStringBuilder; import org.jxmpp.util.XmppStringUtils; @@ -38,7 +39,7 @@ import java.util.Set; * * @author Gaston Dombiak */ -public class DiscoverInfo extends IQ implements Cloneable { +public class DiscoverInfo extends IQ implements TypedCloneable { public static final String ELEMENT = QUERY_ELEMENT; public static final String NAMESPACE = "http://jabber.org/protocol/disco#info"; @@ -266,7 +267,7 @@ public class DiscoverInfo extends IQ implements Cloneable { * attributes. * */ - public static class Identity implements Comparable, Cloneable { + public static class Identity implements Comparable, TypedCloneable { private final String category; private final String type; @@ -473,7 +474,7 @@ public class DiscoverInfo extends IQ implements Cloneable { * as well as specific feature types of interest, if any (e.g., for the purpose of feature * negotiation). */ - public static class Feature implements Cloneable { + public static class Feature implements TypedCloneable { private final String variable; diff --git a/smack-extensions/src/main/java/org/jivesoftware/smackx/muc/packet/MUCItem.java b/smack-extensions/src/main/java/org/jivesoftware/smackx/muc/packet/MUCItem.java index 9fb2216ac..a5ded30ca 100644 --- a/smack-extensions/src/main/java/org/jivesoftware/smackx/muc/packet/MUCItem.java +++ b/smack-extensions/src/main/java/org/jivesoftware/smackx/muc/packet/MUCItem.java @@ -150,9 +150,7 @@ public class MUCItem implements NamedElement { xml.optAttribute("affiliation", getAffiliation()); xml.optAttribute("jid", getJid()); xml.optAttribute("nick", getNick()); - if (role != null && role != MUCRole.none) { - xml.attribute("role", getRole()); - } + xml.optAttribute("role", getRole()); xml.rightAngleBracket(); xml.optElement("reason", getReason()); if (getActor() != null) { diff --git a/smack-extensions/src/main/resources/org.jivesoftware.smack.extensions/extensions.xml b/smack-extensions/src/main/resources/org.jivesoftware.smack.extensions/extensions.xml index b6e64c901..36eaa40ed 100644 --- a/smack-extensions/src/main/resources/org.jivesoftware.smack.extensions/extensions.xml +++ b/smack-extensions/src/main/resources/org.jivesoftware.smack.extensions/extensions.xml @@ -16,5 +16,6 @@ org.jivesoftware.smackx.xdatalayout.XDataLayoutManager org.jivesoftware.smackx.xdatavalidation.XDataValidationManager org.jivesoftware.smackx.receipts.DeliveryReceiptManager + org.jivesoftware.smackx.iqversion.VersionManager diff --git a/smack-tcp/src/main/java/org/jivesoftware/smack/tcp/XMPPTCPConnection.java b/smack-tcp/src/main/java/org/jivesoftware/smack/tcp/XMPPTCPConnection.java index efb8b47c3..1cdbcbb60 100644 --- a/smack-tcp/src/main/java/org/jivesoftware/smack/tcp/XMPPTCPConnection.java +++ b/smack-tcp/src/main/java/org/jivesoftware/smack/tcp/XMPPTCPConnection.java @@ -517,6 +517,9 @@ public class XMPPTCPConnection extends AbstractXMPPConnection { disconnectedButResumeable = true; } else { disconnectedButResumeable = false; + // Reset the stream management session id to null, since if the stream is cleanly closed, i.e. sending a closing + // stream tag, there is no longer a stream to resume. + smSessionId = null; } authenticated = false; connected = false; @@ -524,9 +527,6 @@ public class XMPPTCPConnection extends AbstractXMPPConnection { reader = null; writer = null; - // Reset the stream management session id to null, since if the stream is cleanly closed, i.e. sending a closing - // stream tag, there is no longer a stream to resume. - smSessionId = null; maybeCompressFeaturesReceived.init(); compressSyncPoint.init(); smResumedSyncPoint.init();