Skip to content

Commit b93f7d3

Browse files
author
Lloyd Watkin
committed
At the point where we prevent owners from unsubscribing.
1 parent 4226590 commit b93f7d3

2 files changed

Lines changed: 5 additions & 3 deletions

File tree

src/main/java/org/buddycloud/channelserver/packetprocessor/iq/namespace/pubsub/set/UnsubscribeSet.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -111,8 +111,8 @@ public void process(Element elm, JID actorJID, IQ reqIQ, Element rsm)
111111
if (Affiliations.owner == existingAffiliation.getAffiliation()) {
112112
response.setType(Type.error);
113113
PacketError pe = new PacketError(
114-
org.xmpp.packet.PacketError.Condition.forbidden,
115-
org.xmpp.packet.PacketError.Type.auth);
114+
org.xmpp.packet.PacketError.Condition.not_allowed,
115+
org.xmpp.packet.PacketError.Type.cancel);
116116
response.setError(pe);
117117
outQueue.put(response);
118118
return;

src/test/java/org/buddycloud/channelserver/packetprocessor/iq/namespace/pubsub/set/UnsubscribeSetTest.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -82,8 +82,10 @@ public void testCanNotUnsubscribeAsOnlyNodeOwner() throws Exception {
8282
Assert.assertEquals(IQ.Type.error, response.getType());
8383

8484
PacketError error = response.getError();
85-
System.out.println(response.toXML());
8685
Assert.assertNotNull(error);
86+
87+
Assert.assertEquals(PacketError.Type.cancel, error.getType());
88+
Assert.assertEquals(PacketError.Condition.not_allowed, error.getCondition());
8789

8890
}
8991

0 commit comments

Comments
 (0)