From: jamesbunton <jamesbunton@55fbd22a-6204-0410-b2f0-b6c764c7e90a>
Date: Sun, 16 Apr 2006 10:20:28 +0000 (+0000)
Subject: New resources don't get spammed with presence from offline contacts.
X-Git-Url: https://code.delx.au/pymsnt/commitdiff_plain/253173617cb25feab97831a951f227d0d04d7129

New resources don't get spammed with presence from offline contacts.

git-svn-id: http://delx.cjb.net/svn/pymsnt/trunk@134 55fbd22a-6204-0410-b2f0-b6c764c7e90a

committer: jamesbunton <jamesbunton@55fbd22a-6204-0410-b2f0-b6c764c7e90a>
---

diff --git a/src/contact.py b/src/contact.py
index 2e58902..088f1c0 100644
--- a/src/contact.py
+++ b/src/contact.py
@@ -32,9 +32,9 @@ class Contact:
 	def syncContactGrantedAuth(self):
 		""" Since last using the transport the user has been granted authorisation by this contact.
 		Call this to synchronise the user's Jabber list with their legacy list after logon. """
-		if(self.sub == "none"):
+		if self.sub == "none":
 			self.sub = "to"
-		elif(self.sub == "from"):
+		elif self.sub == "from":
 			self.sub = "both"
 		else:
 			return
@@ -42,9 +42,9 @@ class Contact:
 	def syncContactRemovedAuth(self):
 		""" Since last using the transport the user has been blocked by this contact.
 		Call this to synchronise the user's Jabber list with their legacy list after logon. """
-		if(self.sub == "to"):
+		if self.sub == "to":
 			self.sub = "none"
-		elif(self.sub == "both"):
+		elif self.sub == "both":
 			self.sub = "from"
 		else:
 			return
@@ -52,9 +52,9 @@ class Contact:
 	def syncUserGrantedAuth(self):
 		""" Since last using the transport the user has granted authorisation to this contact.
 		Call this to synchronise the user's Jabber list with their legacy list after logon. """
-		if(self.sub == "none"):
+		if self.sub == "none":
 			self.sub = "from"
-		elif(self.sub == "to"):
+		elif self.sub == "to":
 			self.sub = "both"
 		else:
 			return
@@ -62,9 +62,9 @@ class Contact:
 	def syncUserRemovedAuth(self):
 		""" Since last using the transport the user has removed this contact's authorisation.
 		Call this to synchronise the user's Jabber list with their legacy list after logon. """
-		if(self.sub == "from"):
+		if self.sub == "from":
 			self.sub = "none"
-		elif(self.sub == "both"):
+		elif self.sub == "both":
 			self.sub = "to"
 		else:
 			return
@@ -102,18 +102,18 @@ class Contact:
 
 	def contactGrantsAuth(self):
 		""" Live roster event """
-		if(self.sub == "none"):
+		if self.sub == "none":
 			self.sub = "to"
-		elif(self.sub == "from"):
+		elif self.sub == "from":
 			self.sub = "both"
 		self.sendSub("subscribed")
 		self.sendPresence()
 	
 	def contactRemovesAuth(self):
 		""" Live roster event """
-		if(self.sub == "to"):
+		if self.sub == "to":
 			self.sub = "none"
-		elif(self.sub == "both"):
+		elif self.sub == "both":
 			self.sub = "from"
 		self.sendSub("unsubscribed")
 	
@@ -149,37 +149,39 @@ class Contact:
 				self.sub = "none"
 			self.contactList.legacyList.removeContact(self.jid)
 
-		elif(subtype == "unsubscribed"):
-			if(self.sub == "both"):
+		elif subtype == "unsubscribed":
+			if self.sub == "both":
 				self.sub = "to"
-			if(self.sub == "from"):
+			if self.sub == "from":
 				self.sub = "none"
 			self.contactList.legacyList.deauthContact(self.jid)
 
 	def updateNickname(self, nickname, push=True):
-		if(self.nickname != nickname):
+		if self.nickname != nickname:
 			self.nickname = nickname
-			if(push): self.sendPresence()
+			if push: self.sendPresence()
 	
 	def updatePresence(self, show, status, ptype, force=False):
 		updateFlag = (self.show != show or self.status != status or self.ptype != ptype or force)
 		self.show = show
 		self.status = status
 		self.ptype = ptype
-		if(updateFlag):
+		if updateFlag:
 			self.sendPresence()
 	
 	def updateAvatar(self, avatar=None, push=True):
-		if(self.avatar == avatar): return
+		if self.avatar == avatar:
+			return
 		self.avatar = avatar
-		if(push): self.sendPresence()
+		if push:
+			self.sendPresence()
 	
 	def sendSub(self, ptype):
 		self.contactList.session.sendPresence(to=self.contactList.session.jabberID, fro=self.jid + "/" + legacy.id, ptype=ptype)
 	
 	def sendPresence(self, tojid=""):
 		avatarHash = ""
-		if(self.avatar):
+		if self.avatar:
 			avatarHash = self.avatar.getImageHash()
 		caps = Element((None, "c"))
 		caps.attributes["xmlns"] = disco.CAPS
@@ -210,7 +212,7 @@ class ContactList:
 	
 	def resendLists(self, tojid=""):
 		for jid in self.contacts:
-			if(self.contacts[jid].status != "unavailable"):
+			if self.contacts[jid].ptype != "unavailable" :
 				self.contacts[jid].sendPresence(tojid)
 		LogEvent(INFO, self.session.jabberID)
 	
@@ -225,12 +227,12 @@ class ContactList:
 	
 	def getContact(self, jid):
 		""" Finds the contact. If one doesn't exist then a new one is created, with sub set to "none" """
-		if(not self.contacts.has_key(jid)):
+		if not self.contacts.has_key(jid):
 			self.contacts[jid] = Contact(jid, "none", self)
 		return self.contacts[jid]
 	
 	def findContact(self, jid):
-		if(self.contacts.has_key(jid)):
+		if self.contacts.has_key(jid):
 			return self.contacts[jid]
 		return None