same thing, but nicer code

This commit is contained in:
louiz@4325f9fc-e183-4c21-96ce-0ab188b42d13 2011-01-05 01:46:19 +00:00
parent 1f2959d96e
commit 98c32b05e8

View file

@ -201,16 +201,12 @@ class Core(object):
self.information('%s is offline' % (resource.get_jid()), "Roster") self.information('%s is offline' % (resource.get_jid()), "Roster")
# If a resource got offline, display the message in the conversation with this # If a resource got offline, display the message in the conversation with this
# precise resource. # precise resource.
tab = self.get_tab_by_name(jid.full) self.add_information_message_to_conversation_tab(jid.full, '%s is offline' % (resource.get_jid().full))
if tab and isinstance(tab, tabs.ConversationTab):
self.add_message_to_text_buffer(tab.get_room(), '%s is offline' % (resource.get_jid().full))
contact.remove_resource(resource) contact.remove_resource(resource)
# Display the message in the conversation with the bare JID only if that was # Display the message in the conversation with the bare JID only if that was
# the only resource online (i.e. now the contact is completely disconnected) # the only resource online (i.e. now the contact is completely disconnected)
if not contact.get_highest_priority_resource(): # No resource left: that was the last one if not contact.get_highest_priority_resource(): # No resource left: that was the last one
tab = self.get_tab_by_name(jid.bare) self.add_information_message_to_conversation_tab(jid.bare, '%s is offline' % (jid.bare))
if tab and isinstance(tab, tabs.ConversationTab):
self.add_message_to_text_buffer(tab.get_room(), '%s is offline' % (jid.bare))
if isinstance(self.current_tab(), tabs.RosterInfoTab): if isinstance(self.current_tab(), tabs.RosterInfoTab):
self.refresh_window() self.refresh_window()
@ -231,6 +227,19 @@ class Core(object):
resource.set_priority(priority) resource.set_priority(priority)
contact.add_resource(resource) contact.add_resource(resource)
self.information("%s is online (%s)" % (resource.get_jid().full, status), "Roster") self.information("%s is online (%s)" % (resource.get_jid().full, status), "Roster")
tab = self.get_tab_by_name(jid.full)
if tab and isinstance(tab, tabs.ConversationTab):
self.add_message_to_text_buffer(tab.get_room(), '%s is offline' % (resource.get_jid().full))
def add_information_message_to_conversation_tab(self, jid, msg):
"""
Search for a ConversationTab with the given jid (full or bare), if yes, add
the given message to it
"""
tab = self.get_tab_by_name(jid)
if tab and isinstance(tab, tabs.ConversationTab):
self.add_message_to_text_buffer(tab.get_room(), msg)
def on_connected(self, event): def on_connected(self, event):
""" """