Philipp Hörist pushed to branch master at gajim / gajim


Commits:
944ad9cc by lovetox at 2021-11-07T16:18:04+01:00
Configpaths: Fix get_paths()

_paths is not a dict we can’t use values()

- - - - -
5f15291a by lovetox at 2021-11-07T16:19:40+01:00
Fix dict iteration

- - - - -


2 changed files:

- gajim/common/configpaths.py
- gajim/common/helpers.py


Changes:

=====================================
gajim/common/configpaths.py
=====================================
@@ -53,11 +53,12 @@ def get_plugin_dirs() -> List[Path]:
 
 
 def get_paths(type_: PathType) -> Generator[Path, None, None]:
-    for value in _paths.values():
+    # pylint: disable=unnecessary-dict-index-lookup
+    for key, value in _paths.items():
         path_type = value[2]
         if type_ != path_type:
             continue
-        yield value
+        yield _paths[key]
 
 
 def override_path(*args, **kwargs):


=====================================
gajim/common/helpers.py
=====================================
@@ -569,7 +569,7 @@ def statuses_unified():
     Test if all statuses are the same
     """
     reference = None
-    for con, account in app.connections.items():
+    for account, con in app.connections.items():
         if not app.settings.get_account_setting(account,
                                                 'sync_with_global_status'):
             continue



View it on GitLab: 
https://dev.gajim.org/gajim/gajim/-/compare/064f77e9451f045773452b3788da77809f320e14...5f15291a6b9e6f94770034f567aae9cfd06a19ce

-- 
View it on GitLab: 
https://dev.gajim.org/gajim/gajim/-/compare/064f77e9451f045773452b3788da77809f320e14...5f15291a6b9e6f94770034f567aae9cfd06a19ce
You're receiving this email because of your account on dev.gajim.org.


_______________________________________________
Commits mailing list
[email protected]
https://lists.gajim.org/cgi-bin/listinfo/commits

Reply via email to