Making Async and Timeout callable
[theonering] / src / channel / text.py
index 2b39833..d4cb5c0 100644 (file)
@@ -4,44 +4,38 @@ import logging
 
 import telepathy
 
-import util.go_utils as gobject_utils
+import tp
 import util.coroutines as coroutines
-import gtk_toolbox
+import util.misc as misc_utils
 
 
 _moduleLogger = logging.getLogger("channel.text")
 
 
-class TextChannel(telepathy.server.ChannelTypeText):
-       """
-       Look into implementing ChannelInterfaceMessages for rich text formatting
-       """
+class TextChannel(tp.ChannelTypeText):
+
+       NULL_TIMESTAMP = datetime.datetime(1, 1, 1)
 
        def __init__(self, connection, manager, props, contactHandle):
-               self._manager = manager
-               self._props = props
+               self.__manager = manager
+               self.__props = props
 
-               try:
-                       # HACK Older python-telepathy way
-                       telepathy.server.ChannelTypeText.__init__(self, connection, contactHandle)
-               except TypeError:
-                       # HACK Newer python-telepathy way
-                       telepathy.server.ChannelTypeText.__init__(self, connection, manager, props)
-               self._nextRecievedId = 0
-               self._lastMessageTimestamp = datetime.datetime(1, 1, 1)
+               tp.ChannelTypeText.__init__(self, connection, manager, props)
+               self.__nextRecievedId = 0
+               self.__lastMessageTimestamp = self.NULL_TIMESTAMP
 
-               self._otherHandle = contactHandle
+               self.__otherHandle = contactHandle
 
-               self._callback = coroutines.func_sink(
+               self.__callback = coroutines.func_sink(
                        coroutines.expand_positional(
                                self._on_conversations_updated
                        )
                )
                self._conn.session.voicemails.updateSignalHandler.register_sink(
-                       self._callback
+                       self.__callback
                )
                self._conn.session.texts.updateSignalHandler.register_sink(
-                       self._callback
+                       self.__callback
                )
 
                # The only reason there should be anything in the conversation is if
@@ -49,70 +43,91 @@ class TextChannel(telepathy.server.ChannelTypeText):
                try:
                        mergedConversations = self._conn.session.voicemails.get_conversation(self._contactKey)
                except KeyError:
-                       _moduleLogger.info("Nothing in the conversation yet for %r" % (self._contactKey, ))
+                       _moduleLogger.debug("No voicemails in the conversation yet for %r" % (self._contactKey, ))
                else:
                        self._report_conversation(mergedConversations)
                try:
                        mergedConversations = self._conn.session.texts.get_conversation(self._contactKey)
                except KeyError:
-                       _moduleLogger.info("Nothing in the conversation yet for %r" % (self._contactKey, ))
+                       _moduleLogger.debug("No texts conversation yet for %r" % (self._contactKey, ))
                else:
                        self._report_conversation(mergedConversations)
 
-       @gtk_toolbox.log_exception(_moduleLogger)
+       @misc_utils.log_exception(_moduleLogger)
        def Send(self, messageType, text):
                if messageType != telepathy.CHANNEL_TEXT_MESSAGE_TYPE_NORMAL:
                        raise telepathy.errors.NotImplemented("Unhandled message type: %r" % messageType)
 
-               self._conn.session.backend.send_sms(self._otherHandle.phoneNumber, text)
+               if self.__lastMessageTimestamp == self.NULL_TIMESTAMP:
+                       # Hack: GV marks messages as read when they are replied to.  If GV
+                       # marks them as read than we ignore them.  So reduce the window for
+                       # them being marked as read.  Oh and Conversations already handles
+                       # it if the message was already part of a thread, so we can limit
+                       # this to if we are trying to start a thread.  You might say a
+                       # voicemail could be what is being replied to and that doesn't mean
+                       # anything.  Oh well.
+                       try:
+                               self._conn.session.texts.update(force=True)
+                       except Exception:
+                               _moduleLogger.exception(
+                                       "Update failed when proactively checking for texts"
+                               )
+
+               _moduleLogger.info("Sending message to %r" % (self.__otherHandle, ))
+               self._conn.session.backend.send_sms(self.__otherHandle.phoneNumber, text)
                self._conn.session.textsStateMachine.reset_timers()
 
                self.Sent(int(time.time()), messageType, text)
 
-       @gtk_toolbox.log_exception(_moduleLogger)
+       @misc_utils.log_exception(_moduleLogger)
        def Close(self):
                self.close()
 
        def close(self):
+               _moduleLogger.debug("Closing text")
                self._conn.session.voicemails.updateSignalHandler.unregister_sink(
-                       self._callback
+                       self.__callback
                )
                self._conn.session.texts.updateSignalHandler.unregister_sink(
-                       self._callback
+                       self.__callback
                )
-               self._callback = None
+               self.__callback = None
 
-               telepathy.server.ChannelTypeText.Close(self)
-               if self._manager.channel_exists(self._props):
-                       # HACK Older python-telepathy requires doing this manually
-                       self._manager.remove_channel(self)
+               tp.ChannelTypeText.Close(self)
                self.remove_from_connection()
 
        @property
        def _contactKey(self):
-               contactKey = self._otherHandle.contactID, self._otherHandle.phoneNumber
+               contactKey = self.__otherHandle.phoneNumber
                return contactKey
 
-       @gobject_utils.async
-       @gtk_toolbox.log_exception(_moduleLogger)
+       @misc_utils.log_exception(_moduleLogger)
        def _on_conversations_updated(self, conv, conversationIds):
                if self._contactKey not in conversationIds:
                        return
-               _moduleLogger.info("Incoming messages from %r for existing conversation" % (self._contactKey, ))
+               _moduleLogger.debug("Incoming messages from %r for existing conversation" % (self._contactKey, ))
                mergedConversations = conv.get_conversation(self._contactKey)
                self._report_conversation(mergedConversations)
 
        def _report_conversation(self, mergedConversations):
+               # Can't filter out messages in a texting conversation that came in
+               # before the last one sent because that creates a race condition of two
+               # people sending at about the same time, which happens quite a bit
                newConversations = mergedConversations.conversations
+               if not newConversations:
+                       _moduleLogger.debug(
+                               "No messages ended up existing for %r" % (self._contactKey, )
+                       )
+                       return
                newConversations = self._filter_out_reported(newConversations)
                newConversations = self._filter_out_read(newConversations)
                newConversations = list(newConversations)
                if not newConversations:
-                       _moduleLogger.info(
+                       _moduleLogger.debug(
                                "New messages for %r have already been read externally" % (self._contactKey, )
                        )
                        return
-               self._lastMessageTimestamp = newConversations[-1].time
+               self.__lastMessageTimestamp = newConversations[-1].time
 
                messages = [
                        newMessage
@@ -121,7 +136,7 @@ class TextChannel(telepathy.server.ChannelTypeText):
                        if newMessage.whoFrom != "Me:"
                ]
                if not newConversations:
-                       _moduleLogger.info(
+                       _moduleLogger.debug(
                                "All incoming messages were really outbound messages for %r" % (self._contactKey, )
                        )
                        return
@@ -129,15 +144,18 @@ class TextChannel(telepathy.server.ChannelTypeText):
                for newMessage in messages:
                        formattedMessage = self._format_message(newMessage)
                        self._report_new_message(formattedMessage)
+               for conv in newConversations:
+                       conv.isRead = True
 
        def _filter_out_reported(self, conversations):
                return (
                        conversation
                        for conversation in conversations
-                       if self._lastMessageTimestamp < conversation.time
+                       if self.__lastMessageTimestamp < conversation.time
                )
 
-       def _filter_out_read(self, conversations):
+       @staticmethod
+       def _filter_out_read(conversations):
                return (
                        conversation
                        for conversation in conversations
@@ -148,11 +166,11 @@ class TextChannel(telepathy.server.ChannelTypeText):
                return " ".join(part.text.strip() for part in message.body)
 
        def _report_new_message(self, message):
-               currentReceivedId = self._nextRecievedId
+               currentReceivedId = self.__nextRecievedId
                timestamp = int(time.time())
                type = telepathy.CHANNEL_TEXT_MESSAGE_TYPE_NORMAL
 
-               _moduleLogger.info("Received message from User %r" % self._otherHandle)
-               self.Received(currentReceivedId, timestamp, self._otherHandle, type, 0, message)
+               _moduleLogger.info("Received message from User %r" % self.__otherHandle)
+               self.Received(currentReceivedId, timestamp, self.__otherHandle, type, 0, message)
 
-               self._nextRecievedId += 1
+               self.__nextRecievedId += 1