From 9e7842d0146d64f6a99aa5950a3f4308bb1d8a16 Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Sun, 18 Dec 2011 20:16:19 +0100
Subject: Beep when receiving a MUC invitation (by default)

---
 src/core.py | 2 ++
 1 file changed, 2 insertions(+)

(limited to 'src')

diff --git a/src/core.py b/src/core.py
index 3bb0436e..d0f6c4de 100644
--- a/src/core.py
+++ b/src/core.py
@@ -371,6 +371,8 @@ class Core(object):
         if password:
             msg += ". The password is \"%s\"." % password
         self.information(msg, 'Info')
+        if 'invite' in config.get('beep_on', 'invite').split():
+            curses.beep()
         self.pending_invites[jid.bare] = inviter.full
 
     def command_invite(self, arg):
-- 
cgit v1.2.3


From 2ed6b205940c65206a7732795727fc2ed2338ef4 Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Sun, 18 Dec 2011 20:35:19 +0100
Subject: Add a /invites command

---
 src/core.py | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

(limited to 'src')

diff --git a/src/core.py b/src/core.py
index d0f6c4de..e3d655d5 100644
--- a/src/core.py
+++ b/src/core.py
@@ -144,6 +144,7 @@ class Core(object):
             'set_plugin': (self.command_set_plugin, _("Usage: /set_plugin <plugin> <option> [value]\nSet Plugin: Set the value of the option in a plugin configuration file."), self.completion_set_plugin),
             'invite': (self.command_invite, _("Usage: /invite <jid> <room> [reason]\nInvite: Invite jid in room with reason."), self.completion_invite),
             'decline': (self.command_decline, _("Usage: /decline <room> [reason]\nDecline: Decline the invitation to room with or without reason."), self.completion_decline),
+            'invitations': (self.command_invitations, _("Usage: /invites\nInvites: Show the pending invitations."), None),
             'bookmarks': (self.command_bookmarks, _("Usage: /bookmarks\nBookmarks: Show the current bookmarks."), None),
             'remove_bookmark': (self.command_remove_bookmark, _("Usage: /remove_bookmark [jid]\nRemove Bookmark: Remove the specified bookmark, or the bookmark on the current tab, if any."), self.completion_remove_bookmark),
             'xml_tab': (self.command_xml_tab, _("Usage: /xml_tab\nXML Tab: Open an XML tab."), None),
@@ -376,6 +377,7 @@ class Core(object):
         self.pending_invites[jid.bare] = inviter.full
 
     def command_invite(self, arg):
+        """/invite <to> <room> [reason]"""
         args = common.shell_split(arg)
         if len(args) < 2:
             return
@@ -400,6 +402,7 @@ class Core(object):
             return the_input.auto_completion(rooms, '')
 
     def command_decline(self, arg):
+        """/decline <room@server.tld> [reason]"""
         args = common.shell_split(arg)
         if not len(args):
             return
@@ -409,6 +412,17 @@ class Core(object):
         reason = args[1] if len(args) > 1 else ''
         self.xmpp.plugin['xep_0045'].decline_invite(jid.bare, self.pending_invites[jid.bare], reason)
 
+    def command_invitations(self, arg):
+        """/invitations"""
+        build = ""
+        for invite in self.pending_invites:
+            build += "%s by %s" % (invite, JID(self.pending_invites[invite]).bare)
+        if self.pending_invites:
+            build = "You are invited to the following rooms:\n" + build
+        else:
+            build = "You are do not have any pending invitation."
+        self.information(build, 'Info')
+
     def completion_decline(self, the_input):
         txt = the_input.get_text()
         args = common.shell_split(txt)
-- 
cgit v1.2.3


From 08b374cb297b65eecea5478109f7209bc5528a9b Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Sun, 18 Dec 2011 20:43:40 +0100
Subject: Fix the completion for /decline and /invite

---
 src/core.py | 25 ++++++++++++++-----------
 1 file changed, 14 insertions(+), 11 deletions(-)

(limited to 'src')

diff --git a/src/core.py b/src/core.py
index e3d655d5..883eb8fa 100644
--- a/src/core.py
+++ b/src/core.py
@@ -387,14 +387,15 @@ class Core(object):
         self.xmpp.plugin['xep_0045'].invite(room, to, reason)
 
     def completion_invite(self, the_input):
+        """Completion for /invite"""
         txt = the_input.get_text()
         args = common.shell_split(txt)
         n = len(args)
         if txt.endswith(' '):
             n += 1
-        if len(args) == 1:
+        if n == 2:
             return the_input.auto_completion([contact.bare_jid for contact in roster.get_contacts()], '')
-        elif len(args) == 2:
+        elif n == 3:
             rooms = []
             for tab in self.tabs:
                 if isinstance(tab, tabs.MucTab) and tab.joined:
@@ -410,8 +411,19 @@ class Core(object):
         if jid.bare not in self.pending_invites:
             return
         reason = args[1] if len(args) > 1 else ''
+        del self.pending_invites[jid.bare]
         self.xmpp.plugin['xep_0045'].decline_invite(jid.bare, self.pending_invites[jid.bare], reason)
 
+    def completion_decline(self, the_input):
+        """Completion for /decline"""
+        txt = the_input.get_text()
+        args = common.shell_split(txt)
+        n = len(args)
+        if txt.endswith(' '):
+            n += 1
+        if n == 2:
+            return the_input.auto_completion(list(self.pending_invites.keys()), '')
+
     def command_invitations(self, arg):
         """/invitations"""
         build = ""
@@ -423,15 +435,6 @@ class Core(object):
             build = "You are do not have any pending invitation."
         self.information(build, 'Info')
 
-    def completion_decline(self, the_input):
-        txt = the_input.get_text()
-        args = common.shell_split(txt)
-        n = len(args)
-        if txt.endswith(' '):
-            n += 1
-        if len(args) == 1:
-            return the_input.auto_completion(list(self.pending_invites.keys()), '')
-
     def on_groupchat_decline(self, decline):
         pass
 
-- 
cgit v1.2.3


From 4c5c3101a24ae36b026746937cfd606be9e8d989 Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Mon, 19 Dec 2011 17:19:18 +0100
Subject: Fix a little incoherence in /query

---
 src/tabs.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'src')

diff --git a/src/tabs.py b/src/tabs.py
index e696f46d..45860acc 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -758,7 +758,7 @@ class MucTab(ChatTab):
             if user.nick == nick:
                 r = self.core.open_private_window(self.name, user.nick)
         if r and len(args) > 1:
-            msg = arg[len(nick)+1:]
+            msg = args[1]
             self.core.current_tab().command_say(xhtml.convert_simple_to_full_colors(msg))
         if not r:
             self.core.information(_("Cannot find user: %s" % nick), 'Error')
-- 
cgit v1.2.3


From 3c4b2e9683180f244799caf661d10bb7d9213faf Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Wed, 28 Dec 2011 23:32:23 +0100
Subject: Fix various issues with the /configure tab Vertical tab list, text
 not erased from previous tab

---
 src/data_forms.py | 16 +++++++++++-----
 1 file changed, 11 insertions(+), 5 deletions(-)

(limited to 'src')

diff --git a/src/data_forms.py b/src/data_forms.py
index 2d17a304..d4a8697a 100644
--- a/src/data_forms.py
+++ b/src/data_forms.py
@@ -75,7 +75,7 @@ class DataFormsTab(Tab):
 
     def refresh(self):
         self.topic_win.refresh(self._form['title'])
-        self.tab_win.refresh()
+        self.refresh_tab_win()
         self.help_win.refresh()
         self.help_win_dyn.refresh(self.form_win.get_help_message())
         self.form_win.refresh()
@@ -88,7 +88,7 @@ class FieldInput(object):
     """
     def __init__(self, field):
         self._field = field
-        self.color = (14, -1)
+        self.color = get_theme().COLOR_NORMAL_TEXT
 
     def set_color(self, color):
         self.color = color
@@ -131,6 +131,7 @@ class ColoredLabel(windows.Win):
 
     def refresh(self):
         with g_lock:
+            self._win.erase()
             self._win.attron(to_curses_attr(self.color))
             self.addstr(0, 0, self.text)
             self._win.attroff(to_curses_attr(self.color))
@@ -157,7 +158,7 @@ class DummyInput(FieldInput, windows.Win):
 class ColoredLabel(windows.Win):
     def __init__(self, text):
         self.text = text
-        self.color = (14, -1)
+        self.color = get_theme().COLOR_NORMAL_TEXT
         windows.Win.__init__(self)
 
     def resize(self, height, width, y, x):
@@ -169,6 +170,7 @@ class ColoredLabel(windows.Win):
 
     def refresh(self):
         with g_lock:
+            self._win.erase()
             self._win.attron(to_curses_attr(self.color))
             self.addstr(0, 0, self.text)
             self._win.attroff(to_curses_attr(self.color))
@@ -189,6 +191,7 @@ class BooleanWin(FieldInput, windows.Win):
 
     def refresh(self):
         with g_lock:
+            self._win.erase()
             self._win.attron(to_curses_attr(self.color))
             self.addnstr(0, 0, ' '*(8), self.width)
             self.addstr(0, 2, "%s"%self.value)
@@ -253,6 +256,7 @@ class TextMultiWin(FieldInput, windows.Win):
     def refresh(self):
         if not self.edition_input:
             with g_lock:
+                self._win.erase()
                 self._win.attron(to_curses_attr(self.color))
                 self.addnstr(0, 0, ' '*self.width, self.width)
                 option = self.options[self.val_pos]
@@ -305,6 +309,7 @@ class ListMultiWin(FieldInput, windows.Win):
 
     def refresh(self):
         with g_lock:
+            self._win.erase()
             self._win.attron(to_curses_attr(self.color))
             self.addnstr(0, 0, ' '*self.width, self.width)
             if self.val_pos > 0:
@@ -351,6 +356,7 @@ class ListSingleWin(FieldInput, windows.Win):
 
     def refresh(self):
         with g_lock:
+            self._win.erase()
             self._win.attron(to_curses_attr(self.color))
             self.addnstr(0, 0, ' '*self.width, self.width)
             if self.val_pos > 0:
@@ -377,7 +383,7 @@ class TextSingleWin(FieldInput, windows.Input):
         self.text = field.getValue() if isinstance(field.getValue(), str)\
             else ""
         self.pos = len(self.text)
-        self.color = (14, -1)
+        self.color = get_theme().COLOR_NORMAL_TEXT
 
     def reply(self):
         self._field['label'] = ''
@@ -427,7 +433,7 @@ class FormWin(object):
                      }
     def __init__(self, form, height, width, y, x):
         self._form = form
-        self._win = curses.newwin(height, width, y, x)
+        self._win = windows.Win._tab_win.derwin(height, width, y, x)
         self.current_input = 0
         self.inputs = []        # dict list
         for (name, field) in self._form.getFields().items():
-- 
cgit v1.2.3


From 62c4f67ea53c6b1c7f4092349f4c746e5bdb14f1 Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Thu, 29 Dec 2011 18:55:43 +0100
Subject: Fix the resize issues with the /configure tab

---
 src/data_forms.py | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

(limited to 'src')

diff --git a/src/data_forms.py b/src/data_forms.py
index d4a8697a..d38f392a 100644
--- a/src/data_forms.py
+++ b/src/data_forms.py
@@ -67,13 +67,14 @@ class DataFormsTab(Tab):
     def resize(self):
         self.need_resize = False
         self.topic_win.resize(1, self.width, 0, 0)
-        self.tab_win.resize(1, self.width, self.height-2, 0)
-        self.form_win.resize(self.height-4, self.width, 1, 0)
+        self.form_win.resize(self.height-3 - Tab.tab_win_height(), self.width, 1, 0)
         self.help_win.resize(1, self.width, self.height-1, 0)
-        self.help_win_dyn.resize(1, self.width, self.height-3, 0)
+        self.help_win_dyn.resize(1, self.width, self.height-2 - Tab.tab_win_height(), 0)
         self.lines = []
 
     def refresh(self):
+        if self.need_resize:
+            self.resize()
         self.topic_win.refresh(self._form['title'])
         self.refresh_tab_win()
         self.help_win.refresh()
@@ -453,7 +454,6 @@ class FormWin(object):
                                 'input':inp})
 
     def resize(self, height, width, y, x):
-        self._win.resize(height, width)
         self.height = height
         self.width = width
         if self.current_input >= self.height-2:
-- 
cgit v1.2.3


From 5fee56440c10f81286699a4dd9e32506aa342d54 Mon Sep 17 00:00:00 2001
From: Florent Le Coz <louiz@louiz.org>
Date: Tue, 10 Jan 2012 16:30:59 +0100
Subject: Add a simple_notify plugin

And also fix some issues, like the split of the exec_command arguments.
And the doc for the plugin, too.
---
 src/core.py   | 5 ++++-
 src/daemon.py | 5 +++--
 src/tabs.py   | 3 +++
 3 files changed, 10 insertions(+), 3 deletions(-)

(limited to 'src')

diff --git a/src/core.py b/src/core.py
index 883eb8fa..1023c748 100644
--- a/src/core.py
+++ b/src/core.py
@@ -2164,7 +2164,10 @@ class Core(object):
                 self.remote_fifo = None
         else:
             e = Executor(command.strip())
-            e.start()
+            try:
+                e.start()
+            except ValueError as e: # whenever shlex fails
+                self.information('%s' % (e,), 'Error')
 
     def get_conversation_messages(self):
         """
diff --git a/src/daemon.py b/src/daemon.py
index b413f465..4b4c0b79 100755
--- a/src/daemon.py
+++ b/src/daemon.py
@@ -22,7 +22,7 @@ command on your local machine.
 import sys
 import threading
 import subprocess
-
+import shlex
 import logging
 
 log = logging.getLogger(__name__)
@@ -40,7 +40,8 @@ class Executor(threading.Thread):
 
     def run(self):
         log.info('executing %s' % (self.command.strip(),))
-        subprocess.call(self.command.split())
+        command = shlex.split(self.command)
+        subprocess.call(command)
 
 def main():
     while True:
diff --git a/src/tabs.py b/src/tabs.py
index 45860acc..8316d2a3 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -1318,6 +1318,7 @@ class MucTab(ChatTab):
         Note that user can be None even if nickname is not None. It happens
         when we receive an history message said by someone who is not
         in the room anymore
+        Return True if the message highlighted us. False otherwise.
         """
         self.log_message(txt, time, nickname)
         user = self.get_user_by_name(nickname) if nickname is not None else None
@@ -1331,6 +1332,7 @@ class MucTab(ChatTab):
             if self.state != 'highlight':
                 self.state = 'message'
         nick_color = nick_color or None
+        highlight = False
         if (not nickname or time) and not txt.startswith('/me '):
             txt = '\x195}%s' % (txt,)
         else:                   # TODO
@@ -1339,6 +1341,7 @@ class MucTab(ChatTab):
                 nick_color = highlight
         time = time or datetime.now()
         self._text_buffer.add_message(txt, time, nickname, nick_color, history, user)
+        return highlight
 
 class PrivateTab(ChatTab):
     """
-- 
cgit v1.2.3


From 92d96f3f36cebd0ffb39e703c08af369e70b78fc Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Wed, 11 Jan 2012 18:13:32 +0100
Subject: Fix the alias plugin

---
 src/common.py | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

(limited to 'src')

diff --git a/src/common.py b/src/common.py
index 2da7835b..26b5dd0f 100644
--- a/src/common.py
+++ b/src/common.py
@@ -235,13 +235,16 @@ def parse_secs_to_str(duration=0):
     result += '%ss' % secs if secs else ''
     return result
 
-def parse_command_args_to_alias(args, strto):
+def parse_command_args_to_alias(arg, strto):
     """
     Parse command parameters.
     Numbers can be from 0 to 9.
-    >>> parse_command_args_to_alias(['sdf', 'koin'], '%0 %1')
-    "sdf koin"
+    >>> parse_command_args_to_alias('sdf koin', '%1 %0')
+    "koin sdf"
     """
+    if '%' not in strto:
+        return strto + arg
+    args = shell_split(arg)
     l = len(args)
     dest = ''
     var_num = False
@@ -250,7 +253,7 @@ def parse_command_args_to_alias(args, strto):
             if not var_num:
                 dest += i
             elif i in string.digits:
-                if int(i) < l:
+                if 0 <= int(i) < l:
                     dest += args[int(i)]
                 var_num = False
         elif i == '%':
-- 
cgit v1.2.3


From 1f9a6f588347b5692c60d47966913bb101e479a2 Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Sun, 15 Jan 2012 16:40:49 +0100
Subject: Fixes #2319

---
 src/tabs.py | 4 ++++
 1 file changed, 4 insertions(+)

(limited to 'src')

diff --git a/src/tabs.py b/src/tabs.py
index 8316d2a3..e8a773fe 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -1419,6 +1419,8 @@ class PrivateTab(ChatTab):
                                                              res.get('version') or _('unknown'),
                                                              res.get('os') or _('on an unknown platform'))
             self.core.information(version, 'Info')
+        if arg:
+            return self.core.command_version(arg)
         jid = self.name
         self.core.xmpp.plugin['xep_0092'].get_version(jid, callback=callback)
 
@@ -2172,6 +2174,8 @@ class ConversationTab(ChatTab):
                                                              res.get('version') or _('unknown'),
                                                              res.get('os') or _('on an unknown platform'))
             self.core.information(version, 'Info')
+        if arg:
+            return self.core.command_version(arg)
         jid = self._name
         self.core.xmpp.plugin['xep_0092'].get_version(jid, callback=callback)
 
-- 
cgit v1.2.3


From d4ec0ba442a0676a1b2e3f781d8d27853751ca29 Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Sat, 21 Jan 2012 00:06:18 +0100
Subject: Do not quotify topic completion

---
 src/tabs.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'src')

diff --git a/src/tabs.py b/src/tabs.py
index e8a773fe..188b6389 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -814,7 +814,7 @@ class MucTab(ChatTab):
 
     def completion_topic(self, the_input):
         current_topic = self.topic
-        return the_input.auto_completion([current_topic], '')
+        return the_input.auto_completion([current_topic], '', quotify=False)
 
     def command_kick(self, arg):
         """
-- 
cgit v1.2.3


From 7425231f47536b2f70a87eefb36a39a8b25cd37f Mon Sep 17 00:00:00 2001
From: Florent Le Coz <louiz@louiz.org>
Date: Tue, 24 Jan 2012 14:49:03 +0100
Subject: Display colors for /me messages in a better way.

---
 src/text_buffer.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'src')

diff --git a/src/text_buffer.py b/src/text_buffer.py
index 3541b9c1..f396bd5b 100644
--- a/src/text_buffer.py
+++ b/src/text_buffer.py
@@ -44,7 +44,7 @@ class TextBuffer(object):
             else:
                 color = None
             # TODO: display the bg color too.
-            txt = ("\x19%s}* \x195}" % (color or 5,))+ nickname + ' ' + txt[4:]
+            txt = ("\x195}* \x19%s}" % (color or 5,))+ nickname + ' \x195}' + txt[4:]
             nickname = None
         msg = Message(txt='%s\x19o'%(txt.replace('\t', '    '),), nick_color=nick_color,
                       time=time, str_time=time.strftime("%Y-%m-%d %H:%M:%S")\
-- 
cgit v1.2.3


From 09f16804a3cec50316129a2a3cc0e64628b32fce Mon Sep 17 00:00:00 2001
From: Florent Le Coz <louiz@louiz.org>
Date: Tue, 24 Jan 2012 15:17:54 +0100
Subject: Fix the comments in the theming.py file

---
 src/theming.py | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

(limited to 'src')

diff --git a/src/theming.py b/src/theming.py
index 7e90a5a7..4e1f37ea 100644
--- a/src/theming.py
+++ b/src/theming.py
@@ -11,16 +11,17 @@ used when drawing the interface.
 
 Colors are numbers from -1 to 7 (if only 8 colors are supported) or -1 to 255
 if 256 colors are available.
-We check the number of available colors at startup, and we load a theme accordingly.
-A 8 color theme should NEVER use colors not in the -1 -> 7 range. We won't check that
-at run time. If the case occurs, the THEME should be fixed.
+If only 8 colors are available, all colors > 8 are converted using the
+table_256_to_16 dict.
+
 XHTML-IM colors are converted to -1 -> 255 colors if available, or directly to
 -1 -> 8 if we are in 8-color-mode.
 
 A pair_color is a background-foreground pair. All possible pairs are not created
 at startup, because that would create 256*256 pairs, and almost all of them
 would never be used.
-So, a theme should define color tuples, like (200, -1), and when they are to
+
+A theme should define color tuples, like (200, -1), and when they are to
 be used by poezio's interface, they will be created once, and kept in a list for
 later usage.
 A color tuple is of the form (foreground, background, optional)
-- 
cgit v1.2.3


From 1f7db8cfaa90f3fc628181e6379779f7adcb9f53 Mon Sep 17 00:00:00 2001
From: Florent Le Coz <louiz@louiz.org>
Date: Tue, 24 Jan 2012 15:18:08 +0100
Subject: Remove all the .replace('"', '\\"') which are useless since the new
 way to express colors in messages.

---
 src/tabs.py | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

(limited to 'src')

diff --git a/src/tabs.py b/src/tabs.py
index 188b6389..fa99fc05 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -1142,7 +1142,7 @@ class MucTab(ChatTab):
             if by:
                 kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x195} has been banned by \x194}%(by)s') % {'spec':get_theme().CHAR_KICK, 'nick':from_nick, 'color':color, 'by':by}
             else:
-                kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x195} has been banned') % {'spec':get_theme().CHAR_KICK, 'nick':from_nick.replace('"', '\\"'), 'color':color}
+                kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x195} has been banned') % {'spec':get_theme().CHAR_KICK, 'nick':from_nick, 'color':color}
         if reason is not None and reason.text:
             kick_msg += _('\x195} Reason: \x196}%(reason)s\x195}') % {'reason': reason.text}
         self._text_buffer.add_message(kick_msg)
@@ -1170,9 +1170,9 @@ class MucTab(ChatTab):
         else:
             color = user.color[0] if config.get_by_tabname('display_user_color_in_join_part', '', self.general_jid, True) == 'true' else 3
             if by:
-                kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x195} has been kicked by \x193}%(by)s') % {'spec':get_theme().CHAR_KICK.replace('"', '\\"'), 'nick':from_nick.replace('"', '\\"'), 'color':color, 'by':by.replace('"', '\\"')}
+                kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x195} has been kicked by \x193}%(by)s') % {'spec':get_theme().CHAR_KICK, 'nick':from_nick, 'color':color, 'by':by}
             else:
-                kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x195} has been kicked') % {'spec':get_theme().CHAR_KICK, 'nick':from_nick.replace('"', '\\"'), 'color':color}
+                kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x195} has been kicked') % {'spec':get_theme().CHAR_KICK, 'nick':from_nick, 'color':color}
         if reason is not None and reason.text:
             kick_msg += _('\x195} Reason: \x196}%(reason)s') % {'reason': reason.text}
         self.add_message(kick_msg)
@@ -1212,7 +1212,7 @@ class MucTab(ChatTab):
         if from_nick == self.own_nick:
             msg = _('\x193}You\x195} changed: ')
         else:
-            msg = _('\x19%(color)d}%(nick)s\x195} changed: ') % {'nick': from_nick.replace('"', '\\"'), 'color': color}
+            msg = _('\x19%(color)d}%(nick)s\x195} changed: ') % {'nick': from_nick, 'color': color}
         if show not in SHOW_NAME:
             self.core.information("%s from room %s sent an invalid show: %s" %\
                                       (from_nick, from_room, show), "warning")
@@ -1518,9 +1518,9 @@ class PrivateTab(ChatTab):
         """
         self.deactivate()
         if not status_message:
-            self.add_message(_('\x191}%(spec)s \x193}%(nick)s\x195} has left the room') % {'nick':from_nick.replace('"', '\\"'), 'spec':get_theme().CHAR_QUIT.replace('"', '\\"')})
+            self.add_message(_('\x191}%(spec)s \x193}%(nick)s\x195} has left the room') % {'nick':from_nick, 'spec':get_theme().CHAR_QUIT})
         else:
-            self.add_message(_('\x191}%(spec)s \x193}%(nick)s\x195} has left the room (%(status)s)"') % {'nick':from_nick.replace('"', '\\"'), 'spec':get_theme().CHAR_QUIT, 'status': status_message.replace('"', '\\"')})
+            self.add_message(_('\x191}%(spec)s \x193}%(nick)s\x195} has left the room (%(status)s)"') % {'nick':from_nick, 'spec':get_theme().CHAR_QUIT, 'status': status_message})
         if self.core.current_tab() is self:
             self.refresh()
             self.core.doupdate()
-- 
cgit v1.2.3


From 99a16f2b868a093e390d99154ad67251ab9f3511 Mon Sep 17 00:00:00 2001
From: Florent Le Coz <louiz@louiz.org>
Date: Tue, 24 Jan 2012 15:46:51 +0100
Subject: Make the color for information messages themable

---
 src/tabs.py        | 50 +++++++++++++++++++++++++-------------------------
 src/text_buffer.py |  3 ++-
 src/theming.py     |  2 +-
 3 files changed, 28 insertions(+), 27 deletions(-)

(limited to 'src')

diff --git a/src/tabs.py b/src/tabs.py
index fa99fc05..53405db4 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -1058,9 +1058,9 @@ class MucTab(ChatTab):
                     if self.core.current_tab() == self and self.core.status.show not in ('xa', 'away'):
                         self.send_chat_state('active')
                     new_user.color = get_theme().COLOR_OWN_NICK
-                    self.add_message(_("\x195}Your nickname is \x193}%s") % (from_nick))
+                    self.add_message(_("\x19%(info_col)s}Your nickname is \x193}%(nick)s") % {'nick': from_nick, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]})
                     if '170' in status_codes:
-                        self.add_message('\x191}Warning: \x195}this room is publicly logged')
+                        self.add_message('\x191}Warning: \x19%(info_col)s}this room is publicly logged' % {'info_col': get_theme().COLOR_INFORMATION_TEXT[0]})
         else:
             change_nick = '303' in status_codes
             kick = '307' in status_codes and typ == 'unavailable'
@@ -1101,9 +1101,9 @@ class MucTab(ChatTab):
         if hide_exit_join != 0:
             color = user.color[0] if config.get_by_tabname('display_user_color_in_join_part', '', self.general_jid, True) == 'true' else 3
             if not jid.full:
-                self.add_message('\x194}%(spec)s \x19%(color)d}%(nick)s\x195} joined the room' % {'nick':from_nick, 'color':color, 'spec':get_theme().CHAR_JOIN})
+                self.add_message('\x194}%(spec)s \x19%(color)d}%(nick)s\x19%(info_col)s} joined the room' % {'nick':from_nick, 'color':color, 'spec':get_theme().CHAR_JOIN, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]})
             else:
-                self.add_message('\x194}%(spec)s \x19%(color)d}%(nick)s \x195}(\x194}%(jid)s\x195}) joined the room' % {'spec':get_theme().CHAR_JOIN, 'nick':from_nick, 'color':color, 'jid':jid.full})
+                self.add_message('\x194}%(spec)s \x19%(color)d}%(nick)s \x19%(info_col)s}(\x194}%(jid)s\x19%(info_col)s}) joined the room' % {'spec':get_theme().CHAR_JOIN, 'nick':from_nick, 'color':color, 'jid':jid.full, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]})
         self.core.on_user_rejoined_private_conversation(self.name, from_nick)
 
     def on_user_nick_change(self, presence, user, from_nick, from_room):
@@ -1116,7 +1116,7 @@ class MucTab(ChatTab):
                     _tab.own_nick = new_nick
         user.change_nick(new_nick)
         color = user.color[0] if config.get_by_tabname('display_user_color_in_join_part', '', self.general_jid, True) == 'true' else 3
-        self.add_message('\x19%(color)d}%(old)s\x195} is now known as \x19%(color)d}%(new)s' % {'old':from_nick, 'new':new_nick, 'color':color})
+        self.add_message('\x19%(color)d}%(old)s\x19%(info_col)s} is now known as \x19%(color)d}%(new)s' % {'old':from_nick, 'new':new_nick, 'color':color, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]})
         # rename the private tabs if needed
         self.core.rename_private_tabs(self.name, from_nick, new_nick)
 
@@ -1134,17 +1134,17 @@ class MucTab(ChatTab):
             self.refresh_tab_win()
             self.core.doupdate()
             if by:
-                kick_msg = _('\x191}%(spec)s \x193}You\x195} have been banned by \x194}%(by)s') % {'spec': get_theme().CHAR_KICK, 'by':by}
+                kick_msg = _('\x191}%(spec)s \x193}You\x19%(info_col)s} have been banned by \x194}%(by)s') % {'spec': get_theme().CHAR_KICK, 'by':by, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}
             else:
-                kick_msg = _('\x191}%(spec)s \x193}You\x195} have been banned.') % {'spec':get_theme().CHAR_KICK}
+                kick_msg = _('\x191}%(spec)s \x193}You\x19%(info_col)s} have been banned.') % {'spec':get_theme().CHAR_KICK, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}
         else:
             color = user.color[0] if config.get_by_tabname('display_user_color_in_join_part', '', self.general_jid, True) == 'true' else 3
             if by:
-                kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x195} has been banned by \x194}%(by)s') % {'spec':get_theme().CHAR_KICK, 'nick':from_nick, 'color':color, 'by':by}
+                kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x19%(info_col)s} has been banned by \x194}%(by)s') % {'spec':get_theme().CHAR_KICK, 'nick':from_nick, 'color':color, 'by':by, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}
             else:
-                kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x195} has been banned') % {'spec':get_theme().CHAR_KICK, 'nick':from_nick, 'color':color}
+                kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x19%(info_col)s} has been banned') % {'spec':get_theme().CHAR_KICK, 'nick':from_nick, 'color':color, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}
         if reason is not None and reason.text:
-            kick_msg += _('\x195} Reason: \x196}%(reason)s\x195}') % {'reason': reason.text}
+            kick_msg += _('\x19%(info_col)s} Reason: \x196}%(reason)s\x19%(info_col)s}') % {'reason': reason.text, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}
         self._text_buffer.add_message(kick_msg)
 
     def on_user_kicked(self, presence, user, from_nick):
@@ -1161,20 +1161,20 @@ class MucTab(ChatTab):
             self.refresh_tab_win()
             self.core.doupdate()
             if by:
-                kick_msg = _('\x191}%(spec)s \x193}You\x195} have been kicked by \x193}%(by)s') % {'spec': get_theme().CHAR_KICK, 'by':by}
+                kick_msg = _('\x191}%(spec)s \x193}You\x19%(info_col)s} have been kicked by \x193}%(by)s') % {'spec': get_theme().CHAR_KICK, 'by':by, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}
             else:
-                kick_msg = _('\x191}%(spec)s \x193}You\x195} have been kicked.') % {'spec':get_theme().CHAR_KICK}
+                kick_msg = _('\x191}%(spec)s \x193}You\x19%(info_col)s} have been kicked.') % {'spec':get_theme().CHAR_KICK, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}
             # try to auto-rejoin
             if config.get_by_tabname('autorejoin', 'false', self.general_jid, True) == 'true':
                 muc.join_groupchat(self.core.xmpp, self.name, self.own_nick)
         else:
             color = user.color[0] if config.get_by_tabname('display_user_color_in_join_part', '', self.general_jid, True) == 'true' else 3
             if by:
-                kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x195} has been kicked by \x193}%(by)s') % {'spec':get_theme().CHAR_KICK, 'nick':from_nick, 'color':color, 'by':by}
+                kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x19%(info_col)s} has been kicked by \x193}%(by)s') % {'spec':get_theme().CHAR_KICK, 'nick':from_nick, 'color':color, 'by':by, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}
             else:
-                kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x195} has been kicked') % {'spec':get_theme().CHAR_KICK, 'nick':from_nick, 'color':color}
+                kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x19%(info_col)s} has been kicked') % {'spec':get_theme().CHAR_KICK, 'nick':from_nick, 'color':color, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}
         if reason is not None and reason.text:
-            kick_msg += _('\x195} Reason: \x196}%(reason)s') % {'reason': reason.text}
+            kick_msg += _('\x19%(info_col)s} Reason: \x196}%(reason)s') % {'reason': reason.text, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}
         self.add_message(kick_msg)
 
     def on_user_leave_groupchat(self, user, jid, status, from_nick, from_room):
@@ -1192,9 +1192,9 @@ class MucTab(ChatTab):
         if hide_exit_join == -1 or user.has_talked_since(hide_exit_join):
             color = user.color[0] if config.get_by_tabname('display_user_color_in_join_part', '', self.general_jid, True) == 'true' else 3
             if not jid.full:
-                leave_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x195} has left the room') % {'nick':from_nick, 'color':color, 'spec':get_theme().CHAR_QUIT}
+                leave_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x19%(info_col)s} has left the room') % {'nick':from_nick, 'color':color, 'spec':get_theme().CHAR_QUIT, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}
             else:
-                leave_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x195} (\x194}%(jid)s\x195}) has left the room') % {'spec':get_theme().CHAR_QUIT, 'nick':from_nick, 'color':color, 'jid':jid.full}
+                leave_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x19%(info_col)s} (\x194}%(jid)s\x19%(info_col)s}) has left the room') % {'spec':get_theme().CHAR_QUIT, 'nick':from_nick, 'color':color, 'jid':jid.full, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}
             if status:
                 leave_msg += ' (%s)' % status
             self.add_message(leave_msg)
@@ -1210,9 +1210,9 @@ class MucTab(ChatTab):
         # to be displayed has changed
         color = user.color[0] if config.get_by_tabname('display_user_color_in_join_part', '', self.general_jid, True) == 'true' else 3
         if from_nick == self.own_nick:
-            msg = _('\x193}You\x195} changed: ')
+            msg = _('\x193}You\x19%(info_col)s} changed: ') % {'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}
         else:
-            msg = _('\x19%(color)d}%(nick)s\x195} changed: ') % {'nick': from_nick, 'color': color}
+            msg = _('\x19%(color)d}%(nick)s\x19%(info_col)s} changed: ') % {'nick': from_nick, 'color': color, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}
         if show not in SHOW_NAME:
             self.core.information("%s from room %s sent an invalid show: %s" %\
                                       (from_nick, from_room, show), "warning")
@@ -1334,7 +1334,7 @@ class MucTab(ChatTab):
         nick_color = nick_color or None
         highlight = False
         if (not nickname or time) and not txt.startswith('/me '):
-            txt = '\x195}%s' % (txt,)
+            txt = '\x19%(info_col)s}%(txt)s' % {'txt':txt, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}
         else:                   # TODO
             highlight = self.do_highlight(txt, time, nickname)
             if highlight:
@@ -1508,7 +1508,7 @@ class PrivateTab(ChatTab):
         The user changed her nick in the corresponding muc: update the tab’s name and
         display a message.
         """
-        self.add_message('\x193}%(old)s\x195} is now known as \x193}%(new)s' % {'old':old_nick, 'new':new_nick})
+        self.add_message('\x193}%(old)s\x19%(info_col)s} is now known as \x193}%(new)s' % {'old':old_nick, 'new':new_nick, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]})
         new_jid = JID(self.name).bare+'/'+new_nick
         self.name = new_jid
 
@@ -1518,9 +1518,9 @@ class PrivateTab(ChatTab):
         """
         self.deactivate()
         if not status_message:
-            self.add_message(_('\x191}%(spec)s \x193}%(nick)s\x195} has left the room') % {'nick':from_nick, 'spec':get_theme().CHAR_QUIT})
+            self.add_message(_('\x191}%(spec)s \x193}%(nick)s\x19%(info_col)s} has left the room') % {'nick':from_nick, 'spec':get_theme().CHAR_QUIT, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]})
         else:
-            self.add_message(_('\x191}%(spec)s \x193}%(nick)s\x195} has left the room (%(status)s)"') % {'nick':from_nick, 'spec':get_theme().CHAR_QUIT, 'status': status_message})
+            self.add_message(_('\x191}%(spec)s \x193}%(nick)s\x19%(info_col)s} has left the room (%(status)s)"') % {'nick':from_nick, 'spec':get_theme().CHAR_QUIT, 'status': status_message, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]})
         if self.core.current_tab() is self:
             self.refresh()
             self.core.doupdate()
@@ -1536,7 +1536,7 @@ class PrivateTab(ChatTab):
             user = tab.get_user_by_name(nick)
             if user:
                 color = user.color[0]
-        self.add_message('\x194}%(spec)s \x19%(color)d}%(nick)s\x195} joined the room' % {'nick':nick, 'color': color, 'spec':get_theme().CHAR_JOIN})
+        self.add_message('\x194}%(spec)s \x19%(color)d}%(nick)s\x19%(info_col)s} joined the room' % {'nick':nick, 'color': color, 'spec':get_theme().CHAR_JOIN, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]})
         if self.core.current_tab() is self:
             self.refresh()
             self.core.doupdate()
@@ -2155,7 +2155,7 @@ class ConversationTab(ChatTab):
         else:
             resource = contact.get_highest_priority_resource()
         if resource:
-            self._text_buffer.add_message("\x195}Status: %s\x193}" %resource.status, None, None, None, None, None)
+            self._text_buffer.add_message("\x19%(info_col)s}Status: %(status)s\x193}" % {'status': resource.status, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}, None, None, None, None, None)
             self.refresh()
             self.core.doupdate()
 
diff --git a/src/text_buffer.py b/src/text_buffer.py
index f396bd5b..9b717882 100644
--- a/src/text_buffer.py
+++ b/src/text_buffer.py
@@ -16,6 +16,7 @@ import collections
 
 from datetime import datetime
 from config import config
+from theming import get_theme
 
 Message = collections.namedtuple('Message', 'txt nick_color time str_time nickname user')
 
@@ -44,7 +45,7 @@ class TextBuffer(object):
             else:
                 color = None
             # TODO: display the bg color too.
-            txt = ("\x195}* \x19%s}" % (color or 5,))+ nickname + ' \x195}' + txt[4:]
+            txt = ("\x19%(info_col)s}* \x19%(col)s}" % {'col':color or 5, 'info_col':get_theme().COLOR_INFORMATION_TEXT[0]})+ nickname + ' \x19%(info_col)s}' % {'info_col':get_theme().COLOR_INFORMATION_TEXT[0]} + txt[4:]
             nickname = None
         msg = Message(txt='%s\x19o'%(txt.replace('\t', '    '),), nick_color=nick_color,
                       time=time, str_time=time.strftime("%Y-%m-%d %H:%M:%S")\
diff --git a/src/theming.py b/src/theming.py
index 4e1f37ea..ff2ff74e 100644
--- a/src/theming.py
+++ b/src/theming.py
@@ -79,7 +79,7 @@ class Theme(object):
     """
     # Message text color
     COLOR_NORMAL_TEXT = (-1, -1)
-    COLOR_INFORMATION_TEXT = (137, -1) # TODO
+    COLOR_INFORMATION_TEXT = (5, -1) # TODO
     COLOR_HIGHLIGHT_NICK = (3, 5, 'b')
 
     # User list color
-- 
cgit v1.2.3


From 561af013b1432fbb9b52d7f246be487347991687 Mon Sep 17 00:00:00 2001
From: Florent Le Coz <louiz@louiz.org>
Date: Tue, 24 Jan 2012 17:07:04 +0100
Subject: New participant list. Displaying the nick color, the affiliation,
 role, chatstate and status!

---
 src/tabs.py    |  1 +
 src/windows.py | 49 ++++++++++++++++++++++++++++++-------------------
 2 files changed, 31 insertions(+), 19 deletions(-)

(limited to 'src')

diff --git a/src/tabs.py b/src/tabs.py
index 53405db4..019fae30 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -678,6 +678,7 @@ class MucTab(ChatTab):
         for i, user in enumerate(sorted_users):
             user.color = colors[i % len(colors)]
         self.text_win.rebuild_everything(self._text_buffer)
+        self.user_win.refresh(self.users)
         self.text_win.refresh()
         self.input.refresh()
 
diff --git a/src/windows.py b/src/windows.py
index 1b833290..7d9194b2 100644
--- a/src/windows.py
+++ b/src/windows.py
@@ -196,6 +196,10 @@ class UserList(Win):
                 'none': lambda: get_theme().COLOR_USER_NONE,
                 '': lambda: get_theme().COLOR_USER_NONE
                }
+        self.symbol_affiliation = {'owner': '~',
+                                    'admin': '&',
+                                    'member': '+',
+                                    'none': '-'}
         self.color_show = {'xa': lambda: get_theme().COLOR_STATUS_XA,
                 'none': lambda: get_theme().COLOR_STATUS_NONE,
                 '': lambda: get_theme().COLOR_STATUS_NONE,
@@ -203,7 +207,6 @@ class UserList(Win):
                 'away': lambda: get_theme().COLOR_STATUS_AWAY,
                 'chat': lambda: get_theme().COLOR_STATUS_CHAT
                }
-
     def scroll_up(self):
         self.pos += self.height-1
 
@@ -224,24 +227,9 @@ class UserList(Win):
             if self.pos >= len(users) and self.pos != 0:
                 self.pos = len(users)-1
             for user in users[self.pos:]:
-                if not user.role in self.color_role:
-                    role_col = get_theme().COLOR_USER_NONE
-                else:
-                    role_col = self.color_role[user.role]()
-                if not user.show in self.color_show:
-                    show_col = get_theme().COLOR_STATUS_NONE
-                else:
-                    show_col = self.color_show[user.show]()
-                if user.chatstate == 'composing':
-                    char = get_theme().CHAR_CHATSTATE_COMPOSING
-                elif user.chatstate == 'active':
-                    char = get_theme().CHAR_CHATSTATE_ACTIVE
-                elif user.chatstate == 'paused':
-                    char = get_theme().CHAR_CHATSTATE_PAUSED
-                else:
-                    char = get_theme().CHAR_STATUS
-                self.addstr(y, 0, char, to_curses_attr(show_col))
-                self.addstr(y, 1, user.nick[:self.width-2], to_curses_attr(role_col))
+                self.draw_role_affiliation(y, user)
+                self.draw_status_chatstate(y, user)
+                self.addstr(y, 2, user.nick[:self.width-2], to_curses_attr(user.color))
                 y += 1
                 if y == self.height:
                     break
@@ -252,6 +240,29 @@ class UserList(Win):
                 self.draw_plus(self.height-1)
             self._refresh()
 
+    def draw_role_affiliation(self, y, user):
+        if not user.role in self.color_role:
+            color = get_theme().COLOR_USER_NONE
+        else:
+            color = self.color_role[user.role]()
+        symbol = self.symbol_affiliation.get(user.affiliation) or '-'
+        self.addstr(y, 1, symbol, to_curses_attr(color))
+
+    def draw_status_chatstate(self, y, user):
+        if not user.show in self.color_show:
+            show_col = get_theme().COLOR_STATUS_NONE
+        else:
+            show_col = self.color_show[user.show]()
+        if user.chatstate == 'composing':
+            char = get_theme().CHAR_CHATSTATE_COMPOSING
+        elif user.chatstate == 'active':
+            char = get_theme().CHAR_CHATSTATE_ACTIVE
+        elif user.chatstate == 'paused':
+            char = get_theme().CHAR_CHATSTATE_PAUSED
+        else:
+            char = get_theme().CHAR_STATUS
+        self.addstr(y, 0, char, to_curses_attr(show_col))
+
     def resize(self, height, width, y, x):
         with g_lock:
             self._resize(height, width, y, x)
-- 
cgit v1.2.3


From 7ba606136293aa9b08cb1185e046cfdff7bf1589 Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Wed, 25 Jan 2012 17:51:11 +0100
Subject: Add affiliation chars in the theme

---
 src/theming.py | 11 +++++++++--
 src/windows.py | 10 +++++-----
 2 files changed, 14 insertions(+), 7 deletions(-)

(limited to 'src')

diff --git a/src/theming.py b/src/theming.py
index ff2ff74e..815b059f 100644
--- a/src/theming.py
+++ b/src/theming.py
@@ -101,6 +101,13 @@ class Theme(object):
     CHAR_CHATSTATE_COMPOSING = 'X'
     CHAR_CHATSTATE_PAUSED = 'p'
 
+    # These characters are used for the affiliation in the user list
+    # in a MUC
+    CHAR_AFFILIATION_OWNER = '~'
+    CHAR_AFFILIATION_ADMIN = '&'
+    CHAR_AFFILIATION_MEMBER = '+'
+    CHAR_AFFILIATION_NONE = '-'
+
     # Separators
     COLOR_VERTICAL_SEPARATOR = (4, -1)
     COLOR_NEW_TEXT_SEPARATOR = (2, -1)
@@ -162,8 +169,8 @@ class Theme(object):
     CHAR_JOIN = '--->'
     CHAR_QUIT = '<---'
     CHAR_KICK = '-!-'
-    CHAR_COLUMN_ASC = ' ▲' 
-    CHAR_COLUMN_DESC =' ▼' 
+    CHAR_COLUMN_ASC = ' ▲'
+    CHAR_COLUMN_DESC =' ▼'
 
     COLOR_JOIN_CHAR = (4, -1)
     COLOR_QUIT_CHAR = (1, -1)
diff --git a/src/windows.py b/src/windows.py
index 7d9194b2..89630a12 100644
--- a/src/windows.py
+++ b/src/windows.py
@@ -196,10 +196,10 @@ class UserList(Win):
                 'none': lambda: get_theme().COLOR_USER_NONE,
                 '': lambda: get_theme().COLOR_USER_NONE
                }
-        self.symbol_affiliation = {'owner': '~',
-                                    'admin': '&',
-                                    'member': '+',
-                                    'none': '-'}
+        self.symbol_affiliation = {'owner': lambda: get_theme().CHAR_AFFILIATION_OWNER,
+                'admin': lambda: get_theme().CHAR_AFFILIATION_ADMIN,
+                'member': lambda: get_theme().CHAR_AFFILIATION_MEMBER,
+                'none': lambda: get_theme().CHAR_AFFILIATION_NONE, }
         self.color_show = {'xa': lambda: get_theme().COLOR_STATUS_XA,
                 'none': lambda: get_theme().COLOR_STATUS_NONE,
                 '': lambda: get_theme().COLOR_STATUS_NONE,
@@ -245,7 +245,7 @@ class UserList(Win):
             color = get_theme().COLOR_USER_NONE
         else:
             color = self.color_role[user.role]()
-        symbol = self.symbol_affiliation.get(user.affiliation) or '-'
+        symbol = self.symbol_affiliation.get(user.affiliation, lambda: '-')()
         self.addstr(y, 1, symbol, to_curses_attr(color))
 
     def draw_status_chatstate(self, y, user):
-- 
cgit v1.2.3


From 17fdd5d60618ebc1e84a966b89e57c43fc6dd53a Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Wed, 25 Jan 2012 18:14:07 +0100
Subject: Fixes #2316

---
 src/tabs.py    | 7 ++++++-
 src/windows.py | 2 --
 2 files changed, 6 insertions(+), 3 deletions(-)

(limited to 'src')

diff --git a/src/tabs.py b/src/tabs.py
index 019fae30..374cdc4f 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -2371,7 +2371,12 @@ class MucListTab(Tab):
                   'name': item[2] or '' ,'users': ''} for item in iq['disco_items'].get_items()]
         self.listview.add_lines(items)
         self.upper_message.set_message('Chatroom list on server %s' % self.name)
-        self.upper_message.refresh()
+        if self.core.current_tab() is self:
+            self.listview.refresh()
+            self.upper_message.refresh()
+        else:
+            self.state = 'highlight'
+            self.refresh_tab_win()
         curses.doupdate()
 
     def sort_by(self):
diff --git a/src/windows.py b/src/windows.py
index 89630a12..3a852ebc 100644
--- a/src/windows.py
+++ b/src/windows.py
@@ -1667,8 +1667,6 @@ class ListWin(Win):
         if not lines:
             return
         self.lines += lines
-        self.refresh()
-        curses.doupdate()
 
     def get_selected_row(self):
         """
-- 
cgit v1.2.3


From 8e2e1fcd4e59f67aed8cd248f3202fc10971cbde Mon Sep 17 00:00:00 2001
From: Florent Le Coz <louiz@louiz.org>
Date: Mon, 28 Nov 2011 22:32:44 +0100
Subject: We can now configure each conversation independently, for some
 options.

Fixed #2039.
---
 src/config.py | 18 ++++++++++++++++-
 src/core.py   |  9 ++++++---
 src/tabs.py   | 64 ++++++++++++++++++++++++++++++++++++-----------------------
 3 files changed, 62 insertions(+), 29 deletions(-)

(limited to 'src')

diff --git a/src/config.py b/src/config.py
index 8bc0d863..af9c9fbe 100644
--- a/src/config.py
+++ b/src/config.py
@@ -46,10 +46,26 @@ class Config(RawConfigParser):
                 res = self.getboolean(option, section)
             else:
                 res = self.getstr(option, section)
-        except( NoOptionError, NoSectionError):
+        except (NoOptionError, NoSectionError):
             return default
         return res
 
+    def get_by_tabname(self, option, default, tabname, fallback=True):
+        """
+        Try to get the value for the option. First we look in
+        a section named `tabname`, if the option is not present
+        in the section, we search for the global option if fallback is
+        True. And we return `default` as a fallback as a last resort.
+        """
+        if tabname in self.sections():
+            if option in self.options(tabname):
+                # We go the tab-specific option
+                return self.get(option, default, tabname)
+        if fallback:
+            # We fallback to the global option
+            return self.get(option, default)
+        return default
+
     def __get(self, option, section=DEFSECTION):
         """
         facility for RawConfigParser.get
diff --git a/src/core.py b/src/core.py
index 8704563c..b940bb73 100644
--- a/src/core.py
+++ b/src/core.py
@@ -687,7 +687,8 @@ class Core(object):
             else:
                 conversation.remote_wants_chatstates = False
         if 'private' in config.get('beep_on', 'highlight private').split():
-            curses.beep()
+            if config.get_by_tabname('disable_beep', 'false', jid.full, False).lower() != 'true':
+                curses.beep()
         logger.log_message(jid.full.replace('/', '\\'), nick_from, body)
         if conversation is self.current_tab():
             self.refresh_window()
@@ -745,7 +746,8 @@ class Core(object):
                 conversation.remote_wants_chatstates = False
         logger.log_message(jid.bare, remote_nick, body)
         if 'private' in config.get('beep_on', 'highlight private').split():
-            curses.beep()
+            if config.get_by_tabname('disable_beep', 'false', jid.bare, False).lower() != 'true':
+                curses.beep()
         if self.current_tab() is not conversation:
             conversation.state = 'private'
             self.refresh_tab_win()
@@ -1207,7 +1209,8 @@ class Core(object):
                 tab.info_header.refresh(tab, tab.text_win)
             self.refresh_tab_win()
             if 'message' in config.get('beep_on', 'highlight private').split():
-                curses.beep()
+                if config.get_by_tabname('disable_beep', 'false', jid.bare, False).lower() != 'true':
+                    curses.beep()
 
     def add_message_to_text_buffer(self, buff, txt, time=None, nickname=None, history=None):
         """
diff --git a/src/tabs.py b/src/tabs.py
index f9940d82..e75989a7 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -440,7 +440,8 @@ class ChatTab(Tab):
         Send the "active" or "composing" chatstate, depending
         on the the current status of the input
         """
-        if config.get('send_chat_states', 'true') == 'true' and self.remote_wants_chatstates:
+        name = self.general_jid
+        if config.get_by_tabname('send_chat_states', 'true', name, True) == 'true' and self.remote_wants_chatstates:
             needed = 'inactive' if self.core.status.show in ('xa', 'away') else 'active'
             self.cancel_paused_delay()
             if not empty_after:
@@ -455,7 +456,7 @@ class ChatTab(Tab):
         we create a timed event that will put us to paused
         in a few seconds
         """
-        if config.get('send_chat_states', 'true') != 'true':
+        if config.get_by_tabname('send_chat_states', 'true', self.general_jid, True) != 'true':
             return
         if self.timed_event_paused:
             # check the weakref
@@ -546,6 +547,10 @@ class MucTab(ChatTab):
         self.update_commands()
         self.update_keys()
 
+    @property
+    def general_jid(self):
+        return self.get_name()
+
     def completion_version(self, the_input):
         """Completion for /version"""
         compare_users = lambda x: x.last_talked
@@ -880,7 +885,7 @@ class MucTab(ChatTab):
         if msg['body'].find('\x19') != -1:
             msg['xhtml_im'] = xhtml.poezio_colors_to_html(msg['body'])
             msg['body'] = xhtml.clean_text(msg['body'])
-        if config.get('send_chat_states', 'true') == 'true' and self.remote_wants_chatstates is not False:
+        if config.get_by_tabname('send_chat_states', 'true', self.general_jid, True) == 'true' and self.remote_wants_chatstates is not False:
             msg['chat_state'] = needed
         self.cancel_paused_delay()
         self.core.events.trigger('muc_say_after', msg, self)
@@ -1002,7 +1007,7 @@ class MucTab(ChatTab):
         self.state = 'normal'
         self.text_win.remove_line_separator()
         self.text_win.add_line_separator()
-        if config.get('send_chat_states', 'true') == 'true' and not self.input.get_text():
+        if config.get_by_tabname('send_chat_states', 'true', self.general_jid, True) == 'true' and not self.input.get_text():
             self.send_chat_state('inactive')
 
     def on_gain_focus(self):
@@ -1010,7 +1015,7 @@ class MucTab(ChatTab):
         if self.text_win.built_lines and self.text_win.built_lines[-1] is None:
             self.text_win.remove_line_separator()
         curses.curs_set(1)
-        if self.joined and config.get('send_chat_states', 'true') == 'true' and not self.input.get_text():
+        if self.joined and config.get_by_tabname('send_chat_states', 'true', self.general_jid, True) == 'true' and not self.input.get_text():
             self.send_chat_state('active')
 
     def on_scroll_up(self):
@@ -1089,9 +1094,9 @@ class MucTab(ChatTab):
         user = User(from_nick, affiliation,
                     show, status, role, jid)
         self.users.append(user)
-        hide_exit_join = config.get('hide_exit_join', -1)
+        hide_exit_join = config.get_by_tabname('hide_exit_join', -1, self.general_jid, True)
         if hide_exit_join != 0:
-            color = user.color[0] if config.get('display_user_color_in_join_part', '') == 'true' else 3
+            color = user.color[0] if config.get_by_tabname('display_user_color_in_join_part', '', self.general_jid, True) == 'true' else 3
             if not jid.full:
                 self.add_message('\x194}%(spec)s \x19%(color)d}%(nick)s\x195} joined the room' % {'nick':from_nick, 'color':color, 'spec':get_theme().CHAR_JOIN})
             else:
@@ -1107,7 +1112,7 @@ class MucTab(ChatTab):
                 if isinstance(_tab, PrivateTab) and JID(_tab.get_name()).bare == self.name:
                     _tab.own_nick = new_nick
         user.change_nick(new_nick)
-        color = user.color[0] if config.get('display_user_color_in_join_part', '') == 'true' else 3
+        color = user.color[0] if config.get_by_tabname('display_user_color_in_join_part', '', self.general_jid, True) == 'true' else 3
         self.add_message('\x19%(color)d}%(old)s\x195} is now known as \x19%(color)d}%(new)s' % {'old':from_nick, 'new':new_nick, 'color':color})
         # rename the private tabs if needed
         self.core.rename_private_tabs(self.name, from_nick, new_nick)
@@ -1130,7 +1135,7 @@ class MucTab(ChatTab):
             else:
                 kick_msg = _('\x191}%(spec)s \x193}You\x195} have been banned.') % {'spec':get_theme().CHAR_KICK}
         else:
-            color = user.color[0] if config.get('display_user_color_in_join_part', '') == 'true' else 3
+            color = user.color[0] if config.get_by_tabname('display_user_color_in_join_part', '', self.general_jid, True) == 'true' else 3
             if by:
                 kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x195} has been banned by \x194}%(by)s') % {'spec':get_theme().CHAR_KICK, 'nick':from_nick, 'color':color, 'by':by}
             else:
@@ -1157,10 +1162,10 @@ class MucTab(ChatTab):
             else:
                 kick_msg = _('\x191}%(spec)s \x193}You\x195} have been kicked.') % {'spec':get_theme().CHAR_KICK}
             # try to auto-rejoin
-            if config.get('autorejoin', 'false') == 'true':
+            if config.get_by_tabname('autorejoin', 'false', self.general_jid, True) == 'true':
                 muc.join_groupchat(self.core.xmpp, self.name, self.own_nick)
         else:
-            color = user.color[0] if config.get('display_user_color_in_join_part', '') == 'true' else 3
+            color = user.color[0] if config.get_by_tabname('display_user_color_in_join_part', '', self.general_jid, True) == 'true' else 3
             if by:
                 kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x195} has been kicked by \x193}%(by)s') % {'spec':get_theme().CHAR_KICK.replace('"', '\\"'), 'nick':from_nick.replace('"', '\\"'), 'color':color, 'by':by.replace('"', '\\"')}
             else:
@@ -1180,9 +1185,9 @@ class MucTab(ChatTab):
             self.core.disable_private_tabs(from_room)
             self.refresh_tab_win()
             self.core.doupdate()
-        hide_exit_join = config.get('hide_exit_join', -1) if config.get('hide_exit_join', -1) >= -1 else -1
+        hide_exit_join = config.get_by_tabname('hide_exit_join', -1, self.general_jid, True) if config.get_by_tabname('hide_exit_join', -1, self.general_jid, True) >= -1 else -1
         if hide_exit_join == -1 or user.has_talked_since(hide_exit_join):
-            color = user.color[0] if config.get('display_user_color_in_join_part', '') == 'true' else 3
+            color = user.color[0] if config.get_by_tabname('display_user_color_in_join_part', '', self.general_jid, True) == 'true' else 3
             if not jid.full:
                 leave_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x195} has left the room') % {'nick':from_nick, 'color':color, 'spec':get_theme().CHAR_QUIT}
             else:
@@ -1200,7 +1205,7 @@ class MucTab(ChatTab):
         # build the message
         display_message = False # flag to know if something significant enough
         # to be displayed has changed
-        color = user.color[0] if config.get('display_user_color_in_join_part', '') == 'true' else 3
+        color = user.color[0] if config.get_by_tabname('display_user_color_in_join_part', '', self.general_jid, True) == 'true' else 3
         if from_nick == self.own_nick:
             msg = _('\x193}You\x195} changed: ')
         else:
@@ -1228,7 +1233,7 @@ class MucTab(ChatTab):
         if not display_message:
             return
         msg = msg[:-2] # remove the last ", "
-        hide_status_change = config.get('hide_status_change', -1)
+        hide_status_change = config.get_by_tabname('hide_status_change', -1, self.general_jid, True)
         if hide_status_change < -1:
             hide_status_change = -1
         if (hide_status_change == -1 or \
@@ -1279,7 +1284,7 @@ class MucTab(ChatTab):
                     self.state = 'highlight'
                 color = get_theme().COLOR_HIGHLIGHT_NICK
             else:
-                highlight_words = config.get('highlight_on', '').split(':')
+                highlight_words = config.get_by_tabname('highlight_on', '', self.general_jid, True).split(':')
                 for word in highlight_words:
                     if word and word.lower() in txt.lower():
                         if self.state != 'current':
@@ -1289,7 +1294,8 @@ class MucTab(ChatTab):
         if color:
             beep_on = config.get('beep_on', 'highlight private').split()
             if 'highlight' in beep_on and 'message' not in beep_on:
-                curses.beep()
+                if config.get_by_tabname('disable_beep', 'false', self.name, False).lower() != 'true':
+                    curses.beep()
         return color
 
     def get_user_by_name(self, nick):
@@ -1356,6 +1362,10 @@ class PrivateTab(ChatTab):
         self.update_commands()
         self.update_keys()
 
+    @property
+    def general_jid(self):
+        return self.get_name()
+
     def completion(self):
         self.complete_commands(self.input)
 
@@ -1373,7 +1383,7 @@ class PrivateTab(ChatTab):
         if msg['body'].find('\x19') != -1:
             msg['xhtml_im'] = xhtml.poezio_colors_to_html(msg['body'])
             msg['body'] = xhtml.clean_text(msg['body'])
-        if config.get('send_chat_states', 'true') == 'true' and self.remote_wants_chatstates is not False:
+        if config.get_by_tabname('send_chat_states', 'true', self.general_jid, True) == 'true' and self.remote_wants_chatstates is not False:
             needed = 'inactive' if self.core.status.show in ('xa', 'away') else 'active'
             msg['chat_state'] = needed
         self.core.events.trigger('private_say_after', msg, self)
@@ -1457,14 +1467,14 @@ class PrivateTab(ChatTab):
         self.text_win.remove_line_separator()
         self.text_win.add_line_separator()
         tab = self.core.get_tab_by_name(JID(self.name).bare, MucTab)
-        if tab.joined and config.get('send_chat_states', 'true') == 'true' and not self.input.get_text():
+        if tab.joined and config.get_by_tabname('send_chat_states', 'true', self.general_jid, True) == 'true' and not self.input.get_text():
             self.send_chat_state('inactive')
 
     def on_gain_focus(self):
         self.state = 'current'
         curses.curs_set(1)
         tab = self.core.get_tab_by_name(JID(self.name).bare, MucTab)
-        if tab.joined and config.get('send_chat_states', 'true') == 'true' and not self.input.get_text():
+        if tab.joined and config.get_by_tabname('send_chat_states', 'true', self.general_jid, True) == 'true' and not self.input.get_text():
             self.send_chat_state('active')
 
     def on_scroll_up(self):
@@ -1511,7 +1521,7 @@ class PrivateTab(ChatTab):
         self.activate()
         tab = self.core.get_tab_by_name(JID(self.name).bare, MucTab)
         color = 3
-        if tab and config.get('display_user_color_in_join_part', ''):
+        if tab and config.get_by_tabname('display_user_color_in_join_part', '', self.general_jid, True):
             user = tab.get_user_by_name(nick)
             if user:
                 color = user.color[0]
@@ -2085,6 +2095,10 @@ class ConversationTab(ChatTab):
         self.update_commands()
         self.update_keys()
 
+    @property
+    def general_jid(self):
+        return JID(self.get_name()).bare
+
     @staticmethod
     def add_information_element(plugin_name, callback):
         """
@@ -2112,7 +2126,7 @@ class ConversationTab(ChatTab):
         if msg['body'].find('\x19') != -1:
             msg['xhtml_im'] = xhtml.poezio_colors_to_html(msg['body'])
             msg['body'] = xhtml.clean_text(msg['body'])
-        if config.get('send_chat_states', 'true') == 'true' and self.remote_wants_chatstates is not False:
+        if config.get_by_tabname('send_chat_states', 'true', self.general_jid, True) == 'true' and self.remote_wants_chatstates is not False:
             needed = 'inactive' if self.core.status.show in ('xa', 'away') else 'active'
             msg['chat_state'] = needed
         self.core.events.trigger('conversation_say_after', msg, self)
@@ -2202,7 +2216,7 @@ class ConversationTab(ChatTab):
         self.state = 'normal'
         self.text_win.remove_line_separator()
         self.text_win.add_line_separator()
-        if config.get('send_chat_states', 'true') == 'true' and (not self.input.get_text() or not self.input.get_text().startswith('//')):
+        if config.get_by_tabname('send_chat_states', 'true', self.general_jid, True) == 'true' and (not self.input.get_text() or not self.input.get_text().startswith('//')):
             if resource:
                 self.send_chat_state('inactive')
 
@@ -2219,7 +2233,7 @@ class ConversationTab(ChatTab):
 
         self.state = 'current'
         curses.curs_set(1)
-        if config.get('send_chat_states', 'true') == 'true' and (not self.input.get_text() or not self.input.get_text().startswith('//')):
+        if config.get_by_tabname('send_chat_states', 'true', self.general_jid, True) == 'true' and (not self.input.get_text() or not self.input.get_text().startswith('//')):
             if resource:
                 self.send_chat_state('active')
 
@@ -2241,7 +2255,7 @@ class ConversationTab(ChatTab):
 
     def on_close(self):
         Tab.on_close(self)
-        if config.get('send_chat_states', 'true') == 'true':
+        if config.get_by_tabname('send_chat_states', 'true', self.general_jid, True) == 'true':
             self.send_chat_state('gone')
 
     def add_message(self, txt, time=None, nickname=None, forced_user=None):
-- 
cgit v1.2.3


From 3d879a6677f665b10fde3c252f32295dcff22c0f Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Mon, 28 Nov 2011 16:46:20 +0100
Subject: Curses operations must operate within the lock

---
 src/windows.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'src')

diff --git a/src/windows.py b/src/windows.py
index 81153239..80a8bbb1 100644
--- a/src/windows.py
+++ b/src/windows.py
@@ -660,8 +660,8 @@ class TextWin(Win):
             lines = self.built_lines[-self.height:]
         else:
             lines = self.built_lines[-self.height-self.pos:-self.pos]
-        self._win.move(0, 0)
         with g_lock:
+            self._win.move(0, 0)
             self._win.erase()
             for y, line in enumerate(lines):
                 if line:
-- 
cgit v1.2.3


From a1575237fd0de77e0d623204eae2c369d8d32b2a Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Mon, 28 Nov 2011 21:27:40 +0100
Subject: Python 3.0/.1 compatibility

---
 src/bookmark.py | 15 +++++++++++----
 1 file changed, 11 insertions(+), 4 deletions(-)

(limited to 'src')

diff --git a/src/bookmark.py b/src/bookmark.py
index dbd8855a..45616d93 100644
--- a/src/bookmark.py
+++ b/src/bookmark.py
@@ -1,9 +1,16 @@
 import os
+from sys import version_info
 
 from sleekxmpp.plugins.xep_0048 import *
 from core import JID
 from config import config
 
+def iter(xml, tag=''):
+    if version_info[1] >= 2:
+        return xml.iter(tag)
+    else:
+        return xml.getiterator(tag)
+
 preferred = config.get('use_bookmarks_method', 'pep').lower()
 if preferred not in ('pep', 'privatexml'):
     preferred = 'privatexml'
@@ -73,10 +80,10 @@ class Bookmark(object):
         name = el.get('name')
         autojoin = True if el.get('autojoin', False) == 'true' else False
         nick = None
-        for n in el.iter('nick'):
+        for n in iter(el, 'nick'):
             nick = nick.text
         password = None
-        for p in el.iter('password'):
+        for p in iter(el, 'password'):
             password = p.text
 
         return Bookmark(jid, name, autojoin, nick, password, method)
@@ -150,7 +157,7 @@ def get_pep(xmpp):
         iq = xmpp.plugin['xep_0048'].get_bookmarks()
     except:
         return False
-    for conf in iq.xml.iter('{storage:bookmarks}conference'):
+    for conf in iter(iq.xml, '{storage:bookmarks}conference'):
         b = Bookmark.parse_from_element(conf, method='pep')
         if not get_by_jid(b.jid):
             bookmarks.append(b)
@@ -162,7 +169,7 @@ def get_privatexml(xmpp):
         iq = xmpp.plugin['xep_0048'].get_bookmarks_old()
     except:
         return False
-    for conf in iq.xml.iter('{storage:bookmarks}conference'):
+    for conf in iter(iq.xml, '{storage:bookmarks}conference'):
         b = Bookmark.parse_from_element(conf, method='privatexml')
         if not get_by_jid(b.jid):
             bookmarks.append(b)
-- 
cgit v1.2.3


From 4d7a11e23868c2c8dbeddc9c3dcb9c3d84ec0101 Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Tue, 29 Nov 2011 00:00:47 +0100
Subject: Add an XML tab (/xml_tab) to view incoming/outgoing stanzas Fixes
 #2074

---
 src/connection.py | 13 +++++++++
 src/core.py       | 22 +++++++++++++++
 src/tabs.py       | 82 +++++++++++++++++++++++++++++++++++++++++++++++++++++++
 3 files changed, 117 insertions(+)

(limited to 'src')

diff --git a/src/connection.py b/src/connection.py
index ffa6590e..352a1d5b 100644
--- a/src/connection.py
+++ b/src/connection.py
@@ -41,6 +41,7 @@ class Connection(sleekxmpp.ClientXMPP):
             jid = '%s/%s' % (config.get('server', 'anon.louiz.org'), resource)
             password = None
         sleekxmpp.ClientXMPP.__init__(self, jid, password, ssl=True)
+        self.core = None
         self.auto_reconnect = False
         self.auto_authorize = None
         self.register_plugin('xep_0030')
@@ -75,3 +76,15 @@ class Connection(sleekxmpp.ClientXMPP):
             return False
         self.process(threaded=True)
         return True
+
+    def send_raw(self, data, now=False, reconnect=None):
+        """
+        Overrides XMLStream.send_raw, with an event added
+        """
+        if self.core:
+            self.core.outgoing_stanza(data)
+        sleekxmpp.ClientXMPP.send_raw(self, data, now, reconnect)
+
+class MatchAll(sleekxmpp.xmlstream.matcher.base.MatcherBase):
+    def match(self, xml):
+        return True
diff --git a/src/core.py b/src/core.py
index b940bb73..15126c08 100644
--- a/src/core.py
+++ b/src/core.py
@@ -27,6 +27,7 @@ import collections
 
 from sleekxmpp.xmlstream.stanzabase import JID
 from sleekxmpp.xmlstream.stanzabase import StanzaBase
+from sleekxmpp.xmlstream.handler import Callback
 
 log = logging.getLogger(__name__)
 
@@ -91,6 +92,7 @@ class Core(object):
         self.running = True
         self.events = events.EventHandler()
         self.xmpp = singleton.Singleton(connection.Connection)
+        self.xmpp.core = self
         self.remote_fifo = None
         # a unique buffer used to store global informations
         # that are displayed in almost all tabs, in an
@@ -98,6 +100,7 @@ class Core(object):
         self.information_buffer = TextBuffer()
         self.information_win_size = config.get('info_win_height', 2, 'var')
         self.information_win = windows.TextWin(300)
+        self.xml_buffer = TextBuffer()
         self.tab_win = windows.GlobalInfoBar()
         self.information_buffer.add_window(self.information_win)
         self.tabs = []
@@ -142,6 +145,7 @@ class Core(object):
             'decline': (self.command_decline, _("Usage: /decline <room> [reason]\nDecline: Decline the invitation to room with or without reason."), self.completion_decline),
             'bookmarks': (self.command_bookmarks, _("Usage: /bookmarks\nBookmarks: Show the current bookmarks."), None),
             'remove_bookmark': (self.command_remove_bookmark, _("Usage: /remove_bookmark [jid]\nRemove Bookmark: Remove the specified bookmark, or the bookmark on the current tab, if any."), self.completion_remove_bookmark),
+            'xml_tab': (self.command_xml_tab, _("Usage: /xml_tab\nXML Tab: Open an XML tab."), None),
             }
 
         self.key_func = {
@@ -188,6 +192,7 @@ class Core(object):
         self.xmpp.add_event_handler("chatstate_paused", self.on_chatstate_paused)
         self.xmpp.add_event_handler("chatstate_gone", self.on_chatstate_gone)
         self.xmpp.add_event_handler("chatstate_inactive", self.on_chatstate_inactive)
+        self.xmpp.register_handler(Callback('ALL THE STANZAS', connection.MatchAll(None), self.incoming_stanza))
 
         self.timed_events = set()
 
@@ -228,6 +233,23 @@ class Core(object):
             self.information_win.resize(self.information_win_size, tabs.Tab.width,
                                         tabs.Tab.height - 1 - self.information_win_size - tabs.Tab.tab_win_height(), 0)
 
+    def outgoing_stanza(self, stanza):
+        self.add_message_to_text_buffer(self.xml_buffer, '\x191}<--\x19o %s' % stanza)
+        if isinstance(self.current_tab(), tabs.XMLTab):
+            self.current_tab().refresh()
+            self.doupdate()
+
+    def incoming_stanza(self, stanza):
+        self.add_message_to_text_buffer(self.xml_buffer, '\x192}-->\x19o %s' % stanza)
+        if isinstance(self.current_tab(), tabs.XMLTab):
+            self.current_tab().refresh()
+            self.doupdate()
+
+    def command_xml_tab(self, arg):
+        """/xml_tab"""
+        tab = tabs.XMLTab()
+        self.add_tab(tab, True)
+
     def resize_global_info_bar(self):
         """
         Resize the GlobalInfoBar only once at each resize
diff --git a/src/tabs.py b/src/tabs.py
index e75989a7..802640b1 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -2399,6 +2399,88 @@ class MucListTab(Tab):
     def on_scroll_down(self):
         self.listview.scroll_down()
 
+class XMLTab(Tab):
+
+    def __init__(self):
+        Tab.__init__(self)
+        self.state = 'normal'
+        self.text_win = windows.TextWin()
+        self.core.xml_buffer.add_window(self.text_win)
+        self.default_help_message = windows.HelpText("/ to enter a command")
+        self.commands['close'] = (self.close, _("Usage: /close\nClose: Just close this tab."), None)
+        self.input = self.default_help_message
+        self.key_func['^T'] = self.close
+        self.key_func['^I'] = self.completion
+        self.key_func["KEY_DOWN"] = self.on_scroll_down
+        self.key_func["KEY_UP"] = self.on_scroll_up
+        self.key_func["/"] = self.on_slash
+        self.resize()
+
+    def on_slash(self):
+        """
+        '/' is pressed, activate the input
+        """
+        curses.curs_set(1)
+        self.input = windows.CommandInput("", self.reset_help_message, self.execute_slash_command)
+        self.input.resize(1, self.width, self.height-1, 0)
+        self.input.do_command("/") # we add the slash
+
+    def reset_help_message(self, _=None):
+        if self.core.current_tab() is self:
+            curses.curs_set(0)
+        self.input = self.default_help_message
+        self.input.refresh()
+        self.core.doupdate()
+        return True
+
+    def on_scroll_up(self):
+        self.text_win.scroll_up(self.text_win.height-1)
+
+    def on_scroll_down(self):
+        self.text_win.scroll_down(self.text_win.height-1)
+
+    def execute_slash_command(self, txt):
+        if txt.startswith('/'):
+            self.input.key_enter()
+            self.execute_command(txt)
+        return self.reset_help_message()
+
+    def completion(self):
+        if isinstance(self.input, windows.CommandInput):
+            self.complete_commands(self.input)
+
+    def on_input(self, key, raw):
+        res = self.input.do_command(key, raw=raw)
+        if res:
+            return True
+        if not raw and key in self.key_func:
+            return self.key_func[key]()
+
+    def close(self, arg=None):
+        self.core.close_tab()
+
+    def resize(self):
+        if not self.visible:
+            return
+        self.need_resize = False
+        self.text_win.resize(self.height-2, self.width, 0, 0)
+        self.input.resize(1, self.width, self.height-1, 0)
+
+    def refresh(self):
+        if self.need_resize:
+            self.resize()
+        log.debug('  TAB   Refresh: %s',self.__class__.__name__)
+        self.text_win.refresh()
+        self.refresh_tab_win()
+        self.input.refresh()
+
+    def on_lose_focus(self):
+        self.state = 'normal'
+
+    def on_gain_focus(self):
+        self.state = 'current'
+        curses.curs_set(0)
+
 class SimpleTextTab(Tab):
     """
     A very simple tab, with just a text displaying some
-- 
cgit v1.2.3


From 8eea23f7d75aa688243501f62446547324ebc2c8 Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Tue, 29 Nov 2011 00:18:31 +0100
Subject: =?UTF-8?q?Fix=20a=20traceback=20when=20using=20^W=20with=20xhtml-?=
 =?UTF-8?q?im=20(it=20does=20not=20work=20perfectly=20as=20it=20should,=20?=
 =?UTF-8?q?but=20it=20doesn=E2=80=99t=20crash=20anymore,=20at=20least)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 src/windows.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

(limited to 'src')

diff --git a/src/windows.py b/src/windows.py
index 80a8bbb1..f2ffc8a7 100644
--- a/src/windows.py
+++ b/src/windows.py
@@ -852,9 +852,9 @@ class Input(Win):
         if not len(self.text) or self.pos == 0:
             return
         separators = string.punctuation+' '
-        while self.pos > 0 and self.text[self.pos+self.line_pos-1] in separators:
+        while self.pos <= len(self.text) and self.pos > 0 and self.text[self.pos+self.line_pos-1] in separators:
             self.key_backspace()
-        while self.pos > 0 and self.text[self.pos+self.line_pos-1] not in separators:
+        while self.pos <= len(self.text) and self.pos > 0 and self.text[self.pos+self.line_pos-1] not in separators:
             self.key_backspace()
 
         return True
-- 
cgit v1.2.3


From 23fa1df9ed6ce39bad81adfaa184f4c66526be98 Mon Sep 17 00:00:00 2001
From: Florent Le Coz <louiz@louiz.org>
Date: Wed, 30 Nov 2011 23:25:22 +0100
Subject: show_tab_names option lets you display the name of the tabs in the
 horizontal bar

---
 src/windows.py | 2 ++
 1 file changed, 2 insertions(+)

(limited to 'src')

diff --git a/src/windows.py b/src/windows.py
index f2ffc8a7..a13efdf5 100644
--- a/src/windows.py
+++ b/src/windows.py
@@ -303,6 +303,8 @@ class GlobalInfoBar(Win):
                     continue
                 try:
                     self.addstr("%s" % str(tab.nb), to_curses_attr(color))
+                    if config.get('show_tab_names', 'false') == 'true':
+                        self.addstr(" %s" % str(tab.get_name()), to_curses_attr(color))
                     self.addstr("|", to_curses_attr(get_theme().COLOR_INFORMATION_BAR))
                 except:             # end of line
                     break
-- 
cgit v1.2.3


From 80047cdebe815ea78b6567fd7a1511f221b2f116 Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Sat, 3 Dec 2011 17:50:40 +0100
Subject: Fixes #2241

---
 src/tabs.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'src')

diff --git a/src/tabs.py b/src/tabs.py
index 802640b1..06e89f44 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -2349,7 +2349,7 @@ class MucListTab(Tab):
         if iq['type'] == 'error':
             self.set_error(iq['error']['type'], iq['error']['code'], iq['error']['text'])
             return
-        items = [{'node-part':JID(item[0]).user,
+        items = [{'node-part': JID(item[0]).user if JID(item[0]).server == self.name else JID(item[0]).bare,
                   'jid': item[0],
                   'name': item[2]} for item in iq['disco_items'].get_items()]
         self.listview.add_lines(items)
-- 
cgit v1.2.3


From 94140aba91d302c320f69e2e40a3164ae21ddb01 Mon Sep 17 00:00:00 2001
From: Florent Le Coz <louiz@louiz.org>
Date: Sun, 6 Nov 2011 20:30:41 +0100
Subject: Fix a crash on empty command in muclisttab

---
 src/tabs.py | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

(limited to 'src')

diff --git a/src/tabs.py b/src/tabs.py
index 06e89f44..d41bf9d3 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -2369,7 +2369,9 @@ class MucListTab(Tab):
         return True
 
     def execute_slash_command(self, txt):
-        self.execute_command(txt)
+        if txt.startswith('/'):
+            self.input.key_enter()
+            self.execute_command(txt)
         return self.reset_help_message()
 
     def get_name(self):
-- 
cgit v1.2.3


From 73e9d837110de4881cf8a2cedb517e6ee355efb5 Mon Sep 17 00:00:00 2001
From: manfraid <manfraid@gmail.com>
Date: Thu, 8 Dec 2011 16:22:43 +0100
Subject: Fixe 2104

---
 src/tabs.py    | 22 ++++++++++++++++----
 src/theming.py |  3 +++
 src/windows.py | 64 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++-
 3 files changed, 84 insertions(+), 5 deletions(-)

(limited to 'src')

diff --git a/src/tabs.py b/src/tabs.py
index d41bf9d3..ab803553 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -2286,6 +2286,9 @@ class MucListTab(Tab):
         self.key_func['j'] = self.join_selected
         self.key_func['J'] = self.join_selected_no_focus
         self.key_func['^M'] = self.join_selected
+        self.key_func['KEY_LEFT'] = self.list_header.sel_column_left
+        self.key_func['KEY_RIGHT'] = self.list_header.sel_column_right
+        self.key_func[' '] = self.sort_by
         self.commands['close'] = (self.close, _("Usage: /close\nClose: Just close this tab."), None)
         self.resize()
         self.update_keys()
@@ -2307,9 +2310,9 @@ class MucListTab(Tab):
             return
         self.need_resize = False
         self.upper_message.resize(1, self.width, 0, 0)
-        column_size = {'node-part': (self.width-5)//4,
-                       'name': (self.width-5)//4*3,
-                       'users': 5}
+        column_size = {'node-part': int(self.width*2/8) ,
+                       'name': int(self.width*5/8),
+                       'users': self.width-int(self.width*2/8)-int(self.width*5/8)}
         self.list_header.resize_columns(column_size)
         self.list_header.resize(1, self.width, 1, 0)
         self.listview.resize_columns(column_size)
@@ -2351,12 +2354,23 @@ class MucListTab(Tab):
             return
         items = [{'node-part': JID(item[0]).user if JID(item[0]).server == self.name else JID(item[0]).bare,
                   'jid': item[0],
-                  'name': item[2]} for item in iq['disco_items'].get_items()]
+                  'name': item[2],'users': ''} for item in iq['disco_items'].get_items()]
         self.listview.add_lines(items)
         self.upper_message.set_message('Chatroom list on server %s' % self.name)
         self.upper_message.refresh()
         curses.doupdate()
 
+    def sort_by(self):
+        if self.list_header.get_order():
+          self.listview.sort_by_column(col_name=self.list_header.get_sel_column(),asc=False)
+          self.list_header.set_order(False)
+          self.list_header.refresh()
+        else:    
+          self.listview.sort_by_column(col_name=self.list_header.get_sel_column(),asc=True)
+          self.list_header.set_order(True)
+          self.list_header.refresh()
+        curses.doupdate()
+
     def join_selected(self):
         row = self.listview.get_selected_row()
         if not row:
diff --git a/src/theming.py b/src/theming.py
index e09d4530..7e90a5a7 100644
--- a/src/theming.py
+++ b/src/theming.py
@@ -154,12 +154,15 @@ class Theme(object):
     COLOR_CONVERSATION_NAME = (2, 4)
     COLOR_GROUPCHAT_NAME = (7, 4)
     COLOR_COLUMN_HEADER = (36, 4)
+    COLOR_COLUMN_HEADER_SEL = (4, 36)
 
     # Strings for special messages (like join, quit, nick change, etc)
     # Special messages
     CHAR_JOIN = '--->'
     CHAR_QUIT = '<---'
     CHAR_KICK = '-!-'
+    CHAR_COLUMN_ASC = ' ▲' 
+    CHAR_COLUMN_DESC =' ▼' 
 
     COLOR_JOIN_CHAR = (4, -1)
     COLOR_QUIT_CHAR = (1, -1)
diff --git a/src/windows.py b/src/windows.py
index a13efdf5..6c3c8b5c 100644
--- a/src/windows.py
+++ b/src/windows.py
@@ -1642,6 +1642,12 @@ class ListWin(Win):
         """
         Sort the list by the given column, ascendant or descendant
         """
+        if asc:
+          self.lines.sort(key=lambda x: x[col_name])
+        else:
+          self.lines.sort(key=lambda x: x[col_name],reverse=True)
+        self.refresh()
+        curses.doupdate()
         pass                    # TODO
 
     def add_lines(self, lines):
@@ -1741,6 +1747,9 @@ class ColumnHeaderWin(Win):
         Win.__init__(self)
         self._columns = columns
         self._columns_sizes = {}
+        self._column_sel = ''
+        self._column_order = ''
+        self._column_order_asc = False
 
     def resize_columns(self, dic):
         self._columns_sizes = dic
@@ -1755,12 +1764,65 @@ class ColumnHeaderWin(Win):
             x = 0
             for col in self._columns:
                 txt = col
+                if col in self._column_order:
+                  if self._column_order_asc:
+                    txt += get_theme().CHAR_COLUMN_ASC 
+                  else:
+                    txt += get_theme().CHAR_COLUMN_DESC
+                #⇓⇑↑↓⇧⇩▲▼
                 size = self._columns_sizes[col]
                 txt += ' ' * (size-len(txt))
-                self.addstr(0, x, txt, to_curses_attr(get_theme().COLOR_COLUMN_HEADER))
+                if col in self._column_sel:
+                  self.addstr(0, x, txt, to_curses_attr(get_theme().COLOR_COLUMN_HEADER_SEL))
+                else:
+                  self.addstr(0, x, txt, to_curses_attr(get_theme().COLOR_COLUMN_HEADER))
                 x += size
             self._refresh()
 
+    def sel_column(self,dic):
+        self._column_sel = dic 
+
+    def get_sel_column(self):
+        return self._column_sel
+
+    def set_order(self,order):
+        self._column_order = self._column_sel
+        self._column_order_asc = order
+
+    def get_order(self):
+        if self._column_sel == self._column_order:
+          return self._column_order_asc
+        else:
+          return False
+
+    def sel_column_left(self):
+        if self._column_sel in self._columns:
+          index = self._columns.index(self._column_sel)
+          if index > 1:
+            index = index -1
+          else:
+            index = 0
+        else:
+          index = 0
+        self._column_sel = self._columns[index]
+        self.refresh()
+        return
+
+    def sel_column_right(self):
+        if self._column_sel in self._columns:
+          index = self._columns.index(self._column_sel)
+          if index < len(self._columns)-2:
+            index = index +1
+          else:
+            index = len(self._columns) -1
+        else:
+          index = len(self._columns) - 1 
+        self._column_sel = self._columns[index]
+        self.refresh()
+        return
+
+
+
 class SimpleTextWin(Win):
     def __init__(self, text):
         Win.__init__(self)
-- 
cgit v1.2.3


From 5fe3c49d407ac77b1e8280e1148d82d0fc1b0bc3 Mon Sep 17 00:00:00 2001
From: Florent Le Coz <louiz@louiz.org>
Date: Fri, 9 Dec 2011 13:44:38 +0100
Subject: /recolor random works correctly.

---
 src/tabs.py | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

(limited to 'src')

diff --git a/src/tabs.py b/src/tabs.py
index ab803553..37dd7a1a 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -666,17 +666,17 @@ class MucTab(ChatTab):
         compare_users = lambda x: x.last_talked
         users = list(self.users)
         sorted_users = sorted(users, key=compare_users, reverse=True)
-        if len(args) >= 1:
-            if args[0] == 'random':
-                random.shuffle(sorted_users)
         # search our own user, to remove it from the list
         for user in sorted_users:
             if user.nick == self.own_nick:
                 sorted_users.remove(user)
                 user.color = get_theme().COLOR_OWN_NICK
-        nb_color = len(get_theme().LIST_COLOR_NICKNAMES)
+        colors = list(get_theme().LIST_COLOR_NICKNAMES)
+        if len(args) >= 1:
+            if args[0] == 'random':
+                random.shuffle(colors)
         for i, user in enumerate(sorted_users):
-            user.color = get_theme().LIST_COLOR_NICKNAMES[i % nb_color]
+            user.color = colors[i % len(colors)]
         self.text_win.rebuild_everything(self._text_buffer)
         self.text_win.refresh()
         self.input.refresh()
-- 
cgit v1.2.3


From cf1cdd18342f115ebd7caea4ee34862bcce52551 Mon Sep 17 00:00:00 2001
From: Florent Le Coz <louiz@louiz.org>
Date: Fri, 9 Dec 2011 13:56:14 +0100
Subject: Fix sorting columns when a room has no name. (we use '' instead of
 None)

---
 src/tabs.py    | 2 +-
 src/windows.py | 1 -
 2 files changed, 1 insertion(+), 2 deletions(-)

(limited to 'src')

diff --git a/src/tabs.py b/src/tabs.py
index 37dd7a1a..dc7aa22d 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -2354,7 +2354,7 @@ class MucListTab(Tab):
             return
         items = [{'node-part': JID(item[0]).user if JID(item[0]).server == self.name else JID(item[0]).bare,
                   'jid': item[0],
-                  'name': item[2],'users': ''} for item in iq['disco_items'].get_items()]
+                  'name': item[2] or '' ,'users': ''} for item in iq['disco_items'].get_items()]
         self.listview.add_lines(items)
         self.upper_message.set_message('Chatroom list on server %s' % self.name)
         self.upper_message.refresh()
diff --git a/src/windows.py b/src/windows.py
index 6c3c8b5c..1b833290 100644
--- a/src/windows.py
+++ b/src/windows.py
@@ -1648,7 +1648,6 @@ class ListWin(Win):
           self.lines.sort(key=lambda x: x[col_name],reverse=True)
         self.refresh()
         curses.doupdate()
-        pass                    # TODO
 
     def add_lines(self, lines):
         """
-- 
cgit v1.2.3


From ef72674102d506567e00793ef84c0893a79ff654 Mon Sep 17 00:00:00 2001
From: Florent Le Coz <louiz@louiz.org>
Date: Fri, 9 Dec 2011 14:06:57 +0100
Subject: Add manfraid to thanks section.

---
 src/tabs.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'src')

diff --git a/src/tabs.py b/src/tabs.py
index dc7aa22d..b1d5bec2 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -2365,7 +2365,7 @@ class MucListTab(Tab):
           self.listview.sort_by_column(col_name=self.list_header.get_sel_column(),asc=False)
           self.list_header.set_order(False)
           self.list_header.refresh()
-        else:    
+        else:
           self.listview.sort_by_column(col_name=self.list_header.get_sel_column(),asc=True)
           self.list_header.set_order(True)
           self.list_header.refresh()
-- 
cgit v1.2.3


From 0d5b96fd8dafcb9727c4d65b2191f0cf56872463 Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Sat, 10 Dec 2011 16:36:18 +0100
Subject: Improve /names command

---
 src/tabs.py | 33 ++++++++++++++++++---------------
 1 file changed, 18 insertions(+), 15 deletions(-)

(limited to 'src')

diff --git a/src/tabs.py b/src/tabs.py
index b1d5bec2..14af2206 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -781,7 +781,12 @@ class MucTab(ChatTab):
         """
         if not self.joined:
             return
-        users, visitors, moderators, participants, others = [], [], [], [], []
+        color_visitor = get_theme().COLOR_USER_VISITOR[0]
+        color_other = get_theme().COLOR_USER_NONE[0]
+        color_moderator = get_theme().COLOR_USER_MODERATOR[0]
+        color_participant = get_theme().COLOR_USER_PARTICIPANT[0]
+        color_information = get_theme().COLOR_INFORMATION_TEXT[0]
+        visitors, moderators, participants, others = [], [], [], []
         for user in self.users:
             if user.role == 'visitor':
                 visitors.append(user.nick)
@@ -791,20 +796,18 @@ class MucTab(ChatTab):
                 moderators.append(user.nick)
             else:
                 others.append(user.nick)
-            users.append(user.nick)
-
-        message = ''
-        roles = (('Users', users), ('Visitors', visitors), ('Participants', participants), ('Moderators', moderators), ('Others', others))
-        for role in roles:
-            if role[1]:
-                role[1].sort()
-                message += '%s: %i\n    ' % (role[0], len(role[1]))
-                last = role[1].pop()
-                for item in role[1]:
-                    message += '%s, ' % item
-                message += '%s\n' % last
-
-        # self.core.add_message_to_text_buffer(room, message)
+
+        message = 'Users: %s \n' % len(self.users)
+        for moderator in moderators:
+            message += ' \x19%s}%s\x19o -' % (color_moderator, moderator)
+        for participant in participants:
+            message += ' \x19%s}%s\x19o -' % (color_participant, participant)
+        for visitor in visitors:
+            message += ' \x19%s}%s\x19o -' % (color_visitor, visitor)
+        for other in others:
+            message += ' \x19%s}%s\x19o -' % (color_other, other)
+        message = message[:-2]
+
         self._text_buffer.add_message(message)
         self.text_win.refresh()
         self.input.refresh()
-- 
cgit v1.2.3


From edf2392fcfdf89ae9696bccf8491abadfcef65cc Mon Sep 17 00:00:00 2001
From: Florent Le Coz <louiz@louiz.org>
Date: Mon, 12 Dec 2011 20:03:42 +0100
Subject: Avoid a traceback on focusing a PrivateTab from a MucTab we already
 left. fixed #2308

---
 src/tabs.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

(limited to 'src')

diff --git a/src/tabs.py b/src/tabs.py
index 14af2206..661b4112 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -1470,14 +1470,14 @@ class PrivateTab(ChatTab):
         self.text_win.remove_line_separator()
         self.text_win.add_line_separator()
         tab = self.core.get_tab_by_name(JID(self.name).bare, MucTab)
-        if tab.joined and config.get_by_tabname('send_chat_states', 'true', self.general_jid, True) == 'true' and not self.input.get_text():
+        if tab and tab.joined and config.get_by_tabname('send_chat_states', 'true', self.general_jid, True) == 'true' and not self.input.get_text():
             self.send_chat_state('inactive')
 
     def on_gain_focus(self):
         self.state = 'current'
         curses.curs_set(1)
         tab = self.core.get_tab_by_name(JID(self.name).bare, MucTab)
-        if tab.joined and config.get_by_tabname('send_chat_states', 'true', self.general_jid, True) == 'true' and not self.input.get_text():
+        if tab and tab.joined and config.get_by_tabname('send_chat_states', 'true', self.general_jid, True) == 'true' and not self.input.get_text():
             self.send_chat_state('active')
 
     def on_scroll_up(self):
-- 
cgit v1.2.3


From fe00f0eb23f492f5c15d3ac843c8ce7158daf174 Mon Sep 17 00:00:00 2001
From: Florent Le Coz <louiz@louiz.org>
Date: Mon, 12 Dec 2011 20:31:05 +0100
Subject: ALWAYS display affiliation or role changes, in MucTabs. fixes #2309

---
 src/tabs.py | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

(limited to 'src')

diff --git a/src/tabs.py b/src/tabs.py
index 661b4112..e696f46d 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -1239,14 +1239,17 @@ class MucTab(ChatTab):
         hide_status_change = config.get_by_tabname('hide_status_change', -1, self.general_jid, True)
         if hide_status_change < -1:
             hide_status_change = -1
-        if (hide_status_change == -1 or \
+        if ((hide_status_change == -1 or \
                 user.has_talked_since(hide_status_change) or\
                 user.nick == self.own_nick)\
                 and\
                 (affiliation != user.affiliation or\
                     role != user.role or\
                     show != user.show or\
-                    status != user.status):
+                    status != user.status))\
+                      or\
+                        (affiliation != user.affiliation or\
+                          role != user.role):
             # display the message in the room
             self._text_buffer.add_message(msg)
         self.core.on_user_changed_status_in_private('%s/%s' % (from_room, from_nick), msg)
-- 
cgit v1.2.3


From 3d9193509e44f0767d3b5d0d5f984a90fec572e3 Mon Sep 17 00:00:00 2001
From: Florent Le Coz <louiz@louiz.org>
Date: Thu, 15 Dec 2011 20:29:20 +0100
Subject: Make non-remote commands work.

---
 src/core.py   | 4 +++-
 src/daemon.py | 6 +++++-
 2 files changed, 8 insertions(+), 2 deletions(-)

(limited to 'src')

diff --git a/src/core.py b/src/core.py
index 15126c08..3bb0436e 100644
--- a/src/core.py
+++ b/src/core.py
@@ -54,6 +54,7 @@ from keyboard import read_char
 from theming import get_theme
 from fifo import Fifo
 from windows import g_lock
+from daemon import Executor
 
 # http://xmpp.org/extensions/xep-0045.html#errorstatus
 ERROR_AND_STATUS_CODES = {
@@ -2143,7 +2144,8 @@ class Core(object):
                 self.information('Could not execute [%s]: %s' % (command.strip(), e,), 'Error')
                 self.remote_fifo = None
         else:
-            pass
+            e = Executor(command.strip())
+            e.start()
 
     def get_conversation_messages(self):
         """
diff --git a/src/daemon.py b/src/daemon.py
index f23d6b5f..b413f465 100755
--- a/src/daemon.py
+++ b/src/daemon.py
@@ -23,6 +23,10 @@ import sys
 import threading
 import subprocess
 
+import logging
+
+log = logging.getLogger(__name__)
+
 class Executor(threading.Thread):
     """
     Just a class to execute commands in a thread.
@@ -35,7 +39,7 @@ class Executor(threading.Thread):
         self.command = command
 
     def run(self):
-        print('executing %s' % (self.command.strip(),))
+        log.info('executing %s' % (self.command.strip(),))
         subprocess.call(self.command.split())
 
 def main():
-- 
cgit v1.2.3


From 825d5a6dd1aee29aef12d49d147b50f8ce7c15d0 Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Sun, 18 Dec 2011 20:16:19 +0100
Subject: Beep when receiving a MUC invitation (by default)

---
 src/core.py | 2 ++
 1 file changed, 2 insertions(+)

(limited to 'src')

diff --git a/src/core.py b/src/core.py
index 3bb0436e..d0f6c4de 100644
--- a/src/core.py
+++ b/src/core.py
@@ -371,6 +371,8 @@ class Core(object):
         if password:
             msg += ". The password is \"%s\"." % password
         self.information(msg, 'Info')
+        if 'invite' in config.get('beep_on', 'invite').split():
+            curses.beep()
         self.pending_invites[jid.bare] = inviter.full
 
     def command_invite(self, arg):
-- 
cgit v1.2.3


From f1fae6b38717a4df54e4306608ffa22055548f69 Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Sun, 18 Dec 2011 20:35:19 +0100
Subject: Add a /invites command

---
 src/core.py | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

(limited to 'src')

diff --git a/src/core.py b/src/core.py
index d0f6c4de..e3d655d5 100644
--- a/src/core.py
+++ b/src/core.py
@@ -144,6 +144,7 @@ class Core(object):
             'set_plugin': (self.command_set_plugin, _("Usage: /set_plugin <plugin> <option> [value]\nSet Plugin: Set the value of the option in a plugin configuration file."), self.completion_set_plugin),
             'invite': (self.command_invite, _("Usage: /invite <jid> <room> [reason]\nInvite: Invite jid in room with reason."), self.completion_invite),
             'decline': (self.command_decline, _("Usage: /decline <room> [reason]\nDecline: Decline the invitation to room with or without reason."), self.completion_decline),
+            'invitations': (self.command_invitations, _("Usage: /invites\nInvites: Show the pending invitations."), None),
             'bookmarks': (self.command_bookmarks, _("Usage: /bookmarks\nBookmarks: Show the current bookmarks."), None),
             'remove_bookmark': (self.command_remove_bookmark, _("Usage: /remove_bookmark [jid]\nRemove Bookmark: Remove the specified bookmark, or the bookmark on the current tab, if any."), self.completion_remove_bookmark),
             'xml_tab': (self.command_xml_tab, _("Usage: /xml_tab\nXML Tab: Open an XML tab."), None),
@@ -376,6 +377,7 @@ class Core(object):
         self.pending_invites[jid.bare] = inviter.full
 
     def command_invite(self, arg):
+        """/invite <to> <room> [reason]"""
         args = common.shell_split(arg)
         if len(args) < 2:
             return
@@ -400,6 +402,7 @@ class Core(object):
             return the_input.auto_completion(rooms, '')
 
     def command_decline(self, arg):
+        """/decline <room@server.tld> [reason]"""
         args = common.shell_split(arg)
         if not len(args):
             return
@@ -409,6 +412,17 @@ class Core(object):
         reason = args[1] if len(args) > 1 else ''
         self.xmpp.plugin['xep_0045'].decline_invite(jid.bare, self.pending_invites[jid.bare], reason)
 
+    def command_invitations(self, arg):
+        """/invitations"""
+        build = ""
+        for invite in self.pending_invites:
+            build += "%s by %s" % (invite, JID(self.pending_invites[invite]).bare)
+        if self.pending_invites:
+            build = "You are invited to the following rooms:\n" + build
+        else:
+            build = "You are do not have any pending invitation."
+        self.information(build, 'Info')
+
     def completion_decline(self, the_input):
         txt = the_input.get_text()
         args = common.shell_split(txt)
-- 
cgit v1.2.3


From 91698b6f1b54982460a7f1869b96980d194a9bb1 Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Sun, 18 Dec 2011 20:43:40 +0100
Subject: Fix the completion for /decline and /invite

---
 src/core.py | 25 ++++++++++++++-----------
 1 file changed, 14 insertions(+), 11 deletions(-)

(limited to 'src')

diff --git a/src/core.py b/src/core.py
index e3d655d5..883eb8fa 100644
--- a/src/core.py
+++ b/src/core.py
@@ -387,14 +387,15 @@ class Core(object):
         self.xmpp.plugin['xep_0045'].invite(room, to, reason)
 
     def completion_invite(self, the_input):
+        """Completion for /invite"""
         txt = the_input.get_text()
         args = common.shell_split(txt)
         n = len(args)
         if txt.endswith(' '):
             n += 1
-        if len(args) == 1:
+        if n == 2:
             return the_input.auto_completion([contact.bare_jid for contact in roster.get_contacts()], '')
-        elif len(args) == 2:
+        elif n == 3:
             rooms = []
             for tab in self.tabs:
                 if isinstance(tab, tabs.MucTab) and tab.joined:
@@ -410,8 +411,19 @@ class Core(object):
         if jid.bare not in self.pending_invites:
             return
         reason = args[1] if len(args) > 1 else ''
+        del self.pending_invites[jid.bare]
         self.xmpp.plugin['xep_0045'].decline_invite(jid.bare, self.pending_invites[jid.bare], reason)
 
+    def completion_decline(self, the_input):
+        """Completion for /decline"""
+        txt = the_input.get_text()
+        args = common.shell_split(txt)
+        n = len(args)
+        if txt.endswith(' '):
+            n += 1
+        if n == 2:
+            return the_input.auto_completion(list(self.pending_invites.keys()), '')
+
     def command_invitations(self, arg):
         """/invitations"""
         build = ""
@@ -423,15 +435,6 @@ class Core(object):
             build = "You are do not have any pending invitation."
         self.information(build, 'Info')
 
-    def completion_decline(self, the_input):
-        txt = the_input.get_text()
-        args = common.shell_split(txt)
-        n = len(args)
-        if txt.endswith(' '):
-            n += 1
-        if len(args) == 1:
-            return the_input.auto_completion(list(self.pending_invites.keys()), '')
-
     def on_groupchat_decline(self, decline):
         pass
 
-- 
cgit v1.2.3


From adf3cfe6947ab9d6f6465b4f3564e142ffb1f6a0 Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Mon, 19 Dec 2011 17:19:18 +0100
Subject: Fix a little incoherence in /query

---
 src/tabs.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'src')

diff --git a/src/tabs.py b/src/tabs.py
index e696f46d..45860acc 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -758,7 +758,7 @@ class MucTab(ChatTab):
             if user.nick == nick:
                 r = self.core.open_private_window(self.name, user.nick)
         if r and len(args) > 1:
-            msg = arg[len(nick)+1:]
+            msg = args[1]
             self.core.current_tab().command_say(xhtml.convert_simple_to_full_colors(msg))
         if not r:
             self.core.information(_("Cannot find user: %s" % nick), 'Error')
-- 
cgit v1.2.3


From 765f1807a9c67894712fd680de6e6de6deddf808 Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Wed, 28 Dec 2011 23:32:23 +0100
Subject: Fix various issues with the /configure tab Vertical tab list, text
 not erased from previous tab

---
 src/data_forms.py | 16 +++++++++++-----
 1 file changed, 11 insertions(+), 5 deletions(-)

(limited to 'src')

diff --git a/src/data_forms.py b/src/data_forms.py
index 2d17a304..d4a8697a 100644
--- a/src/data_forms.py
+++ b/src/data_forms.py
@@ -75,7 +75,7 @@ class DataFormsTab(Tab):
 
     def refresh(self):
         self.topic_win.refresh(self._form['title'])
-        self.tab_win.refresh()
+        self.refresh_tab_win()
         self.help_win.refresh()
         self.help_win_dyn.refresh(self.form_win.get_help_message())
         self.form_win.refresh()
@@ -88,7 +88,7 @@ class FieldInput(object):
     """
     def __init__(self, field):
         self._field = field
-        self.color = (14, -1)
+        self.color = get_theme().COLOR_NORMAL_TEXT
 
     def set_color(self, color):
         self.color = color
@@ -131,6 +131,7 @@ class ColoredLabel(windows.Win):
 
     def refresh(self):
         with g_lock:
+            self._win.erase()
             self._win.attron(to_curses_attr(self.color))
             self.addstr(0, 0, self.text)
             self._win.attroff(to_curses_attr(self.color))
@@ -157,7 +158,7 @@ class DummyInput(FieldInput, windows.Win):
 class ColoredLabel(windows.Win):
     def __init__(self, text):
         self.text = text
-        self.color = (14, -1)
+        self.color = get_theme().COLOR_NORMAL_TEXT
         windows.Win.__init__(self)
 
     def resize(self, height, width, y, x):
@@ -169,6 +170,7 @@ class ColoredLabel(windows.Win):
 
     def refresh(self):
         with g_lock:
+            self._win.erase()
             self._win.attron(to_curses_attr(self.color))
             self.addstr(0, 0, self.text)
             self._win.attroff(to_curses_attr(self.color))
@@ -189,6 +191,7 @@ class BooleanWin(FieldInput, windows.Win):
 
     def refresh(self):
         with g_lock:
+            self._win.erase()
             self._win.attron(to_curses_attr(self.color))
             self.addnstr(0, 0, ' '*(8), self.width)
             self.addstr(0, 2, "%s"%self.value)
@@ -253,6 +256,7 @@ class TextMultiWin(FieldInput, windows.Win):
     def refresh(self):
         if not self.edition_input:
             with g_lock:
+                self._win.erase()
                 self._win.attron(to_curses_attr(self.color))
                 self.addnstr(0, 0, ' '*self.width, self.width)
                 option = self.options[self.val_pos]
@@ -305,6 +309,7 @@ class ListMultiWin(FieldInput, windows.Win):
 
     def refresh(self):
         with g_lock:
+            self._win.erase()
             self._win.attron(to_curses_attr(self.color))
             self.addnstr(0, 0, ' '*self.width, self.width)
             if self.val_pos > 0:
@@ -351,6 +356,7 @@ class ListSingleWin(FieldInput, windows.Win):
 
     def refresh(self):
         with g_lock:
+            self._win.erase()
             self._win.attron(to_curses_attr(self.color))
             self.addnstr(0, 0, ' '*self.width, self.width)
             if self.val_pos > 0:
@@ -377,7 +383,7 @@ class TextSingleWin(FieldInput, windows.Input):
         self.text = field.getValue() if isinstance(field.getValue(), str)\
             else ""
         self.pos = len(self.text)
-        self.color = (14, -1)
+        self.color = get_theme().COLOR_NORMAL_TEXT
 
     def reply(self):
         self._field['label'] = ''
@@ -427,7 +433,7 @@ class FormWin(object):
                      }
     def __init__(self, form, height, width, y, x):
         self._form = form
-        self._win = curses.newwin(height, width, y, x)
+        self._win = windows.Win._tab_win.derwin(height, width, y, x)
         self.current_input = 0
         self.inputs = []        # dict list
         for (name, field) in self._form.getFields().items():
-- 
cgit v1.2.3


From 13430de1f4843527e07ae9edc75e114a064836e7 Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Thu, 29 Dec 2011 18:55:43 +0100
Subject: Fix the resize issues with the /configure tab

---
 src/data_forms.py | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

(limited to 'src')

diff --git a/src/data_forms.py b/src/data_forms.py
index d4a8697a..d38f392a 100644
--- a/src/data_forms.py
+++ b/src/data_forms.py
@@ -67,13 +67,14 @@ class DataFormsTab(Tab):
     def resize(self):
         self.need_resize = False
         self.topic_win.resize(1, self.width, 0, 0)
-        self.tab_win.resize(1, self.width, self.height-2, 0)
-        self.form_win.resize(self.height-4, self.width, 1, 0)
+        self.form_win.resize(self.height-3 - Tab.tab_win_height(), self.width, 1, 0)
         self.help_win.resize(1, self.width, self.height-1, 0)
-        self.help_win_dyn.resize(1, self.width, self.height-3, 0)
+        self.help_win_dyn.resize(1, self.width, self.height-2 - Tab.tab_win_height(), 0)
         self.lines = []
 
     def refresh(self):
+        if self.need_resize:
+            self.resize()
         self.topic_win.refresh(self._form['title'])
         self.refresh_tab_win()
         self.help_win.refresh()
@@ -453,7 +454,6 @@ class FormWin(object):
                                 'input':inp})
 
     def resize(self, height, width, y, x):
-        self._win.resize(height, width)
         self.height = height
         self.width = width
         if self.current_input >= self.height-2:
-- 
cgit v1.2.3


From 83abd9f19666eea8edfcd30f80fbd08388bc8816 Mon Sep 17 00:00:00 2001
From: Florent Le Coz <louiz@louiz.org>
Date: Tue, 10 Jan 2012 16:30:59 +0100
Subject: Add a simple_notify plugin

And also fix some issues, like the split of the exec_command arguments.
And the doc for the plugin, too.
---
 src/core.py   | 5 ++++-
 src/daemon.py | 5 +++--
 src/tabs.py   | 3 +++
 3 files changed, 10 insertions(+), 3 deletions(-)

(limited to 'src')

diff --git a/src/core.py b/src/core.py
index 883eb8fa..1023c748 100644
--- a/src/core.py
+++ b/src/core.py
@@ -2164,7 +2164,10 @@ class Core(object):
                 self.remote_fifo = None
         else:
             e = Executor(command.strip())
-            e.start()
+            try:
+                e.start()
+            except ValueError as e: # whenever shlex fails
+                self.information('%s' % (e,), 'Error')
 
     def get_conversation_messages(self):
         """
diff --git a/src/daemon.py b/src/daemon.py
index b413f465..4b4c0b79 100755
--- a/src/daemon.py
+++ b/src/daemon.py
@@ -22,7 +22,7 @@ command on your local machine.
 import sys
 import threading
 import subprocess
-
+import shlex
 import logging
 
 log = logging.getLogger(__name__)
@@ -40,7 +40,8 @@ class Executor(threading.Thread):
 
     def run(self):
         log.info('executing %s' % (self.command.strip(),))
-        subprocess.call(self.command.split())
+        command = shlex.split(self.command)
+        subprocess.call(command)
 
 def main():
     while True:
diff --git a/src/tabs.py b/src/tabs.py
index 45860acc..8316d2a3 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -1318,6 +1318,7 @@ class MucTab(ChatTab):
         Note that user can be None even if nickname is not None. It happens
         when we receive an history message said by someone who is not
         in the room anymore
+        Return True if the message highlighted us. False otherwise.
         """
         self.log_message(txt, time, nickname)
         user = self.get_user_by_name(nickname) if nickname is not None else None
@@ -1331,6 +1332,7 @@ class MucTab(ChatTab):
             if self.state != 'highlight':
                 self.state = 'message'
         nick_color = nick_color or None
+        highlight = False
         if (not nickname or time) and not txt.startswith('/me '):
             txt = '\x195}%s' % (txt,)
         else:                   # TODO
@@ -1339,6 +1341,7 @@ class MucTab(ChatTab):
                 nick_color = highlight
         time = time or datetime.now()
         self._text_buffer.add_message(txt, time, nickname, nick_color, history, user)
+        return highlight
 
 class PrivateTab(ChatTab):
     """
-- 
cgit v1.2.3


From 88f7e690bda7e53e41b9cfaac2027c76fc774727 Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Wed, 11 Jan 2012 18:13:32 +0100
Subject: Fix the alias plugin

---
 src/common.py | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

(limited to 'src')

diff --git a/src/common.py b/src/common.py
index 2da7835b..26b5dd0f 100644
--- a/src/common.py
+++ b/src/common.py
@@ -235,13 +235,16 @@ def parse_secs_to_str(duration=0):
     result += '%ss' % secs if secs else ''
     return result
 
-def parse_command_args_to_alias(args, strto):
+def parse_command_args_to_alias(arg, strto):
     """
     Parse command parameters.
     Numbers can be from 0 to 9.
-    >>> parse_command_args_to_alias(['sdf', 'koin'], '%0 %1')
-    "sdf koin"
+    >>> parse_command_args_to_alias('sdf koin', '%1 %0')
+    "koin sdf"
     """
+    if '%' not in strto:
+        return strto + arg
+    args = shell_split(arg)
     l = len(args)
     dest = ''
     var_num = False
@@ -250,7 +253,7 @@ def parse_command_args_to_alias(args, strto):
             if not var_num:
                 dest += i
             elif i in string.digits:
-                if int(i) < l:
+                if 0 <= int(i) < l:
                     dest += args[int(i)]
                 var_num = False
         elif i == '%':
-- 
cgit v1.2.3


From 2240916efecb86f35301dfc83b96444f8a638476 Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Sun, 15 Jan 2012 16:40:49 +0100
Subject: Fixes #2319

---
 src/tabs.py | 4 ++++
 1 file changed, 4 insertions(+)

(limited to 'src')

diff --git a/src/tabs.py b/src/tabs.py
index 8316d2a3..e8a773fe 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -1419,6 +1419,8 @@ class PrivateTab(ChatTab):
                                                              res.get('version') or _('unknown'),
                                                              res.get('os') or _('on an unknown platform'))
             self.core.information(version, 'Info')
+        if arg:
+            return self.core.command_version(arg)
         jid = self.name
         self.core.xmpp.plugin['xep_0092'].get_version(jid, callback=callback)
 
@@ -2172,6 +2174,8 @@ class ConversationTab(ChatTab):
                                                              res.get('version') or _('unknown'),
                                                              res.get('os') or _('on an unknown platform'))
             self.core.information(version, 'Info')
+        if arg:
+            return self.core.command_version(arg)
         jid = self._name
         self.core.xmpp.plugin['xep_0092'].get_version(jid, callback=callback)
 
-- 
cgit v1.2.3


From 3d7d7425ee26d73aedb9dc1a51d87d25e37180d4 Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Sat, 21 Jan 2012 00:06:18 +0100
Subject: Do not quotify topic completion

---
 src/tabs.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'src')

diff --git a/src/tabs.py b/src/tabs.py
index e8a773fe..188b6389 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -814,7 +814,7 @@ class MucTab(ChatTab):
 
     def completion_topic(self, the_input):
         current_topic = self.topic
-        return the_input.auto_completion([current_topic], '')
+        return the_input.auto_completion([current_topic], '', quotify=False)
 
     def command_kick(self, arg):
         """
-- 
cgit v1.2.3


From e17b70e285406da6a3a564ff4b7c0da2fc208df2 Mon Sep 17 00:00:00 2001
From: Florent Le Coz <louiz@louiz.org>
Date: Tue, 24 Jan 2012 14:49:03 +0100
Subject: Display colors for /me messages in a better way.

---
 src/text_buffer.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'src')

diff --git a/src/text_buffer.py b/src/text_buffer.py
index 3541b9c1..f396bd5b 100644
--- a/src/text_buffer.py
+++ b/src/text_buffer.py
@@ -44,7 +44,7 @@ class TextBuffer(object):
             else:
                 color = None
             # TODO: display the bg color too.
-            txt = ("\x19%s}* \x195}" % (color or 5,))+ nickname + ' ' + txt[4:]
+            txt = ("\x195}* \x19%s}" % (color or 5,))+ nickname + ' \x195}' + txt[4:]
             nickname = None
         msg = Message(txt='%s\x19o'%(txt.replace('\t', '    '),), nick_color=nick_color,
                       time=time, str_time=time.strftime("%Y-%m-%d %H:%M:%S")\
-- 
cgit v1.2.3


From 7d5fa2a3169a95c3562b89814046f7460450cbef Mon Sep 17 00:00:00 2001
From: Florent Le Coz <louiz@louiz.org>
Date: Tue, 24 Jan 2012 15:17:54 +0100
Subject: Fix the comments in the theming.py file

---
 src/theming.py | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

(limited to 'src')

diff --git a/src/theming.py b/src/theming.py
index 7e90a5a7..4e1f37ea 100644
--- a/src/theming.py
+++ b/src/theming.py
@@ -11,16 +11,17 @@ used when drawing the interface.
 
 Colors are numbers from -1 to 7 (if only 8 colors are supported) or -1 to 255
 if 256 colors are available.
-We check the number of available colors at startup, and we load a theme accordingly.
-A 8 color theme should NEVER use colors not in the -1 -> 7 range. We won't check that
-at run time. If the case occurs, the THEME should be fixed.
+If only 8 colors are available, all colors > 8 are converted using the
+table_256_to_16 dict.
+
 XHTML-IM colors are converted to -1 -> 255 colors if available, or directly to
 -1 -> 8 if we are in 8-color-mode.
 
 A pair_color is a background-foreground pair. All possible pairs are not created
 at startup, because that would create 256*256 pairs, and almost all of them
 would never be used.
-So, a theme should define color tuples, like (200, -1), and when they are to
+
+A theme should define color tuples, like (200, -1), and when they are to
 be used by poezio's interface, they will be created once, and kept in a list for
 later usage.
 A color tuple is of the form (foreground, background, optional)
-- 
cgit v1.2.3


From 5924cd4a13ee293b0b9c7bfe87ad70a7e896ae05 Mon Sep 17 00:00:00 2001
From: Florent Le Coz <louiz@louiz.org>
Date: Tue, 24 Jan 2012 15:18:08 +0100
Subject: Remove all the .replace('"', '\\"') which are useless since the new
 way to express colors in messages.

---
 src/tabs.py | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

(limited to 'src')

diff --git a/src/tabs.py b/src/tabs.py
index 188b6389..fa99fc05 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -1142,7 +1142,7 @@ class MucTab(ChatTab):
             if by:
                 kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x195} has been banned by \x194}%(by)s') % {'spec':get_theme().CHAR_KICK, 'nick':from_nick, 'color':color, 'by':by}
             else:
-                kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x195} has been banned') % {'spec':get_theme().CHAR_KICK, 'nick':from_nick.replace('"', '\\"'), 'color':color}
+                kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x195} has been banned') % {'spec':get_theme().CHAR_KICK, 'nick':from_nick, 'color':color}
         if reason is not None and reason.text:
             kick_msg += _('\x195} Reason: \x196}%(reason)s\x195}') % {'reason': reason.text}
         self._text_buffer.add_message(kick_msg)
@@ -1170,9 +1170,9 @@ class MucTab(ChatTab):
         else:
             color = user.color[0] if config.get_by_tabname('display_user_color_in_join_part', '', self.general_jid, True) == 'true' else 3
             if by:
-                kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x195} has been kicked by \x193}%(by)s') % {'spec':get_theme().CHAR_KICK.replace('"', '\\"'), 'nick':from_nick.replace('"', '\\"'), 'color':color, 'by':by.replace('"', '\\"')}
+                kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x195} has been kicked by \x193}%(by)s') % {'spec':get_theme().CHAR_KICK, 'nick':from_nick, 'color':color, 'by':by}
             else:
-                kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x195} has been kicked') % {'spec':get_theme().CHAR_KICK, 'nick':from_nick.replace('"', '\\"'), 'color':color}
+                kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x195} has been kicked') % {'spec':get_theme().CHAR_KICK, 'nick':from_nick, 'color':color}
         if reason is not None and reason.text:
             kick_msg += _('\x195} Reason: \x196}%(reason)s') % {'reason': reason.text}
         self.add_message(kick_msg)
@@ -1212,7 +1212,7 @@ class MucTab(ChatTab):
         if from_nick == self.own_nick:
             msg = _('\x193}You\x195} changed: ')
         else:
-            msg = _('\x19%(color)d}%(nick)s\x195} changed: ') % {'nick': from_nick.replace('"', '\\"'), 'color': color}
+            msg = _('\x19%(color)d}%(nick)s\x195} changed: ') % {'nick': from_nick, 'color': color}
         if show not in SHOW_NAME:
             self.core.information("%s from room %s sent an invalid show: %s" %\
                                       (from_nick, from_room, show), "warning")
@@ -1518,9 +1518,9 @@ class PrivateTab(ChatTab):
         """
         self.deactivate()
         if not status_message:
-            self.add_message(_('\x191}%(spec)s \x193}%(nick)s\x195} has left the room') % {'nick':from_nick.replace('"', '\\"'), 'spec':get_theme().CHAR_QUIT.replace('"', '\\"')})
+            self.add_message(_('\x191}%(spec)s \x193}%(nick)s\x195} has left the room') % {'nick':from_nick, 'spec':get_theme().CHAR_QUIT})
         else:
-            self.add_message(_('\x191}%(spec)s \x193}%(nick)s\x195} has left the room (%(status)s)"') % {'nick':from_nick.replace('"', '\\"'), 'spec':get_theme().CHAR_QUIT, 'status': status_message.replace('"', '\\"')})
+            self.add_message(_('\x191}%(spec)s \x193}%(nick)s\x195} has left the room (%(status)s)"') % {'nick':from_nick, 'spec':get_theme().CHAR_QUIT, 'status': status_message})
         if self.core.current_tab() is self:
             self.refresh()
             self.core.doupdate()
-- 
cgit v1.2.3


From 7512001612a42cc46552a619d876c551e8ee51b1 Mon Sep 17 00:00:00 2001
From: Florent Le Coz <louiz@louiz.org>
Date: Tue, 24 Jan 2012 15:46:51 +0100
Subject: Make the color for information messages themable

---
 src/tabs.py        | 50 +++++++++++++++++++++++++-------------------------
 src/text_buffer.py |  3 ++-
 src/theming.py     |  2 +-
 3 files changed, 28 insertions(+), 27 deletions(-)

(limited to 'src')

diff --git a/src/tabs.py b/src/tabs.py
index fa99fc05..53405db4 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -1058,9 +1058,9 @@ class MucTab(ChatTab):
                     if self.core.current_tab() == self and self.core.status.show not in ('xa', 'away'):
                         self.send_chat_state('active')
                     new_user.color = get_theme().COLOR_OWN_NICK
-                    self.add_message(_("\x195}Your nickname is \x193}%s") % (from_nick))
+                    self.add_message(_("\x19%(info_col)s}Your nickname is \x193}%(nick)s") % {'nick': from_nick, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]})
                     if '170' in status_codes:
-                        self.add_message('\x191}Warning: \x195}this room is publicly logged')
+                        self.add_message('\x191}Warning: \x19%(info_col)s}this room is publicly logged' % {'info_col': get_theme().COLOR_INFORMATION_TEXT[0]})
         else:
             change_nick = '303' in status_codes
             kick = '307' in status_codes and typ == 'unavailable'
@@ -1101,9 +1101,9 @@ class MucTab(ChatTab):
         if hide_exit_join != 0:
             color = user.color[0] if config.get_by_tabname('display_user_color_in_join_part', '', self.general_jid, True) == 'true' else 3
             if not jid.full:
-                self.add_message('\x194}%(spec)s \x19%(color)d}%(nick)s\x195} joined the room' % {'nick':from_nick, 'color':color, 'spec':get_theme().CHAR_JOIN})
+                self.add_message('\x194}%(spec)s \x19%(color)d}%(nick)s\x19%(info_col)s} joined the room' % {'nick':from_nick, 'color':color, 'spec':get_theme().CHAR_JOIN, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]})
             else:
-                self.add_message('\x194}%(spec)s \x19%(color)d}%(nick)s \x195}(\x194}%(jid)s\x195}) joined the room' % {'spec':get_theme().CHAR_JOIN, 'nick':from_nick, 'color':color, 'jid':jid.full})
+                self.add_message('\x194}%(spec)s \x19%(color)d}%(nick)s \x19%(info_col)s}(\x194}%(jid)s\x19%(info_col)s}) joined the room' % {'spec':get_theme().CHAR_JOIN, 'nick':from_nick, 'color':color, 'jid':jid.full, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]})
         self.core.on_user_rejoined_private_conversation(self.name, from_nick)
 
     def on_user_nick_change(self, presence, user, from_nick, from_room):
@@ -1116,7 +1116,7 @@ class MucTab(ChatTab):
                     _tab.own_nick = new_nick
         user.change_nick(new_nick)
         color = user.color[0] if config.get_by_tabname('display_user_color_in_join_part', '', self.general_jid, True) == 'true' else 3
-        self.add_message('\x19%(color)d}%(old)s\x195} is now known as \x19%(color)d}%(new)s' % {'old':from_nick, 'new':new_nick, 'color':color})
+        self.add_message('\x19%(color)d}%(old)s\x19%(info_col)s} is now known as \x19%(color)d}%(new)s' % {'old':from_nick, 'new':new_nick, 'color':color, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]})
         # rename the private tabs if needed
         self.core.rename_private_tabs(self.name, from_nick, new_nick)
 
@@ -1134,17 +1134,17 @@ class MucTab(ChatTab):
             self.refresh_tab_win()
             self.core.doupdate()
             if by:
-                kick_msg = _('\x191}%(spec)s \x193}You\x195} have been banned by \x194}%(by)s') % {'spec': get_theme().CHAR_KICK, 'by':by}
+                kick_msg = _('\x191}%(spec)s \x193}You\x19%(info_col)s} have been banned by \x194}%(by)s') % {'spec': get_theme().CHAR_KICK, 'by':by, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}
             else:
-                kick_msg = _('\x191}%(spec)s \x193}You\x195} have been banned.') % {'spec':get_theme().CHAR_KICK}
+                kick_msg = _('\x191}%(spec)s \x193}You\x19%(info_col)s} have been banned.') % {'spec':get_theme().CHAR_KICK, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}
         else:
             color = user.color[0] if config.get_by_tabname('display_user_color_in_join_part', '', self.general_jid, True) == 'true' else 3
             if by:
-                kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x195} has been banned by \x194}%(by)s') % {'spec':get_theme().CHAR_KICK, 'nick':from_nick, 'color':color, 'by':by}
+                kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x19%(info_col)s} has been banned by \x194}%(by)s') % {'spec':get_theme().CHAR_KICK, 'nick':from_nick, 'color':color, 'by':by, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}
             else:
-                kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x195} has been banned') % {'spec':get_theme().CHAR_KICK, 'nick':from_nick, 'color':color}
+                kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x19%(info_col)s} has been banned') % {'spec':get_theme().CHAR_KICK, 'nick':from_nick, 'color':color, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}
         if reason is not None and reason.text:
-            kick_msg += _('\x195} Reason: \x196}%(reason)s\x195}') % {'reason': reason.text}
+            kick_msg += _('\x19%(info_col)s} Reason: \x196}%(reason)s\x19%(info_col)s}') % {'reason': reason.text, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}
         self._text_buffer.add_message(kick_msg)
 
     def on_user_kicked(self, presence, user, from_nick):
@@ -1161,20 +1161,20 @@ class MucTab(ChatTab):
             self.refresh_tab_win()
             self.core.doupdate()
             if by:
-                kick_msg = _('\x191}%(spec)s \x193}You\x195} have been kicked by \x193}%(by)s') % {'spec': get_theme().CHAR_KICK, 'by':by}
+                kick_msg = _('\x191}%(spec)s \x193}You\x19%(info_col)s} have been kicked by \x193}%(by)s') % {'spec': get_theme().CHAR_KICK, 'by':by, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}
             else:
-                kick_msg = _('\x191}%(spec)s \x193}You\x195} have been kicked.') % {'spec':get_theme().CHAR_KICK}
+                kick_msg = _('\x191}%(spec)s \x193}You\x19%(info_col)s} have been kicked.') % {'spec':get_theme().CHAR_KICK, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}
             # try to auto-rejoin
             if config.get_by_tabname('autorejoin', 'false', self.general_jid, True) == 'true':
                 muc.join_groupchat(self.core.xmpp, self.name, self.own_nick)
         else:
             color = user.color[0] if config.get_by_tabname('display_user_color_in_join_part', '', self.general_jid, True) == 'true' else 3
             if by:
-                kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x195} has been kicked by \x193}%(by)s') % {'spec':get_theme().CHAR_KICK, 'nick':from_nick, 'color':color, 'by':by}
+                kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x19%(info_col)s} has been kicked by \x193}%(by)s') % {'spec':get_theme().CHAR_KICK, 'nick':from_nick, 'color':color, 'by':by, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}
             else:
-                kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x195} has been kicked') % {'spec':get_theme().CHAR_KICK, 'nick':from_nick, 'color':color}
+                kick_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x19%(info_col)s} has been kicked') % {'spec':get_theme().CHAR_KICK, 'nick':from_nick, 'color':color, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}
         if reason is not None and reason.text:
-            kick_msg += _('\x195} Reason: \x196}%(reason)s') % {'reason': reason.text}
+            kick_msg += _('\x19%(info_col)s} Reason: \x196}%(reason)s') % {'reason': reason.text, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}
         self.add_message(kick_msg)
 
     def on_user_leave_groupchat(self, user, jid, status, from_nick, from_room):
@@ -1192,9 +1192,9 @@ class MucTab(ChatTab):
         if hide_exit_join == -1 or user.has_talked_since(hide_exit_join):
             color = user.color[0] if config.get_by_tabname('display_user_color_in_join_part', '', self.general_jid, True) == 'true' else 3
             if not jid.full:
-                leave_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x195} has left the room') % {'nick':from_nick, 'color':color, 'spec':get_theme().CHAR_QUIT}
+                leave_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x19%(info_col)s} has left the room') % {'nick':from_nick, 'color':color, 'spec':get_theme().CHAR_QUIT, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}
             else:
-                leave_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x195} (\x194}%(jid)s\x195}) has left the room') % {'spec':get_theme().CHAR_QUIT, 'nick':from_nick, 'color':color, 'jid':jid.full}
+                leave_msg = _('\x191}%(spec)s \x19%(color)d}%(nick)s\x19%(info_col)s} (\x194}%(jid)s\x19%(info_col)s}) has left the room') % {'spec':get_theme().CHAR_QUIT, 'nick':from_nick, 'color':color, 'jid':jid.full, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}
             if status:
                 leave_msg += ' (%s)' % status
             self.add_message(leave_msg)
@@ -1210,9 +1210,9 @@ class MucTab(ChatTab):
         # to be displayed has changed
         color = user.color[0] if config.get_by_tabname('display_user_color_in_join_part', '', self.general_jid, True) == 'true' else 3
         if from_nick == self.own_nick:
-            msg = _('\x193}You\x195} changed: ')
+            msg = _('\x193}You\x19%(info_col)s} changed: ') % {'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}
         else:
-            msg = _('\x19%(color)d}%(nick)s\x195} changed: ') % {'nick': from_nick, 'color': color}
+            msg = _('\x19%(color)d}%(nick)s\x19%(info_col)s} changed: ') % {'nick': from_nick, 'color': color, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}
         if show not in SHOW_NAME:
             self.core.information("%s from room %s sent an invalid show: %s" %\
                                       (from_nick, from_room, show), "warning")
@@ -1334,7 +1334,7 @@ class MucTab(ChatTab):
         nick_color = nick_color or None
         highlight = False
         if (not nickname or time) and not txt.startswith('/me '):
-            txt = '\x195}%s' % (txt,)
+            txt = '\x19%(info_col)s}%(txt)s' % {'txt':txt, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}
         else:                   # TODO
             highlight = self.do_highlight(txt, time, nickname)
             if highlight:
@@ -1508,7 +1508,7 @@ class PrivateTab(ChatTab):
         The user changed her nick in the corresponding muc: update the tab’s name and
         display a message.
         """
-        self.add_message('\x193}%(old)s\x195} is now known as \x193}%(new)s' % {'old':old_nick, 'new':new_nick})
+        self.add_message('\x193}%(old)s\x19%(info_col)s} is now known as \x193}%(new)s' % {'old':old_nick, 'new':new_nick, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]})
         new_jid = JID(self.name).bare+'/'+new_nick
         self.name = new_jid
 
@@ -1518,9 +1518,9 @@ class PrivateTab(ChatTab):
         """
         self.deactivate()
         if not status_message:
-            self.add_message(_('\x191}%(spec)s \x193}%(nick)s\x195} has left the room') % {'nick':from_nick, 'spec':get_theme().CHAR_QUIT})
+            self.add_message(_('\x191}%(spec)s \x193}%(nick)s\x19%(info_col)s} has left the room') % {'nick':from_nick, 'spec':get_theme().CHAR_QUIT, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]})
         else:
-            self.add_message(_('\x191}%(spec)s \x193}%(nick)s\x195} has left the room (%(status)s)"') % {'nick':from_nick, 'spec':get_theme().CHAR_QUIT, 'status': status_message})
+            self.add_message(_('\x191}%(spec)s \x193}%(nick)s\x19%(info_col)s} has left the room (%(status)s)"') % {'nick':from_nick, 'spec':get_theme().CHAR_QUIT, 'status': status_message, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]})
         if self.core.current_tab() is self:
             self.refresh()
             self.core.doupdate()
@@ -1536,7 +1536,7 @@ class PrivateTab(ChatTab):
             user = tab.get_user_by_name(nick)
             if user:
                 color = user.color[0]
-        self.add_message('\x194}%(spec)s \x19%(color)d}%(nick)s\x195} joined the room' % {'nick':nick, 'color': color, 'spec':get_theme().CHAR_JOIN})
+        self.add_message('\x194}%(spec)s \x19%(color)d}%(nick)s\x19%(info_col)s} joined the room' % {'nick':nick, 'color': color, 'spec':get_theme().CHAR_JOIN, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]})
         if self.core.current_tab() is self:
             self.refresh()
             self.core.doupdate()
@@ -2155,7 +2155,7 @@ class ConversationTab(ChatTab):
         else:
             resource = contact.get_highest_priority_resource()
         if resource:
-            self._text_buffer.add_message("\x195}Status: %s\x193}" %resource.status, None, None, None, None, None)
+            self._text_buffer.add_message("\x19%(info_col)s}Status: %(status)s\x193}" % {'status': resource.status, 'info_col': get_theme().COLOR_INFORMATION_TEXT[0]}, None, None, None, None, None)
             self.refresh()
             self.core.doupdate()
 
diff --git a/src/text_buffer.py b/src/text_buffer.py
index f396bd5b..9b717882 100644
--- a/src/text_buffer.py
+++ b/src/text_buffer.py
@@ -16,6 +16,7 @@ import collections
 
 from datetime import datetime
 from config import config
+from theming import get_theme
 
 Message = collections.namedtuple('Message', 'txt nick_color time str_time nickname user')
 
@@ -44,7 +45,7 @@ class TextBuffer(object):
             else:
                 color = None
             # TODO: display the bg color too.
-            txt = ("\x195}* \x19%s}" % (color or 5,))+ nickname + ' \x195}' + txt[4:]
+            txt = ("\x19%(info_col)s}* \x19%(col)s}" % {'col':color or 5, 'info_col':get_theme().COLOR_INFORMATION_TEXT[0]})+ nickname + ' \x19%(info_col)s}' % {'info_col':get_theme().COLOR_INFORMATION_TEXT[0]} + txt[4:]
             nickname = None
         msg = Message(txt='%s\x19o'%(txt.replace('\t', '    '),), nick_color=nick_color,
                       time=time, str_time=time.strftime("%Y-%m-%d %H:%M:%S")\
diff --git a/src/theming.py b/src/theming.py
index 4e1f37ea..ff2ff74e 100644
--- a/src/theming.py
+++ b/src/theming.py
@@ -79,7 +79,7 @@ class Theme(object):
     """
     # Message text color
     COLOR_NORMAL_TEXT = (-1, -1)
-    COLOR_INFORMATION_TEXT = (137, -1) # TODO
+    COLOR_INFORMATION_TEXT = (5, -1) # TODO
     COLOR_HIGHLIGHT_NICK = (3, 5, 'b')
 
     # User list color
-- 
cgit v1.2.3


From fc6db466f518b8b54f3ce23695d63c77b09e3891 Mon Sep 17 00:00:00 2001
From: Florent Le Coz <louiz@louiz.org>
Date: Tue, 24 Jan 2012 17:07:04 +0100
Subject: New participant list. Displaying the nick color, the affiliation,
 role, chatstate and status!

---
 src/tabs.py    |  1 +
 src/windows.py | 49 ++++++++++++++++++++++++++++++-------------------
 2 files changed, 31 insertions(+), 19 deletions(-)

(limited to 'src')

diff --git a/src/tabs.py b/src/tabs.py
index 53405db4..019fae30 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -678,6 +678,7 @@ class MucTab(ChatTab):
         for i, user in enumerate(sorted_users):
             user.color = colors[i % len(colors)]
         self.text_win.rebuild_everything(self._text_buffer)
+        self.user_win.refresh(self.users)
         self.text_win.refresh()
         self.input.refresh()
 
diff --git a/src/windows.py b/src/windows.py
index 1b833290..7d9194b2 100644
--- a/src/windows.py
+++ b/src/windows.py
@@ -196,6 +196,10 @@ class UserList(Win):
                 'none': lambda: get_theme().COLOR_USER_NONE,
                 '': lambda: get_theme().COLOR_USER_NONE
                }
+        self.symbol_affiliation = {'owner': '~',
+                                    'admin': '&',
+                                    'member': '+',
+                                    'none': '-'}
         self.color_show = {'xa': lambda: get_theme().COLOR_STATUS_XA,
                 'none': lambda: get_theme().COLOR_STATUS_NONE,
                 '': lambda: get_theme().COLOR_STATUS_NONE,
@@ -203,7 +207,6 @@ class UserList(Win):
                 'away': lambda: get_theme().COLOR_STATUS_AWAY,
                 'chat': lambda: get_theme().COLOR_STATUS_CHAT
                }
-
     def scroll_up(self):
         self.pos += self.height-1
 
@@ -224,24 +227,9 @@ class UserList(Win):
             if self.pos >= len(users) and self.pos != 0:
                 self.pos = len(users)-1
             for user in users[self.pos:]:
-                if not user.role in self.color_role:
-                    role_col = get_theme().COLOR_USER_NONE
-                else:
-                    role_col = self.color_role[user.role]()
-                if not user.show in self.color_show:
-                    show_col = get_theme().COLOR_STATUS_NONE
-                else:
-                    show_col = self.color_show[user.show]()
-                if user.chatstate == 'composing':
-                    char = get_theme().CHAR_CHATSTATE_COMPOSING
-                elif user.chatstate == 'active':
-                    char = get_theme().CHAR_CHATSTATE_ACTIVE
-                elif user.chatstate == 'paused':
-                    char = get_theme().CHAR_CHATSTATE_PAUSED
-                else:
-                    char = get_theme().CHAR_STATUS
-                self.addstr(y, 0, char, to_curses_attr(show_col))
-                self.addstr(y, 1, user.nick[:self.width-2], to_curses_attr(role_col))
+                self.draw_role_affiliation(y, user)
+                self.draw_status_chatstate(y, user)
+                self.addstr(y, 2, user.nick[:self.width-2], to_curses_attr(user.color))
                 y += 1
                 if y == self.height:
                     break
@@ -252,6 +240,29 @@ class UserList(Win):
                 self.draw_plus(self.height-1)
             self._refresh()
 
+    def draw_role_affiliation(self, y, user):
+        if not user.role in self.color_role:
+            color = get_theme().COLOR_USER_NONE
+        else:
+            color = self.color_role[user.role]()
+        symbol = self.symbol_affiliation.get(user.affiliation) or '-'
+        self.addstr(y, 1, symbol, to_curses_attr(color))
+
+    def draw_status_chatstate(self, y, user):
+        if not user.show in self.color_show:
+            show_col = get_theme().COLOR_STATUS_NONE
+        else:
+            show_col = self.color_show[user.show]()
+        if user.chatstate == 'composing':
+            char = get_theme().CHAR_CHATSTATE_COMPOSING
+        elif user.chatstate == 'active':
+            char = get_theme().CHAR_CHATSTATE_ACTIVE
+        elif user.chatstate == 'paused':
+            char = get_theme().CHAR_CHATSTATE_PAUSED
+        else:
+            char = get_theme().CHAR_STATUS
+        self.addstr(y, 0, char, to_curses_attr(show_col))
+
     def resize(self, height, width, y, x):
         with g_lock:
             self._resize(height, width, y, x)
-- 
cgit v1.2.3


From d6989db025b2f391c2d309ae9a4c4c8c094e764c Mon Sep 17 00:00:00 2001
From: Florent Le Coz <louiz@louiz.org>
Date: Tue, 24 Jan 2012 17:45:17 +0100
Subject: Make /theme error more descriptive

---
 src/theming.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

(limited to 'src')

diff --git a/src/theming.py b/src/theming.py
index ff2ff74e..96c47337 100644
--- a/src/theming.py
+++ b/src/theming.py
@@ -271,8 +271,8 @@ def reload_theme():
         file_path  = os.path.join(themes_dir, theme_name)+'.py'
         log.debug('Theme file to load: %s' %(file_path,))
         new_theme = imp.load_source('theme', os.path.join(themes_dir, theme_name)+'.py')
-    except:
-        return 'Theme not found'
+    except Exception as e:
+        return 'Failed to load theme: %s' % (e,)
     theme = new_theme.theme
 
 if __name__ == '__main__':
-- 
cgit v1.2.3


From a3344b11b2ea7ac17097b8db4b703bb9f6c02912 Mon Sep 17 00:00:00 2001
From: Florent Le Coz <louiz@louiz.org>
Date: Thu, 26 Jan 2012 10:04:42 +0100
Subject: Handle autojoin="1" for bookmarks.

---
 src/bookmark.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'src')

diff --git a/src/bookmark.py b/src/bookmark.py
index 45616d93..b63ddbdf 100644
--- a/src/bookmark.py
+++ b/src/bookmark.py
@@ -78,7 +78,7 @@ class Bookmark(object):
         """
         jid = el.get('jid')
         name = el.get('name')
-        autojoin = True if el.get('autojoin', False) == 'true' else False
+        autojoin = True if el.get('autojoin', 'false').lower() in ('true', '1') else False
         nick = None
         for n in iter(el, 'nick'):
             nick = nick.text
-- 
cgit v1.2.3


From bbcd9c631ca6b47ad07eedc3ded9909045d6cc9e Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Thu, 26 Jan 2012 17:21:13 +0100
Subject: Add an auto_reconnect option

---
 src/connection.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'src')

diff --git a/src/connection.py b/src/connection.py
index 352a1d5b..1079b5a6 100644
--- a/src/connection.py
+++ b/src/connection.py
@@ -42,7 +42,7 @@ class Connection(sleekxmpp.ClientXMPP):
             password = None
         sleekxmpp.ClientXMPP.__init__(self, jid, password, ssl=True)
         self.core = None
-        self.auto_reconnect = False
+        self.auto_reconnect = True if config.get('auto_reconnect', 'false').lower() in ('true', '1') else False
         self.auto_authorize = None
         self.register_plugin('xep_0030')
         self.register_plugin('xep_0004')
-- 
cgit v1.2.3


From d77e79cd6bc98aa588ac7112dada67ad0df74e68 Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Thu, 26 Jan 2012 21:23:14 +0100
Subject: Fixes #2312 (delay element in normal conversations)

---
 src/core.py | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

(limited to 'src')

diff --git a/src/core.py b/src/core.py
index 1023c748..28ccebba 100644
--- a/src/core.py
+++ b/src/core.py
@@ -768,20 +768,25 @@ class Core(object):
         """
         jid = message['from']
         body = xhtml.get_body_from_message_stanza(message)
-        conversation = self.get_tab_of_conversation_with_jid(jid, create=False)
         if not body:
             if message['type'] == 'error':
                 self.information(self.get_error_message_from_error_stanza(message), 'Error')
             return
         conversation = self.get_tab_of_conversation_with_jid(jid, create=True)
         self.events.trigger('conversation_msg', message, conversation)
-        body = xhtml.get_body_from_message_stanza(message)
         if roster.get_contact_by_jid(jid.bare):
             remote_nick = roster.get_contact_by_jid(jid.bare).name or jid.user
         else:
             remote_nick = jid.user
-        conversation._text_buffer.add_message(body, nickname=remote_nick, nick_color=get_theme().COLOR_REMOTE_USER)
-        if conversation.remote_wants_chatstates is None:
+        delay_tag = message.find('{urn:xmpp:delay}delay')
+        if delay_tag is not None:
+            delayed = True
+            date = common.datetime_tuple(delay_tag.attrib['stamp'])
+        else:
+            delayed = False
+            date = None
+        conversation._text_buffer.add_message(body, date, nickname=remote_nick, nick_color=get_theme().COLOR_REMOTE_USER, history=delayed)
+        if conversation.remote_wants_chatstates is None and not delayed:
             if message['chat_state']:
                 conversation.remote_wants_chatstates = True
             else:
-- 
cgit v1.2.3


From e8b0c302fe59b9f1f9ea0b7c6190fb62a62ab010 Mon Sep 17 00:00:00 2001
From: Florent Le Coz <louiz@louiz.org>
Date: Fri, 27 Jan 2012 18:26:33 +0100
Subject: Typo.

---
 src/core.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'src')

diff --git a/src/core.py b/src/core.py
index 1023c748..2fdc97aa 100644
--- a/src/core.py
+++ b/src/core.py
@@ -60,7 +60,7 @@ from daemon import Executor
 ERROR_AND_STATUS_CODES = {
     '401': _('A password is required'),
     '403': _('Permission denied'),
-    '404': _('The room does\'nt exist'),
+    '404': _('The room doesn’t exist'),
     '405': _('Your are not allowed to create a new room'),
     '406': _('A reserved nick must be used'),
     '407': _('You are not in the member list'),
-- 
cgit v1.2.3


From c9c066ae336fd315d87de39894f06ce3aca3a2fc Mon Sep 17 00:00:00 2001
From: Florent Le Coz <louiz@louiz.org>
Date: Fri, 27 Jan 2012 19:13:04 +0100
Subject: [Link Mauve] Add the ability to send and receive Attention
 (XEP-0224).

---
 src/connection.py |  1 +
 src/core.py       | 20 ++++++++++++++++++
 src/tabs.py       | 62 +++++++++++++++++++++++++++++++++++++++++++++++++------
 src/theming.py    |  6 ++++--
 4 files changed, 81 insertions(+), 8 deletions(-)

(limited to 'src')

diff --git a/src/connection.py b/src/connection.py
index 1079b5a6..74a21cdb 100644
--- a/src/connection.py
+++ b/src/connection.py
@@ -59,6 +59,7 @@ class Connection(sleekxmpp.ClientXMPP):
             self.register_plugin('xep_0092', pconfig=info)
         if config.get('send_time', 'true') == 'true':
             self.register_plugin('xep_0202')
+        self.register_plugin('xep_0224')
 
     def start(self):
         # TODO, try multiple servers
diff --git a/src/core.py b/src/core.py
index 9b74c1a5..4364734e 100644
--- a/src/core.py
+++ b/src/core.py
@@ -194,6 +194,7 @@ class Core(object):
         self.xmpp.add_event_handler("chatstate_paused", self.on_chatstate_paused)
         self.xmpp.add_event_handler("chatstate_gone", self.on_chatstate_gone)
         self.xmpp.add_event_handler("chatstate_inactive", self.on_chatstate_inactive)
+        self.xmpp.add_event_handler("attention", self.on_attention)
         self.xmpp.register_handler(Callback('ALL THE STANZAS', connection.MatchAll(None), self.incoming_stanza))
 
         self.timed_events = set()
@@ -503,6 +504,21 @@ class Core(object):
             tab.input.refresh()
             self.doupdate()
 
+    def on_attention(self, message):
+        jid_from = message['from']
+        self.information('%s requests your attention!' % jid_from, 'Info')
+        for tab in self.tabs:
+            if tab.get_name() == jid_from:
+                tab.state = 'attention'
+                self.refresh_tab_win()
+                return
+        for tab in self.tabs:
+            if tab.get_name() == jid_from.bare:
+                tab.state = 'attention'
+                self.refresh_tab_win()
+                return
+        self.information('%s tab not found.' % jid_from, 'Error')
+
     def open_new_form(self, form, on_cancel, on_send, **kwargs):
         """
         Open a new tab containing the form
@@ -1075,6 +1091,10 @@ class Core(object):
         - A Muc with an highlight
         - A Muc with any new message
         """
+        for tab in self.tabs:
+           if tab.state == 'attention':
+               self.command_win('%s' % tab.nb)
+               return
         for tab in self.tabs:
             if tab.state == 'private':
                 self.command_win('%s' % tab.nb)
diff --git a/src/tabs.py b/src/tabs.py
index 7df200b0..9bc6e24c 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -67,7 +67,7 @@ STATE_COLORS = {
         'private': lambda: get_theme().COLOR_TAB_PRIVATE,
         'normal': lambda: get_theme().COLOR_TAB_NORMAL,
         'current': lambda: get_theme().COLOR_TAB_CURRENT,
-#        'attention': lambda: get_theme().COLOR_TAB_ATTENTION,
+        'attention': lambda: get_theme().COLOR_TAB_ATTENTION,
     }
 
 VERTICAL_STATE_COLORS = {
@@ -77,7 +77,7 @@ VERTICAL_STATE_COLORS = {
         'private': lambda: get_theme().COLOR_VERTICAL_TAB_PRIVATE,
         'normal': lambda: get_theme().COLOR_VERTICAL_TAB_NORMAL,
         'current': lambda: get_theme().COLOR_VERTICAL_TAB_CURRENT,
-#        'attention': lambda: get_theme().COLOR_VERTICAL_TAB_ATTENTION,
+        'attention': lambda: get_theme().COLOR_VERTICAL_TAB_ATTENTION,
     }
 
 
@@ -88,7 +88,7 @@ STATE_PRIORITY = {
         'message': 1,
         'highlight': 2,
         'private': 2,
-#        'attention': 3
+        'attention': 3
     }
 
 class Tab(object):
@@ -156,7 +156,7 @@ class Tab(object):
             log.debug("Invalid value for tab state: %s", value)
         elif STATE_PRIORITY[value] < STATE_PRIORITY[self._state] and \
                 value != 'current':
-            log.debug("Did not set status because of lower priority, asked: %s, kept: %s", (value, self.state))
+            log.debug("Did not set status because of lower priority, asked: %s, kept: %s", (value, self._state))
         else:
             self._state = value
 
@@ -349,6 +349,7 @@ class ChatTab(Tab):
         # if that’s None, then no paused chatstate was sent recently
         # if that’s a weakref returning None, then a paused chatstate was sent
         # since the last input
+        self.remote_supports_attention = False
         self.key_func['M-v'] = self.move_separator
         self.key_func['M-/'] = self.last_words_completion
         self.key_func['^M'] = self.on_enter
@@ -495,6 +496,7 @@ class ChatTab(Tab):
     def command_say(self, line):
         raise NotImplementedError
 
+
 class MucTab(ChatTab):
     """
     The tab containing a multi-user-chat room.
@@ -1359,6 +1361,7 @@ class PrivateTab(ChatTab):
         self._text_buffer.add_window(self.text_win)
         self.info_header = windows.PrivateInfoWin()
         self.input = windows.MessageInput()
+        self.check_attention()
         # keys
         self.key_func['^I'] = self.completion
         # commands
@@ -1379,7 +1382,7 @@ class PrivateTab(ChatTab):
     def completion(self):
         self.complete_commands(self.input)
 
-    def command_say(self, line):
+    def command_say(self, line, attention=False):
         if not self.on:
             return
         msg = self.core.xmpp.make_message(self.get_name())
@@ -1396,12 +1399,35 @@ class PrivateTab(ChatTab):
         if config.get_by_tabname('send_chat_states', 'true', self.general_jid, True) == 'true' and self.remote_wants_chatstates is not False:
             needed = 'inactive' if self.core.status.show in ('xa', 'away') else 'active'
             msg['chat_state'] = needed
+        if attention and self.remote_supports_attention:
+            msg['attention'] = True
         self.core.events.trigger('private_say_after', msg, self)
         msg.send()
         self.cancel_paused_delay()
         self.text_win.refresh()
         self.input.refresh()
 
+    def command_attention(self, message=''):
+        if message is not '':
+            self.command_say(message, attention=True)
+        else:
+            msg = self.core.xmpp.make_message(self.get_name())
+            msg['type'] = 'chat'
+            msg['attention'] = True
+            msg.send()
+
+    def check_attention(self):
+        self.core.xmpp.plugin['xep_0030'].get_info(jid=self.get_name(), block=False, timeout=5, callback=self.on_attention_checked)
+
+    def on_attention_checked(self, iq):
+        if 'urn:xmpp:attention:0' in iq['disco_info'].get_features():
+            self.core.information('Attention is supported!', 'Info')
+            self.remote_supports_attention = True
+            self.commands['attention'] =  (self.command_attention, _('Usage: /attention [message]\nAttention: Require the attention of the contact. Can also send a message along with the attention.'), None)
+        else:
+            self.core.information('Attention is not supported. :(', 'Info')
+            self.remote_supports_attention = False
+
     def command_unquery(self, arg):
         """
         /unquery
@@ -2096,6 +2122,7 @@ class ConversationTab(ChatTab):
         self.upper_bar = windows.ConversationStatusMessageWin()
         self.info_header = windows.ConversationInfoWin()
         self.input = windows.MessageInput()
+        self.check_attention()
         # keys
         self.key_func['^I'] = self.completion
         # commands
@@ -2125,7 +2152,7 @@ class ConversationTab(ChatTab):
     def completion(self):
         self.complete_commands(self.input)
 
-    def command_say(self, line):
+    def command_say(self, line, attention=False):
         msg = self.core.xmpp.make_message(self.get_name())
         msg['type'] = 'chat'
         msg['body'] = line
@@ -2141,6 +2168,8 @@ class ConversationTab(ChatTab):
         if config.get_by_tabname('send_chat_states', 'true', self.general_jid, True) == 'true' and self.remote_wants_chatstates is not False:
             needed = 'inactive' if self.core.status.show in ('xa', 'away') else 'active'
             msg['chat_state'] = needed
+        if attention and self.remote_supports_attention:
+            msg['attention'] = True
         self.core.events.trigger('conversation_say_after', msg, self)
         msg.send()
         logger.log_message(JID(self.get_name()).bare, self.core.own_nick, line)
@@ -2160,6 +2189,27 @@ class ConversationTab(ChatTab):
             self.refresh()
             self.core.doupdate()
 
+    def command_attention(self, message=''):
+        if message is not '':
+            self.command_say(message, attention=True)
+        else:
+            msg = self.core.xmpp.make_message(self.get_name())
+            msg['type'] = 'chat'
+            msg['attention'] = True
+            msg.send()
+
+    def check_attention(self):
+        self.core.xmpp.plugin['xep_0030'].get_info(jid=self.get_name(), block=False, timeout=5, callback=self.on_attention_checked)
+
+    def on_attention_checked(self, iq):
+        if 'urn:xmpp:attention:0' in iq['disco_info'].get_features():
+            self.core.information('Attention is supported!', 'Info')
+            self.remote_supports_attention = True
+            self.commands['attention'] =  (self.command_attention, _('Usage: /attention [message]\nAttention: Require the attention of the contact. Can also send a message along with the attention.'), None)
+        else:
+            self.core.information('Attention is not supported. :(', 'Info')
+            self.remote_supports_attention = False
+
     def command_unquery(self, arg):
         self.core.close_tab()
 
diff --git a/src/theming.py b/src/theming.py
index 25188f81..fba33366 100644
--- a/src/theming.py
+++ b/src/theming.py
@@ -124,15 +124,17 @@ class Theme(object):
     COLOR_TAB_NORMAL = (7, 4)
     COLOR_TAB_CURRENT = (7, 6)
     COLOR_TAB_NEW_MESSAGE = (7, 5)
-    COLOR_TAB_HIGHLIGHT = (7, 1)
+    COLOR_TAB_HIGHLIGHT = (7, 3)
     COLOR_TAB_PRIVATE = (7, 2)
+    COLOR_TAB_ATTENTION = (7, 1)
     COLOR_TAB_DISCONNECTED = (7, 8)
 
     COLOR_VERTICAL_TAB_NORMAL = (4, -1)
     COLOR_VERTICAL_TAB_CURRENT = (7, 4)
     COLOR_VERTICAL_TAB_NEW_MESSAGE = (5, -1)
-    COLOR_VERTICAL_TAB_HIGHLIGHT = (1, -1)
+    COLOR_VERTICAL_TAB_HIGHLIGHT = (3, -1)
     COLOR_VERTICAL_TAB_PRIVATE = (2, -1)
+    COLOR_VERTICAL_TAB_ATTENTION = (1, -1)
     COLOR_VERTICAL_TAB_DISCONNECTED = (8, -1)
 
     # Nickname colors
-- 
cgit v1.2.3


From e74a640fdddf9d18c8692db7f759582a57386e88 Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Mon, 30 Jan 2012 01:10:41 +0100
Subject: Do not crash when given a link without text (e.g <a
 href="example.com"/>)

---
 src/xhtml.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

(limited to 'src')

diff --git a/src/xhtml.py b/src/xhtml.py
index e7a045fa..37da5f5d 100644
--- a/src/xhtml.py
+++ b/src/xhtml.py
@@ -288,9 +288,9 @@ def xhtml_to_poezio_colors(text):
     for elem in elems:
         if elem.tag == '{http://www.w3.org/1999/xhtml}a':
             if 'href' in elem.attrib and elem.attrib['href'] != elem.text:
-                message += '\x19u%s\x19o (%s)' % (trim(elem.attrib['href']), trim(elem.text))
+                message += '\x19u%s\x19o (%s)' % (trim(elem.attrib['href']), trim(elem.text if elem.text else ""))
             else:
-                message += '\x19u' + elem.text + '\x19o'
+                message += '\x19u' + (elem.text if elem.text else "") + '\x19o'
         elif elem.tag == '{http://www.w3.org/1999/xhtml}blockquote':
             message += '“'
         elif elem.tag == '{http://www.w3.org/1999/xhtml}body':
-- 
cgit v1.2.3


From 5593fabe6c79dd5c6b44a48c004ea71e789d53aa Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Sun, 5 Feb 2012 16:38:40 +0100
Subject: Log the private messages sent

---
 src/tabs.py | 1 +
 1 file changed, 1 insertion(+)

(limited to 'src')

diff --git a/src/tabs.py b/src/tabs.py
index 9bc6e24c..e0c71758 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -1388,6 +1388,7 @@ class PrivateTab(ChatTab):
         msg = self.core.xmpp.make_message(self.get_name())
         msg['type'] = 'chat'
         msg['body'] = line
+        logger.log_message(self.get_name().replace('/', '\\'), self.own_nick, line)
         # trigger the event BEFORE looking for colors.
         # This lets a plugin insert \x19xxx} colors, that will
         # be converted in xhtml.
-- 
cgit v1.2.3


From 93c17f8ece0681592aeacd2ac541ba7d2194ebf5 Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Tue, 7 Feb 2012 21:47:40 +0100
Subject: Fixed #2328

---
 src/xhtml.py | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

(limited to 'src')

diff --git a/src/xhtml.py b/src/xhtml.py
index 37da5f5d..cf7a5fc0 100644
--- a/src/xhtml.py
+++ b/src/xhtml.py
@@ -194,7 +194,9 @@ def get_body_from_message_stanza(message):
     if config.get('enable_xhtml_im', 'true') == 'true':
         xhtml_body = message['xhtml_im']
         if xhtml_body:
-            return xhtml_to_poezio_colors(xhtml_body)
+            content = xhtml_to_poezio_colors(xhtml_body)
+            content = content if content else message['body']
+            return content or " "
     return message['body']
 
 def ncurses_color_to_html(color):
-- 
cgit v1.2.3


From 42c8deb9e19bde492c3f17d8f07f5b277eee4d04 Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Thu, 9 Feb 2012 11:08:14 +0100
Subject: Remove some debugging prints

---
 src/tabs.py | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

(limited to 'src')

diff --git a/src/tabs.py b/src/tabs.py
index e0c71758..4cf5c479 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -1422,11 +1422,10 @@ class PrivateTab(ChatTab):
 
     def on_attention_checked(self, iq):
         if 'urn:xmpp:attention:0' in iq['disco_info'].get_features():
-            self.core.information('Attention is supported!', 'Info')
+            self.core.information('Attention is supported', 'Info')
             self.remote_supports_attention = True
             self.commands['attention'] =  (self.command_attention, _('Usage: /attention [message]\nAttention: Require the attention of the contact. Can also send a message along with the attention.'), None)
         else:
-            self.core.information('Attention is not supported. :(', 'Info')
             self.remote_supports_attention = False
 
     def command_unquery(self, arg):
@@ -2204,11 +2203,10 @@ class ConversationTab(ChatTab):
 
     def on_attention_checked(self, iq):
         if 'urn:xmpp:attention:0' in iq['disco_info'].get_features():
-            self.core.information('Attention is supported!', 'Info')
+            self.core.information('Attention is supported', 'Info')
             self.remote_supports_attention = True
             self.commands['attention'] =  (self.command_attention, _('Usage: /attention [message]\nAttention: Require the attention of the contact. Can also send a message along with the attention.'), None)
         else:
-            self.core.information('Attention is not supported. :(', 'Info')
             self.remote_supports_attention = False
 
     def command_unquery(self, arg):
-- 
cgit v1.2.3


From 6df406b7c1c06b8c7c58cd74d028310008cea6a4 Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Thu, 9 Feb 2012 15:45:18 +0100
Subject: Config.write_in_file() now works as expected (can add sections, add
 inexistant options, and edit in place)

---
 src/config.py | 27 +++++++++++++++++++++------
 1 file changed, 21 insertions(+), 6 deletions(-)

(limited to 'src')

diff --git a/src/config.py b/src/config.py
index af9c9fbe..2ee4abb1 100644
--- a/src/config.py
+++ b/src/config.py
@@ -113,16 +113,31 @@ class Config(RawConfigParser):
         df.close()
         result_lines = []
         we_are_in_the_right_section = False
+        written = False
+        section_found = False
         for line in lines_before:
             if line.startswith('['): # check the section
+                if we_are_in_the_right_section and not written:
+                    result_lines.append('%s= %s' % (option, value))
                 if line == '[%s]' % section:
                     we_are_in_the_right_section = True
+                    section_found = True
                 else:
                     we_are_in_the_right_section = False
             if (line.startswith('%s ' % (option,)) or
-                line.startswith('%s=' % (option,))) and we_are_in_the_right_section:
-                line = '%s = %s' % (option, value)
+                line.startswith('%s=' % (option,)) or
+                line.startswith('%s = ' % (option,))) and we_are_in_the_right_section:
+                line = '%s= %s' % (option, value)
+                written = True
             result_lines.append(line)
+
+        if not section_found:
+            result_lines.append('[%s]' % section)
+            result_lines.append('%s= %s' % (option, value))
+        elif not written:
+            result_lines.append('%s= %s' % (option, value))
+
+
         df = open(self.file_name, 'w')
         for line in result_lines:
             df.write('%s\n' % line)
@@ -133,11 +148,11 @@ class Config(RawConfigParser):
         set the value in the configuration then save it
         to the file
         """
-        try:
+        if self.has_section(section):
+            RawConfigParser.set(self, section, option, value)
+        else:
+            self.add_section(section)
             RawConfigParser.set(self, section, option, value)
-        except NoSectionError:
-            # TODO, add this section if it didn't exist
-            return
         self.write_in_file(section, option, value)
 
     def set(self, option, value, section=DEFSECTION):
-- 
cgit v1.2.3


From f96c895a8dcc32f8f90a3a90fee55913e0b49312 Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Thu, 9 Feb 2012 16:13:45 +0100
Subject: New unified /set command (fixes #2323)

---
 src/core.py | 86 ++++++++++++++++---------------------------------------------
 1 file changed, 22 insertions(+), 64 deletions(-)

(limited to 'src')

diff --git a/src/core.py b/src/core.py
index 4364734e..95ad22af 100644
--- a/src/core.py
+++ b/src/core.py
@@ -128,7 +128,7 @@ class Core(object):
             'status': (self.command_status, _('Usage: /status <availability> [status message]\nStatus: Sets your availability and (optionally) your status message. The <availability> argument is one of \"available, chat, away, afk, dnd, busy, xa\" and the optional [status message] argument will be your status message.'), self.completion_status),
             'bookmark_local': (self.command_bookmark_local, _("Usage: /bookmark_local [roomname][/nick]\nBookmark Local: Bookmark locally the specified room (you will then auto-join it on each poezio start). This commands uses almost the same syntaxe as /join. Type /help join for syntaxe examples. Note that when typing \"/bookmark\" on its own, the room will be bookmarked with the nickname you\'re currently using in this room (instead of default_nick)"), self.completion_bookmark_local),
             'bookmark': (self.command_bookmark, _("Usage: /bookmark [roomname][/nick] [autojoin] [password]\nBookmark: Bookmark online the specified room (you will then auto-join it on each poezio start if autojoin is specified and is 'true'). This commands uses almost the same syntaxe as /join. Type /help join for syntaxe examples. Note that when typing \"/bookmark\" on its own, the room will be bookmarked with the nickname you\'re currently using in this room (instead of default_nick)"), self.completion_bookmark),
-            'set': (self.command_set, _("Usage: /set <option> [value]\nSet: Set the value of the option in your configuration file. You can, for example, change your default nickname by doing `/set default_nick toto` or your resource with `/set resource blabla`. You can also set an empty value (nothing) by providing no [value] after <option>."), self.completion_set),
+            'set': (self.command_set, _("Usage: /set [plugin|][section] <option> [value]\nSet: Set the value of an option in your configuration file. You can, for example, change your default nickname by doing `/set default_nick toto` or your resource with `/set resource blabla`. You can also set options in specific sections with `/set bindings M-i ^i` or in specific plugin with `/set mpd_client| host 127.0.0.1`"), None),
             'theme': (self.command_theme, _('Usage: /theme [theme_name]\nTheme: Reload the theme defined in the config file. If theme_name is provided, set that theme before reloading it.'), self.completion_theme),
             'list': (self.command_list, _('Usage: /list\nList: Get the list of public chatrooms on the specified server.'), self.completion_list),
             'message': (self.command_message, _('Usage: /message <jid> [optional message]\nMessage: Open a conversation with the specified JID (even if it is not in our roster), and send a message to it, if the message is specified.'), self.completion_version),
@@ -141,7 +141,6 @@ class Core(object):
             'plugins': (self.command_plugins, _('Usage: /plugins\nPlugins: Show the plugins in use.'), None),
             'presence': (self.command_presence, _('Usage: /presence <JID> [type] [status]\nPresence: Send a directed presence to <JID> and using [type] and [status] if provided.'), self.completion_presence),
             'rawxml': (self.command_rawxml, _('Usage: /rawxml\nRawXML: Send a custom xml stanza.'), None),
-            'set_plugin': (self.command_set_plugin, _("Usage: /set_plugin <plugin> <option> [value]\nSet Plugin: Set the value of the option in a plugin configuration file."), self.completion_set_plugin),
             'invite': (self.command_invite, _("Usage: /invite <jid> <room> [reason]\nInvite: Invite jid in room with reason."), self.completion_invite),
             'decline': (self.command_decline, _("Usage: /decline <room> [reason]\nDecline: Decline the invitation to room with or without reason."), self.completion_decline),
             'invitations': (self.command_invitations, _("Usage: /invites\nInvites: Show the pending invitations."), None),
@@ -1871,18 +1870,32 @@ class Core(object):
 
     def command_set(self, arg):
         """
-        /set <option> [value]
+        /set [module|][section] <option> <value>
         """
-        args = arg.split()
-        if len(args) != 2 and len(args) != 1:
+        args = common.shell_split(arg)
+        if len(args) != 2 and len(args) != 3:
             self.command_help('set')
             return
-        option = args[0]
         if len(args) == 2:
+            option = args[0]
             value = args[1]
-        else:
-            value = ''
-        config.set_and_save(option, value)
+            config.set_and_save(option, value)
+        elif len(args) == 3:
+            if '|' in args[0]:
+                plugin_name, section = args[0].split('|')
+                if not section:
+                    section = plugin_name
+                option = args[1]
+                value = args[2]
+                if not plugin_name in self.plugin_manager.plugins:
+                    return
+                plugin = self.plugin_manager.plugins[plugin_name]
+                plugin.config.set_and_save(option, value, section)
+            else:
+                section = args[0]
+                option = args[1]
+                value = args[2]
+                config.set_and_save(option, value, section)
         msg = "%s=%s" % (option, value)
         self.information(msg, 'Info')
 
@@ -1902,61 +1915,6 @@ class Core(object):
                         serv_list.append(serv)
             return the_input.auto_completion(serv_list, ' ')
 
-    def completion_set(self, the_input):
-        """Completion for /set"""
-        txt = the_input.get_text()
-        args = txt.split()
-        n = len(args)
-        if txt.endswith(' '):
-            n += 1
-        if n == 2:
-            return the_input.auto_completion(config.options('Poezio'), '')
-        elif n == 3:
-            return the_input.auto_completion([config.get(args[1], '')], '')
-
-    def command_set_plugin(self, arg):
-        """
-        /set_plugin <plugin> <option> [value]
-        """
-        args = arg.split()
-        if len(args) != 3 and len(args) != 2:
-            self.command_help('set_plugin')
-            return
-        plugin_name = args[0]
-        if not plugin_name in self.plugin_manager.plugins:
-            return
-        plugin = self.plugin_manager.plugins[plugin_name]
-        option = args[1]
-        if len(args) == 3:
-            value = args[2]
-        else:
-            value = ''
-        plugin.config.set_and_save(option, value, plugin_name)
-        if not plugin.config.write():
-            self.core.information('Could not save the plugin config', 'Error')
-            return
-        msg = "%s=%s" % (option, value)
-        self.information(msg, 'Info')
-
-    def completion_set_plugin(self, the_input):
-        """Completion for /set_plugin"""
-        txt = the_input.get_text()
-        args = txt.split()
-        n = len(args)
-        if txt.endswith(' '):
-            n += 1
-
-        if n == 2:
-            return the_input.auto_completion(list(self.plugin_manager.plugins.keys()), '')
-        elif n == 3:
-            if not args[1] in self.plugin_manager.plugins:
-                return
-            return the_input.auto_completion(self.plugin_manager.plugins[args[1]].config.options(args[1]), '')
-        elif n == 4:
-            if not args[1] in self.plugin_manager.plugins:
-                return
-            return the_input.auto_completion([self.plugin_manager.plugins[args[1]].config.get(args[2], '', args[1])], ' ')
-
     def close_tab(self, tab=None):
         """
         Close the given tab. If None, close the current one
-- 
cgit v1.2.3


From f76959c78c99044a3f43714ad285178a25498a90 Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Thu, 9 Feb 2012 17:45:28 +0100
Subject: Restrict nick change for gmail users because gmail.com is buggy

---
 src/tabs.py | 4 ++++
 1 file changed, 4 insertions(+)

(limited to 'src')

diff --git a/src/tabs.py b/src/tabs.py
index 4cf5c479..19ac06b5 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -545,6 +545,10 @@ class MucTab(ChatTab):
         self.commands['configure'] = (self.command_configure, _('Usage: /configure\nConfigure: Configure the current room, through a form.'), None)
         self.commands['version'] = (self.command_version, _('Usage: /version <jid or nick>\nVersion: Get the software version of the given JID or nick in room (usually its XMPP client and Operating System).'), self.completion_version)
         self.commands['names'] = (self.command_names, _('Usage: /names\nNames: Get the list of the users in the room, and the list of the people assuming the different roles.'), None)
+
+        if self.core.xmpp.boundjid.server == "gmail.com": #gmail sucks
+            del self.commands["nick"]
+
         self.resize()
         self.update_commands()
         self.update_keys()
-- 
cgit v1.2.3


From ed4f74faf1855a676c3407c0236baede34aba3ab Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Sun, 12 Feb 2012 00:51:01 +0100
Subject: Allow setting the affiliation of a JID, and add outcast affiliation
 (also, remove the reason of the affiliation change for the time being)

---
 src/multiuserchat.py | 18 ++++--------------
 src/tabs.py          | 22 +++++++++++++---------
 2 files changed, 17 insertions(+), 23 deletions(-)

(limited to 'src')

diff --git a/src/multiuserchat.py b/src/multiuserchat.py
index f537c2c1..3f0c80b8 100644
--- a/src/multiuserchat.py
+++ b/src/multiuserchat.py
@@ -88,21 +88,11 @@ def set_user_role(xmpp, jid, nick, reason, role):
     except Exception as e:
         return e.iq
 
-def set_user_affiliation(xmpp, jid, nick, reason, affiliation):
+def set_user_affiliation(xmpp, muc_jid, affiliation, nick=None, jid=None, reason=None):
     """
     (try to) Set the affiliation of a MUC user
     """
-    iq = xmpp.makeIqSet()
-    query = ET.Element('{%s}query' % NS_MUC_ADMIN)
-    item = ET.Element('{%s}item' % NS_MUC_ADMIN, {'nick':nick, 'affiliation':affiliation})
-    if reason:
-        reason_el = ET.Element('{%s}reason' % NS_MUC_ADMIN)
-        reason_el.text = reason
-        item.append(reason_el)
-    query.append(item)
-    iq.append(query)
-    iq['to'] = jid
     try:
-        return iq.send()
-    except Exception as e:
-        return e.iq
+        return xmpp.plugin['xep_0045'].set_affiliation(muc_jid, jid, nick, affiliation)
+    except:
+        return False
diff --git a/src/tabs.py b/src/tabs.py
index 19ac06b5..f9b2bb29 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -533,7 +533,7 @@ class MucTab(ChatTab):
         self.commands['unignore'] = (self.command_unignore, _("Usage: /unignore <nickname>\nUnignore: Remove the specified nickname from the ignore list."), self.completion_unignore)
         self.commands['kick'] =  (self.command_kick, _("Usage: /kick <nick> [reason]\nKick: Kick the user with the specified nickname. You also can give an optional reason."), self.completion_ignore)
         self.commands['role'] =  (self.command_role, _("Usage: /role <nick> <role> [reason]\nRole: Set the role of an user. Roles can be: none, visitor, participant, moderator. You also can give an optional reason."), self.completion_role)
-        self.commands['affiliation'] =  (self.command_affiliation, _("Usage: /affiliation <nick> <affiliation> [reason]\nAffiliation: Set the affiliation of an user. Affiliations can be: none, member, admin, owner. You also can give an optional reason."), self.completion_affiliation)
+        self.commands['affiliation'] =  (self.command_affiliation, _("Usage: /affiliation <nick or jid> <affiliation>\nAffiliation: Set the affiliation of an user. Affiliations can be: outcast, none, member, admin, owner."), self.completion_affiliation)
         self.commands['topic'] = (self.command_topic, _("Usage: /topic <subject>\nTopic: Change the subject of the room."), self.completion_topic)
         self.commands['query'] = (self.command_query, _('Usage: /query <nick> [message]\nQuery: Open a private conversation with <nick>. This nick has to be present in the room you\'re currently in. If you specified a message after the nickname, it will immediately be sent to this user.'), self.completion_ignore)
         self.commands['part'] = (self.command_part, _("Usage: /part [message]\nPart: Disconnect from a room. You can specify an optional message."), None)
@@ -606,6 +606,10 @@ class MucTab(ChatTab):
             n += 1
         if n == 2:
             userlist = [user.nick for user in self.users]
+            userlist.remove(self.own_nick)
+            jidlist = [user.jid.bare for user in self.users]
+            jidlist.remove(self.core.xmpp.boundjid.bare)
+            userlist.extend(jidlist)
             return the_input.auto_completion(userlist, '')
         elif n == 3:
             possible_affiliations = ['none', 'member', 'admin', 'owner']
@@ -861,7 +865,7 @@ class MucTab(ChatTab):
 
     def command_affiliation(self, arg):
         """
-        /affiliation <nick> <role> [reason]
+        /affiliation <nick> <role>
         Changes the affiliation of an user
         roles can be: none, visitor, participant, moderator
         """
@@ -874,14 +878,14 @@ class MucTab(ChatTab):
             reason = ' '.join(args[2:])
         else:
             reason = ''
-        if not self.joined or \
-                not affiliation in ('none', 'member', 'admin', 'owner'):
-#                replace this ↑ with this ↓ when the ban list support is done
-#                not affiliation in ('outcast', 'none', 'member', 'admin', 'owner'):
+        if not self.joined:
             return
-        res = muc.set_user_affiliation(self.core.xmpp, self.get_name(), nick, reason, affiliation)
-        if res['type'] == 'error':
-            self.core.room_error(res, self.get_name())
+        if nick in [user.nick for user in self.users]:
+            res = muc.set_user_affiliation(self.core.xmpp, self.get_name(), affiliation, nick=nick)
+        else:
+            res = muc.set_user_affiliation(self.core.xmpp, self.get_name(), affiliation, jid=nick)
+        if not res:
+            self.core.information('Could not set affiliation', 'Error')
 
     def command_say(self, line):
         needed = 'inactive' if self.core.status.show in ('xa', 'away') else 'active'
-- 
cgit v1.2.3


From be5ccce421143527f7779c662f4c920aba58f79c Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Mon, 13 Feb 2012 14:04:23 +0100
Subject: Useless option ssl=True (deprecated & unused)

---
 src/connection.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'src')

diff --git a/src/connection.py b/src/connection.py
index 74a21cdb..8a7c1ea7 100644
--- a/src/connection.py
+++ b/src/connection.py
@@ -40,7 +40,7 @@ class Connection(sleekxmpp.ClientXMPP):
             self.anon = True
             jid = '%s/%s' % (config.get('server', 'anon.louiz.org'), resource)
             password = None
-        sleekxmpp.ClientXMPP.__init__(self, jid, password, ssl=True)
+        sleekxmpp.ClientXMPP.__init__(self, jid, password)
         self.core = None
         self.auto_reconnect = True if config.get('auto_reconnect', 'false').lower() in ('true', '1') else False
         self.auto_authorize = None
-- 
cgit v1.2.3


From af110581290d28b86f991dc48ced55c085852225 Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Mon, 13 Feb 2012 20:01:11 +0100
Subject: Fix a traceback when in debug mode

---
 src/tabs.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

(limited to 'src')

diff --git a/src/tabs.py b/src/tabs.py
index f9b2bb29..e2d40e87 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -156,7 +156,7 @@ class Tab(object):
             log.debug("Invalid value for tab state: %s", value)
         elif STATE_PRIORITY[value] < STATE_PRIORITY[self._state] and \
                 value != 'current':
-            log.debug("Did not set status because of lower priority, asked: %s, kept: %s", (value, self._state))
+            log.debug("Did not set status because of lower priority, asked: %s, kept: %s", value, self._state)
         else:
             self._state = value
 
@@ -1703,7 +1703,7 @@ class RosterInfoTab(Tab):
                 roster.remove_contact(jid)
         except Exception as e:
             import traceback
-            log.debug(_('Traceback when removing %s from the roster:\n')+traceback.format_exc())
+            log.debug(_('Traceback when removing %s from the roster:\n' % jid)+traceback.format_exc())
 
     def command_add(self, args):
         """
-- 
cgit v1.2.3


From 560390793d29728ea96e1a547424a28098dcb610 Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Tue, 14 Feb 2012 00:33:29 +0100
Subject: Add 'joined' tab state, and rewrite the function handling the
 priorities

---
 src/core.py    | 39 ++++++++++-----------------------------
 src/tabs.py    | 12 ++++++++++--
 src/theming.py |  2 ++
 src/windows.py |  4 ++--
 4 files changed, 24 insertions(+), 33 deletions(-)

(limited to 'src')

diff --git a/src/core.py b/src/core.py
index 95ad22af..dfea7dd3 100644
--- a/src/core.py
+++ b/src/core.py
@@ -1085,35 +1085,16 @@ class Core(object):
 
     def go_to_important_room(self):
         """
-        Go to the next room with activity, in this order:
-        - A personal conversation with a new message
-        - A Muc with an highlight
-        - A Muc with any new message
-        """
-        for tab in self.tabs:
-           if tab.state == 'attention':
-               self.command_win('%s' % tab.nb)
-               return
-        for tab in self.tabs:
-            if tab.state == 'private':
-                self.command_win('%s' % tab.nb)
-                return
-        for tab in self.tabs:
-            if tab.state == 'highlight':
-                self.command_win('%s' % tab.nb)
-                return
-        for tab in self.tabs:
-            if tab.state == 'message':
-                self.command_win('%s' % tab.nb)
-                return
-        for tab in self.tabs:
-            if tab.state == 'disconnected':
-                self.command_win('%s' % tab.nb)
-                return
-        for tab in self.tabs:
-            if isinstance(tab, tabs.ChatTab) and not tab.input.is_empty():
-                self.command_win('%s' % tab.nb)
-                return
+        Go to the next room with activity, in the order defined in the
+        dict tabs.STATE_PRIORITY
+        """
+        priority = tabs.STATE_PRIORITY
+        sorted_tabs = sorted(self.tabs, key=lambda tab: priority[tab.state],
+                reverse=True)
+        tab = sorted_tabs.pop(0) if sorted_tabs else None
+        if priority[tab.state] < 0 or not tab:
+            return
+        self.command_win('%s' % tab.nb)
 
     def rotate_rooms_right(self, args=None):
         """
diff --git a/src/tabs.py b/src/tabs.py
index e2d40e87..365f19c0 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -62,6 +62,7 @@ NS_MUC_USER = 'http://jabber.org/protocol/muc#user'
 
 STATE_COLORS = {
         'disconnected': lambda: get_theme().COLOR_TAB_DISCONNECTED,
+        'joined': lambda: get_theme().COLOR_TAB_JOINED,
         'message': lambda: get_theme().COLOR_TAB_NEW_MESSAGE,
         'highlight': lambda: get_theme().COLOR_TAB_HIGHLIGHT,
         'private': lambda: get_theme().COLOR_TAB_PRIVATE,
@@ -72,6 +73,7 @@ STATE_COLORS = {
 
 VERTICAL_STATE_COLORS = {
         'disconnected': lambda: get_theme().COLOR_VERTICAL_TAB_DISCONNECTED,
+        'joined': lambda: get_theme().COLOR_VERTICAL_TAB_JOINED,
         'message': lambda: get_theme().COLOR_VERTICAL_TAB_NEW_MESSAGE,
         'highlight': lambda: get_theme().COLOR_VERTICAL_TAB_HIGHLIGHT,
         'private': lambda: get_theme().COLOR_VERTICAL_TAB_PRIVATE,
@@ -84,10 +86,11 @@ VERTICAL_STATE_COLORS = {
 STATE_PRIORITY = {
         'normal': -1,
         'current': -1,
-        'disconnected': 0,
         'message': 1,
+        'joined': 1,
         'highlight': 2,
         'private': 2,
+        'disconnected': 3,
         'attention': 3
     }
 
@@ -155,7 +158,7 @@ class Tab(object):
         if not value in STATE_COLORS:
             log.debug("Invalid value for tab state: %s", value)
         elif STATE_PRIORITY[value] < STATE_PRIORITY[self._state] and \
-                value != 'current':
+                value != 'current' and value != 'joined':
             log.debug("Did not set status because of lower priority, asked: %s, kept: %s", value, self._state)
         else:
             self._state = value
@@ -1066,6 +1069,8 @@ class MucTab(ChatTab):
                 self.users.append(new_user)
                 if from_nick == self.own_nick:
                     self.joined = True
+                    if self != self.core.current_tab():
+                        self.state = 'joined'
                     if self.core.current_tab() == self and self.core.status.show not in ('xa', 'away'):
                         self.send_chat_state('active')
                     new_user.color = get_theme().COLOR_OWN_NICK
@@ -1100,6 +1105,9 @@ class MucTab(ChatTab):
             self.info_header.refresh(self, self.text_win)
             self.input.refresh()
             self.core.doupdate()
+        else:
+            self.core.current_tab().refresh_tab_win()
+            self.core.doupdate()
 
     def on_user_join(self, from_nick, affiliation, show, status, role, jid):
         """
diff --git a/src/theming.py b/src/theming.py
index fba33366..c29d044d 100644
--- a/src/theming.py
+++ b/src/theming.py
@@ -122,6 +122,7 @@ class Theme(object):
 
     # Tabs
     COLOR_TAB_NORMAL = (7, 4)
+    COLOR_TAB_JOINED = (82, 4)
     COLOR_TAB_CURRENT = (7, 6)
     COLOR_TAB_NEW_MESSAGE = (7, 5)
     COLOR_TAB_HIGHLIGHT = (7, 3)
@@ -130,6 +131,7 @@ class Theme(object):
     COLOR_TAB_DISCONNECTED = (7, 8)
 
     COLOR_VERTICAL_TAB_NORMAL = (4, -1)
+    COLOR_VERTICAL_TAB_JOINED = (82, -1)
     COLOR_VERTICAL_TAB_CURRENT = (7, 4)
     COLOR_VERTICAL_TAB_NEW_MESSAGE = (5, -1)
     COLOR_VERTICAL_TAB_HIGHLIGHT = (3, -1)
diff --git a/src/windows.py b/src/windows.py
index 3a852ebc..91363f26 100644
--- a/src/windows.py
+++ b/src/windows.py
@@ -310,7 +310,7 @@ class GlobalInfoBar(Win):
             for tab in sorted_tabs:
                 color = tab.color
                 if config.get('show_inactive_tabs', 'true') == 'false' and\
-                        color == get_theme().COLOR_TAB_NORMAL:
+                        color is get_theme().COLOR_TAB_NORMAL:
                     continue
                 try:
                     self.addstr("%s" % str(tab.nb), to_curses_attr(color))
@@ -342,7 +342,7 @@ class VerticalGlobalInfoBar(Win):
             sorted_tabs = sorted(self.core.tabs, key=comp)
             if config.get('show_inactive_tabs', 'true') == 'false':
                 sorted_tabs = [tab for tab in sorted_tabs if\
-                                   tab.vertical_color != get_theme().COLOR_VERTICAL_TAB_NORMAL]
+                                   tab.vertical_color is not get_theme().COLOR_VERTICAL_TAB_NORMAL]
             nb_tabs = len(sorted_tabs)
             if nb_tabs >= height:
                 for y, tab in enumerate(sorted_tabs):
-- 
cgit v1.2.3


From 0f5ba0a7119ab6d2aa61e436fa19a8046ffbf712 Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Tue, 14 Feb 2012 02:53:31 +0100
Subject: Fixes #2310

---
 src/tabs.py | 27 +++++++++++++++++++--------
 1 file changed, 19 insertions(+), 8 deletions(-)

(limited to 'src')

diff --git a/src/tabs.py b/src/tabs.py
index 365f19c0..c98bbed4 100644
--- a/src/tabs.py
+++ b/src/tabs.py
@@ -801,25 +801,36 @@ class MucTab(ChatTab):
         color_participant = get_theme().COLOR_USER_PARTICIPANT[0]
         color_information = get_theme().COLOR_INFORMATION_TEXT[0]
         visitors, moderators, participants, others = [], [], [], []
+        aff = {
+                'owner': lambda: get_theme().CHAR_AFFILIATION_OWNER,
+                'admin': lambda: get_theme().CHAR_AFFILIATION_ADMIN,
+                'member': lambda: get_theme().CHAR_AFFILIATION_MEMBER,
+                'none': lambda: get_theme().CHAR_AFFILIATION_NONE,
+                }
+
         for user in self.users:
             if user.role == 'visitor':
-                visitors.append(user.nick)
+                visitors.append((user.nick, aff[user.affiliation]()))
             elif user.role == 'participant':
-                participants.append(user.nick)
+                participants.append((user.nick, aff[user.affiliation]()))
             elif user.role == 'moderator':
-                moderators.append(user.nick)
+                moderators.append((user.nick, aff[user.affiliation]()))
             else:
-                others.append(user.nick)
+                others.append((user.nick, aff[user.affiliation]()))
 
         message = 'Users: %s \n' % len(self.users)
         for moderator in moderators:
-            message += ' \x19%s}%s\x19o -' % (color_moderator, moderator)
+            message += ' [%s] \x19%s}%s\x19o -' % (moderator[1],
+                    color_moderator, moderator[0])
         for participant in participants:
-            message += ' \x19%s}%s\x19o -' % (color_participant, participant)
+            message += ' [%s] \x19%s}%s\x19o -' % (participant[1],
+                    color_participant, participant[0])
         for visitor in visitors:
-            message += ' \x19%s}%s\x19o -' % (color_visitor, visitor)
+            message += ' [%s] \x19%s}%s\x19o -' % (visitor[1],
+                    color_visitor, visitor[0])
         for other in others:
-            message += ' \x19%s}%s\x19o -' % (color_other, other)
+            message += ' [%s] \x19%s}%s\x19o -' % (other[1],
+                    color_other, other[0])
         message = message[:-2]
 
         self._text_buffer.add_message(message)
-- 
cgit v1.2.3


From b89cd8fd8322bf8aa23130398a0a70defcba708d Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Tue, 14 Feb 2012 11:49:09 +0100
Subject: Fixes #2303 (add user_list_sort option)

---
 src/windows.py | 15 ++++++++++++---
 1 file changed, 12 insertions(+), 3 deletions(-)

(limited to 'src')

diff --git a/src/windows.py b/src/windows.py
index 91363f26..542c027d 100644
--- a/src/windows.py
+++ b/src/windows.py
@@ -222,15 +222,24 @@ class UserList(Win):
         log.debug('Refresh: %s',self.__class__.__name__)
         with g_lock:
             self._win.erase()
-            y = 0
-            users = sorted(users)
+            if config.get('user_list_sort', 'desc').lower() == 'asc':
+                y, x = self._win.getmaxyx()
+                y -= 1
+                users = sorted(users, reverse=True)
+            else:
+                y = 0
+                users = sorted(users)
+
             if self.pos >= len(users) and self.pos != 0:
                 self.pos = len(users)-1
             for user in users[self.pos:]:
                 self.draw_role_affiliation(y, user)
                 self.draw_status_chatstate(y, user)
                 self.addstr(y, 2, user.nick[:self.width-2], to_curses_attr(user.color))
-                y += 1
+                if config.get('user_list_sort', 'desc').lower() == 'asc':
+                    y -= 1
+                else:
+                    y += 1
                 if y == self.height:
                     break
             # draw indicators of position in the list
-- 
cgit v1.2.3