From 3afa155cfbc48198ba3d9821c5d0dae31084123d Mon Sep 17 00:00:00 2001
From: "louiz@4325f9fc-e183-4c21-96ce-0ab188b42d13"
 <louiz@4325f9fc-e183-4c21-96ce-0ab188b42d13>
Date: Thu, 22 Jul 2010 17:47:34 +0000
Subject: Don't send <show>None</show> presence. fixed #1666

---
 src/gui.py           | 8 ++++----
 src/multiuserchat.py | 3 ++-
 2 files changed, 6 insertions(+), 5 deletions(-)

(limited to 'src')

diff --git a/src/gui.py b/src/gui.py
index 299770eb..4c95566f 100644
--- a/src/gui.py
+++ b/src/gui.py
@@ -744,10 +744,10 @@ class Gui(object):
         """
         /show <status> [msg]
         """
-        possible_show = {'avail':'None',
-                         'available':'None',
-                         'ok':'None',
-                         'here':'None',
+        possible_show = {'avail':None,
+                         'available':None,
+                         'ok':None,
+                         'here':None,
                          'chat':'chat',
                          'away':'away',
                          'afk':'away',
diff --git a/src/multiuserchat.py b/src/multiuserchat.py
index a64b5796..d2598397 100644
--- a/src/multiuserchat.py
+++ b/src/multiuserchat.py
@@ -291,7 +291,8 @@ class MultiUserChat(object):
 
     def change_show(self, room, nick, show, status):
         pres = Presence(to='%s/%s' % (room, nick))
-        pres.setShow(show)
+        if show: # if show is None, don't put a <show /> tag. It means "online"
+            pres.setShow(show)
         if status:
             pres.setStatus(status)
         self.connection.send(pres)
-- 
cgit v1.2.3