Skip to content
Snippets Groups Projects
Commit 55c56f40 authored by Mateusz Żebrak's avatar Mateusz Żebrak
Browse files

DRY in KeyManager sorting

parent a016867c
No related branches found
No related tags found
2 merge requests!412V1.27.5.13 release,!408Make keys stored in profile (and not account)
...@@ -36,10 +36,10 @@ class KeyManager: ...@@ -36,10 +36,10 @@ class KeyManager:
self.__keys_to_import: set[PrivateKeyAliased] = set() self.__keys_to_import: set[PrivateKeyAliased] = set()
def __iter__(self) -> Iterator[PublicKeyAliased]: def __iter__(self) -> Iterator[PublicKeyAliased]:
return iter(sorted(self.__keys, key=lambda key: key.alias)) return iter(self._sorted_keys())
def __reversed__(self) -> Iterator[PublicKeyAliased]: def __reversed__(self) -> Iterator[PublicKeyAliased]:
return iter(sorted(self.__keys, key=lambda key: key.alias, reverse=True)) return iter(self._sorted_keys(reverse=True))
def __len__(self) -> int: def __len__(self) -> int:
return len(self.__keys) return len(self.__keys)
...@@ -106,6 +106,9 @@ class KeyManager: ...@@ -106,6 +106,9 @@ class KeyManager:
self.add(*imported_keys) self.add(*imported_keys)
logger.debug("Imported all pending keys to beekeeper.") logger.debug("Imported all pending keys to beekeeper.")
def _sorted_keys(self, *, reverse: bool = False) -> list[PublicKeyAliased]:
return sorted(self.__keys, key=lambda key: key.alias, reverse=reverse)
def _is_public_alias_available(self, alias: str) -> bool: def _is_public_alias_available(self, alias: str) -> bool:
return self._is_alias_available(alias, self.__keys) return self._is_alias_available(alias, self.__keys)
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment