params has always keys in _execute_request()

mib1185 · Jan 5, 2025 · 8f62661 · 8f62661
1 parent 68082e9
commit 8f62661
Showing 1 changed file with 5 additions and 8 deletions.
diff --git a/src/synology_dsm/synology_dsm.py b/src/synology_dsm/synology_dsm.py
@@ -342,14 +342,11 @@ async def _execute_request(
         self, method: str, url: URL, params: dict, **kwargs: Any
     ) -> bytes | dict | str:
         """Function to execute and handle a request."""
-        if params:
-            # special handling for spaces in parameters
-            # because yarl.URL does encode a space as + instead of %20
-            # safe extracted from yarl.URL._QUERY_PART_QUOTER
-            query = urlencode(params, safe="?/:@-._~!$'()*,", quote_via=quote)
-            url_encoded = url.join(URL(f"?{query}", encoded=True))
-        else:
-            url_encoded = url
+        # special handling for spaces in parameters
+        # because yarl.URL does encode a space as + instead of %20
+        # safe extracted from yarl.URL._QUERY_PART_QUOTER
+        query = urlencode(params, safe="?/:@-._~!$'()*,", quote_via=quote)
+        url_encoded = url.join(URL(f"?{query}", encoded=True))
 
         try:
             if method == "GET":