diff --git a/smack-core/src/main/java/org/jivesoftware/smack/packet/RosterPacket.java b/smack-core/src/main/java/org/jivesoftware/smack/packet/RosterPacket.java index 3ab7d87b5..d1b92ac1d 100644 --- a/smack-core/src/main/java/org/jivesoftware/smack/packet/RosterPacket.java +++ b/smack-core/src/main/java/org/jivesoftware/smack/packet/RosterPacket.java @@ -34,6 +34,9 @@ import java.util.concurrent.CopyOnWriteArraySet; */ public class RosterPacket extends IQ { + public static final String ELEMENT = QUERY_ELEMENT; + public static final String NAMESPACE = "jabber:iq:roster"; + private final List rosterItems = new ArrayList(); private String rosterVersion; @@ -73,8 +76,8 @@ public class RosterPacket extends IQ { @Override public XmlStringBuilder getChildElementXML() { XmlStringBuilder buf = new XmlStringBuilder(); - buf.halfOpenElement("query"); - buf.xmlnsAttribute("jabber:iq:roster"); + buf.halfOpenElement(ELEMENT); + buf.xmlnsAttribute(NAMESPACE); buf.optAttribute("ver", rosterVersion); buf.rightAngleBracket(); @@ -83,7 +86,7 @@ public class RosterPacket extends IQ { buf.append(entry.toXML()); } } - buf.closeElement("query"); + buf.closeElement(ELEMENT); return buf; } diff --git a/smack-core/src/main/java/org/jivesoftware/smack/util/PacketParserUtils.java b/smack-core/src/main/java/org/jivesoftware/smack/util/PacketParserUtils.java index 3b9976380..239bd7529 100644 --- a/smack-core/src/main/java/org/jivesoftware/smack/util/PacketParserUtils.java +++ b/smack-core/src/main/java/org/jivesoftware/smack/util/PacketParserUtils.java @@ -534,7 +534,7 @@ public class PacketParserUtils { if (elementName.equals("error")) { error = PacketParserUtils.parseError(parser); } - else if (elementName.equals("query") && namespace.equals("jabber:iq:roster")) { + else if (elementName.equals(RosterPacket.ELEMENT) && namespace.equals(RosterPacket.NAMESPACE)) { iqPacket = parseRoster(parser); } else if (elementName.equals(Bind.ELEMENT) && namespace.equals(Bind.NAMESPACE)) {