diff options
author | Patrick Cloke <clokep@users.noreply.github.com> | 2020-08-27 12:07:13 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-08-27 12:07:13 -0400 |
commit | c9fa696ea2ad5bc32430aeb1bc555df537a71a59 (patch) | |
tree | 750186b5fe88c21e5d501b47f704fcd711a47198 /synapse | |
parent | Fix missing _add_persisted_position (#8179) (diff) | |
download | synapse-c9fa696ea2ad5bc32430aeb1bc555df537a71a59.tar.xz |
simple_search_list_txn should return None, not 0. (#8187)
Diffstat (limited to 'synapse')
-rw-r--r-- | synapse/storage/database.py | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/synapse/storage/database.py b/synapse/storage/database.py index 2f6f49a4bf..ba4c0c9af6 100644 --- a/synapse/storage/database.py +++ b/synapse/storage/database.py @@ -28,7 +28,6 @@ from typing import ( Optional, Tuple, TypeVar, - Union, overload, ) @@ -1655,7 +1654,7 @@ class DatabasePool(object): term: Optional[str], col: str, retcols: Iterable[str], - ) -> Union[List[Dict[str, Any]], int]: + ) -> Optional[List[Dict[str, Any]]]: """Executes a SELECT query on the named table, which may return zero or more rows, returning the result as a list of dicts. @@ -1667,14 +1666,14 @@ class DatabasePool(object): retcols: the names of the columns to return Returns: - 0 if no term is given, otherwise a list of dictionaries. + None if no term is given, otherwise a list of dictionaries. """ if term: sql = "SELECT %s FROM %s WHERE %s LIKE ?" % (", ".join(retcols), table, col) termvalues = ["%%" + term + "%%"] txn.execute(sql, termvalues) else: - return 0 + return None return cls.cursor_to_dict(txn) |