vanitasvitae/Smack
1
0
Fork 0
mirror of https://github.com/vanitasvitae/Smack.git synced 2024-11-29 07:22:07 +01:00
Code Issues Releases Wiki Activity

Always include role in MUCItem if it's set

Browse source 
As not including "role='none'" when kicking a user will result in an
XMPPErrorException. Also there appears to be nothing in XEP-45 which
says "if role is not set, then it defaults to 'none'".
This commit is contained in:
Florian Schmaus 2015-03-11 12:36:15 +01:00
parent e85c1881d7
commit f6ba363843
1 changed files with 1 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
smack-extensions/src/main/java/org/jivesoftware/smackx/muc/packet/MUCItem.java
View file

@ -148,9 +148,7 @@ public class MUCItem implements NamedElement {
xml.optAttribute("affiliation", getAffiliation()); xml.optAttribute("affiliation", getAffiliation());
xml.optAttribute("jid", getJid()); xml.optAttribute("jid", getJid());
xml.optAttribute("nick", getNick()); xml.optAttribute("nick", getNick());
if (role != null && role != MUCRole.none) { xml.optAttribute("role", getRole());
xml.attribute("role", getRole());
}
xml.rightAngleBracket(); xml.rightAngleBracket();
xml.optElement("reason", getReason()); xml.optElement("reason", getReason());
if (getActor() != null) { if (getActor() != null) {