From ceebea0dd343ae7e7590292f174514f748dcd4d5 Mon Sep 17 00:00:00 2001 From: beckerfy <fynn.becker@hs-hannover.de> Date: Thu, 29 Nov 2018 11:56:59 +0100 Subject: [PATCH] Rename path argument to reflect proper usage Query params should really be passed as params kwarg instead. --- postgrestutils/client/postgrestclient.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/postgrestutils/client/postgrestclient.py b/postgrestutils/client/postgrestclient.py index 61bfb11..64c008b 100644 --- a/postgrestutils/client/postgrestclient.py +++ b/postgrestutils/client/postgrestclient.py @@ -11,9 +11,9 @@ class PostgrestClient: self.session = requests.Session() self.session.headers["Authorization"] = "Bearer {}".format(token) - def get(self, path_and_query, singular=False, parse_dt=True, **kwargs): + def get(self, path, singular=False, parse_dt=True, **kwargs): """ - :param path_and_query: specifies the endpoint and optionally filter queries + :param path: specifies the endpoint :param singular: if True returns a JSON object rather than a list (406 when multiple results are returned) :param parse_dt: if True attempts to parse datetime strings to python datetime objects :return: result(s) as python object or raises HTTPError @@ -22,7 +22,7 @@ class PostgrestClient: self.session.headers["Accept"] = "application/vnd.pgrst.object+json" else: self.session.headers["Accept"] = "application/json" - res = self.session.get(urljoin(self.base_uri, path_and_query), **kwargs) + res = self.session.get(urljoin(self.base_uri, path), **kwargs) try: res.raise_for_status() except requests.HTTPError as e: -- GitLab