copyleftie
/
gajim-plural
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Fix sending empty pep data

This commit is contained in:
Philipp Hörist 2018-09-16 12:58:37 +02:00
parent 1462f89229
commit e85bc52c8b
5 changed files with 8 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
gajim/common/modules/pep.py
View File

@ -106,7 +106,7 @@ class PEP:
retract = items.getTag('retract')
if retract is not None:
for handler in handlers:
handler[PEPHandlerType.RETRACT](jid, retract.getID())
handler[PEPHandlerType.RETRACT](jid, retract.getAttr('id'))
raise nbxmpp.NodeProcessed
# Check if we have items

2
gajim/common/modules/user_activity.py
View File

@ -89,7 +89,7 @@ class UserActivity(AbstractPEPModule):
def _build_node(self, data):
item = nbxmpp.Node('activity', {'xmlns': self.namespace})
if data is None:
return
return item
activity, subactivity, message = data
if activity:
i = item.addChild(activity)

2
gajim/common/modules/user_mood.py
View File

@ -88,7 +88,7 @@ class UserMood(AbstractPEPModule):
def _build_node(self, data: Optional[Tuple[str, str]]) -> nbxmpp.Node:
item = nbxmpp.Node('mood', {'xmlns': nbxmpp.NS_MOOD})
if data is None:
return
return item
mood, text = data
if mood:
item.addChild(mood)

5
gajim/common/modules/user_nickname.py
View File

@ -67,9 +67,8 @@ class UserNickname(AbstractPEPModule):
def _build_node(self, data: Optional[str]) -> Optional[nbxmpp.Node]:
item = nbxmpp.Node('nick', {'xmlns': nbxmpp.NS_NICK})
if data is None:
return None
item.addData(data)
if data is not None:
item.addData(data)
return item
def _notification_received(self,

7
gajim/gtk/profile.py
View File

@ -331,11 +331,10 @@ class ProfileWindow(Gtk.ApplicationWindow):
transient_for=self)
return
vcard_, sha = self.make_vcard()
nick = ''
if 'NICKNAME' in vcard_:
nick = vcard_['NICKNAME']
nick = vcard_.get('NICKNAME')
if nick:
app.connections[self.account].get_module('UserNickname').send(nick)
if nick == '':
else:
app.connections[self.account].get_module('UserNickname').retract()
nick = app.config.get_per('accounts', self.account, 'name')
app.nicks[self.account] = nick