summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMaxime Buquet <pep@bouah.net>2020-04-01 01:37:55 +0200
committerMaxime Buquet <pep@bouah.net>2020-04-01 01:37:55 +0200
commit1942e087fd1147f1fa34e32667244ceea654899b (patch)
treebc8ad34665f8f9f7da5210e0c1667b54a248ddef
parentdd536d9e625d5364ca167494469cff673a024dc0 (diff)
parentda695768b099d91e24c678d7abc70560ff75444a (diff)
downloadpoezio-1942e087fd1147f1fa34e32667244ceea654899b.tar.gz
poezio-1942e087fd1147f1fa34e32667244ceea654899b.tar.bz2
poezio-1942e087fd1147f1fa34e32667244ceea654899b.tar.xz
poezio-1942e087fd1147f1fa34e32667244ceea654899b.zip
Merge branch 'reorder-fixes' into 'master'
Reorder fixes See merge request poezio/poezio!77
-rw-r--r--plugins/reorder.py29
1 files changed, 21 insertions, 8 deletions
diff --git a/plugins/reorder.py b/plugins/reorder.py
index 8d9516f8..7be0b350 100644
--- a/plugins/reorder.py
+++ b/plugins/reorder.py
@@ -59,6 +59,8 @@ And finally, the ``[tab name]`` must be:
- For a type ``static``, the full JID of the contact
"""
+from slixmpp import InvalidJID, JID
+
from poezio import tabs
from poezio.decorators import command_args_parser
from poezio.plugin import BasePlugin
@@ -162,21 +164,32 @@ class Plugin(BasePlugin):
new_tabs += [
tabs.GapTab(self.core) for i in range(pos - last - 1)
]
- cls, name = tabs_spec[pos]
- tab = self.core.tabs.by_name_and_class(name, cls=cls)
- if tab and tab in old_tabs:
- new_tabs.append(tab)
- old_tabs.remove(tab)
- else:
- self.api.information('Tab %s not found' % name, 'Warning')
+ cls, jid = tabs_spec[pos]
+ try:
+ jid = JID(jid)
+ tab = self.core.tabs.by_name_and_class(str(jid), cls=cls)
+ if tab and tab in old_tabs:
+ new_tabs.append(tab)
+ old_tabs.remove(tab)
+ else:
+ self.api.information('Tab %s not found. Creating it' % jid, 'Warning')
+ # TODO: Add support for MucTab. Requires nickname.
+ if cls in (tabs.DynamicConversationTab, tabs.StaticConversationTab):
+ new_tab = cls(self.core, jid)
+ new_tabs.append(new_tab)
+ except:
+ self.api.information('Failed to create tab \'%s\'.' % jid, 'Error')
if create_gaps:
new_tabs.append(tabs.GapTab(self.core))
- last = pos
+ finally:
+ last = pos
for tab in old_tabs:
if tab:
new_tabs.append(tab)
+ # TODO: Ensure we don't break poezio and call this with whatever
+ # tablist we have. The roster tab at least needs to be in there.
self.core.tabs.replace_tabs(new_tabs)
self.core.refresh_window()