cleanup for presence parser

iNPUTmice created 11 years ago

Change summary

src/eu/siacs/conversations/generator/MessageGenerator.java  |  2 
src/eu/siacs/conversations/generator/PresenceGenerator.java | 26 ++----
src/eu/siacs/conversations/ui/MucDetailsActivity.java       |  2 
3 files changed, 11 insertions(+), 19 deletions(-)

Detailed changes

src/eu/siacs/conversations/generator/MessageGenerator.java

@@ -118,7 +118,7 @@ public class MessageGenerator {
 		return packet;
 	}
 	
-	public MessagePacket conversationSubject(Conversation conversation,String subject) {
+	public MessagePacket conferenceSubject(Conversation conversation,String subject) {
 		MessagePacket packet = new MessagePacket();
 		packet.setType(MessagePacket.TYPE_GROUPCHAT);
 		packet.setTo(conversation.getContactJid().split("/")[0]);

src/eu/siacs/conversations/generator/PresenceGenerator.java

@@ -6,36 +6,28 @@ import eu.siacs.conversations.xmpp.stanzas.PresencePacket;
 
 public class PresenceGenerator {
 
-	public PresencePacket requestPresenceUpdatesFrom(Contact contact) {
+	private PresencePacket subscription(String type, Contact contact) {
 		PresencePacket packet = new PresencePacket();
-		packet.setAttribute("type", "subscribe");
+		packet.setAttribute("type", type);
 		packet.setAttribute("to", contact.getJid());
 		packet.setAttribute("from", contact.getAccount().getJid());
 		return packet;
 	}
+	
+	public PresencePacket requestPresenceUpdatesFrom(Contact contact) {
+		return subscription("subscribe", contact);
+	}
 
 	public PresencePacket stopPresenceUpdatesFrom(Contact contact) {
-		PresencePacket packet = new PresencePacket();
-		packet.setAttribute("type", "unsubscribe");
-		packet.setAttribute("to", contact.getJid());
-		packet.setAttribute("from", contact.getAccount().getJid());
-		return packet;
+		return subscription("unsubscribe", contact);
 	}
 
 	public PresencePacket stopPresenceUpdatesTo(Contact contact) {
-		PresencePacket packet = new PresencePacket();
-		packet.setAttribute("type", "unsubscribed");
-		packet.setAttribute("to", contact.getJid());
-		packet.setAttribute("from", contact.getAccount().getJid());
-		return packet;
+		return subscription("unsubscribed", contact);
 	}
 
 	public PresencePacket sendPresenceUpdatesTo(Contact contact) {
-		PresencePacket packet = new PresencePacket();
-		packet.setAttribute("type", "subscribed");
-		packet.setAttribute("to", contact.getJid());
-		packet.setAttribute("from", contact.getAccount().getJid());
-		return packet;
+		return subscription("subscribed", contact);
 	}
 
 	public PresencePacket sendPresence(Account account) {

src/eu/siacs/conversations/ui/MucDetailsActivity.java

@@ -62,7 +62,7 @@ public class MucDetailsActivity extends XmppActivity {
 			String subject = mSubject.getText().toString();
 			MucOptions options = conversation.getMucOptions();
 			if (!subject.equals(options.getSubject())) {
-				MessagePacket packet = xmppConnectionService.getMessageGenerator().conversationSubject(conversation, subject);
+				MessagePacket packet = xmppConnectionService.getMessageGenerator().conferenceSubject(conversation, subject);
 				xmppConnectionService.sendMessagePacket(conversation.getAccount(), packet);
 				finish();
 			}