X-Git-Url: http://git.maemo.org/git/?p=theonering;a=blobdiff_plain;f=src%2Faliasing.py;h=22fa414d1828d3f2b821b900bb94d3db14254bcd;hp=cd107abb742180dd284c8a74aebded3d6c933747;hb=1c892d1b9bf14b28eb54ce3590ed2ee29d5e3d25;hpb=6ef9231c94a33ecc288d5927d0c8e22917be7f0f

diff --git a/src/aliasing.py b/src/aliasing.py
index cd107ab..22fa414 100644
--- a/src/aliasing.py
+++ b/src/aliasing.py
@@ -3,8 +3,7 @@ import logging
 import telepathy
 
 import tp
-import gtk_toolbox
-import util.misc as util_misc
+import util.misc as misc_utils
 import handle
 
 
@@ -72,7 +71,7 @@ def make_pretty(phonenumber):
 	if phonenumber is None or phonenumber is "":
 		return ""
 
-	phonenumber = util_misc.normalize_number(phonenumber)
+	phonenumber = misc_utils.normalize_number(phonenumber)
 
 	if phonenumber[0] == "+":
 		prettynumber = _make_pretty_international(phonenumber[1:])
@@ -101,22 +100,29 @@ class AliasingMixin(tp.ConnectionInterfaceAliasing):
 		"""
 		raise NotImplementedError("Abstract property called")
 
-	def handle(self, handleType, handleId):
+	@property
+	def callbackNumberParameter(self):
+		"""
+		@abstract
+		"""
+		raise NotImplementedError("Abstract property called")
+
+	def get_handle_by_id(self, handleType, handleId):
 		"""
 		@abstract
 		"""
 		raise NotImplementedError("Abstract function called")
 
-	@gtk_toolbox.log_exception(_moduleLogger)
+	@misc_utils.log_exception(_moduleLogger)
 	def GetAliasFlags(self):
 		return 0
 
-	@gtk_toolbox.log_exception(_moduleLogger)
+	@misc_utils.log_exception(_moduleLogger)
 	def RequestAliases(self, contactHandleIds):
 		_moduleLogger.debug("Called RequestAliases")
 		return [self._get_alias(handleId) for handleId in contactHandleIds]
 
-	@gtk_toolbox.log_exception(_moduleLogger)
+	@misc_utils.log_exception(_moduleLogger)
 	def GetAliases(self, contactHandleIds):
 		_moduleLogger.debug("Called GetAliases")
 
@@ -126,7 +132,7 @@ class AliasingMixin(tp.ConnectionInterfaceAliasing):
 		)
 		return idToAlias
 
-	@gtk_toolbox.log_exception(_moduleLogger)
+	@misc_utils.log_exception(_moduleLogger)
 	def SetAliases(self, aliases):
 		_moduleLogger.debug("Called SetAliases")
 		# first validate that no other handle types are included
@@ -137,11 +143,11 @@ class AliasingMixin(tp.ConnectionInterfaceAliasing):
 		else:
 			raise telepathy.errors.PermissionDenied("No user customizable aliases")
 
-		uglyNumber = util_misc.normalize_number(alias)
+		uglyNumber = misc_utils.normalize_number(alias)
 		if len(uglyNumber) == 0:
 			# Reset to the original from login if one was provided
 			uglyNumber = self.callbackNumberParameter
-		if not util_misc.is_valid_number(uglyNumber):
+		if not misc_utils.is_valid_number(uglyNumber):
 			raise telepathy.errors.InvalidArgument("Invalid phone number %r" % (uglyNumber, ))
 
 		# Update callback
@@ -160,8 +166,8 @@ class AliasingMixin(tp.ConnectionInterfaceAliasing):
 			return userAlias
 		else:
 			number = h.phoneNumber
-			if number:
+			try:
 				contactAlias = self.session.addressbook.get_contact_name(number)
-			else:
+			except KeyError:
 				contactAlias = make_pretty(number)
 			return contactAlias