diff options
| author | mathieui <mathieui@mathieui.net> | 2020-05-28 00:28:23 +0200 |
|---|---|---|
| committer | mathieui <mathieui@mathieui.net> | 2020-05-28 00:30:13 +0200 |
| commit | 9e3958c67d2e7d16a069b50ef2e76d92280b6584 (patch) | |
| tree | e40ecac3e9c886f9027f9f9a30d070914e94ec4c /poezio | |
| parent | 5cf90368c66e0ccc3a93e51911f8fa82b496e02b (diff) | |
| download | poezio-9e3958c67d2e7d16a069b50ef2e76d92280b6584.tar.gz poezio-9e3958c67d2e7d16a069b50ef2e76d92280b6584.tar.bz2 poezio-9e3958c67d2e7d16a069b50ef2e76d92280b6584.tar.xz poezio-9e3958c67d2e7d16a069b50ef2e76d92280b6584.zip | |
Only add autojoin to a bookmark on creation if the room is not bookmarked
With synchronise_open_rooms, otherwise you just have autojoin added
everytime, which is probably not what you want.
Diffstat (limited to 'poezio')
| -rw-r--r-- | poezio/bookmarks.py | 2 | ||||
| -rw-r--r-- | poezio/core/commands.py | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/poezio/bookmarks.py b/poezio/bookmarks.py index ccda2b93..d842d2dd 100644 --- a/poezio/bookmarks.py +++ b/poezio/bookmarks.py @@ -171,7 +171,7 @@ class BookmarkList: return self.bookmarks[key] return None - def __in__(self, key) -> bool: + def __contains__(self, key) -> bool: if isinstance(key, (str, JID)): for bookmark in self.bookmarks: if bookmark.jid == key: diff --git a/poezio/core/commands.py b/poezio/core/commands.py index 176d0e73..6fc800ae 100644 --- a/poezio/core/commands.py +++ b/poezio/core/commands.py @@ -401,7 +401,7 @@ class CommandCore: tab.password = password tab.join() - if config.get('synchronise_open_rooms'): + if config.get('synchronise_open_rooms') and room not in self.core.bookmarks: method = 'remote' if config.get( 'use_remote_bookmarks') else 'local' self._add_bookmark('%s/%s' % (room, nick), True, password, method) |