fix pr comments

Author: dvora-h

redis/commands/search/commands.py

@@ -384,7 +384,11 @@ def info(self):
         it = map(to_string, res)
         return dict(zip(it, it))
 
-    def get_params_args(self, query_params: Dict[str, Union[str, int, float]]):
+    def get_params_args(
+        self, query_params: Union[Dict[str, Union[str, int, float]], None]
+    ):
+        if query_params is None:
+            return []
         args = []
         if len(query_params) > 0:
             args.append("params")
@@ -404,8 +408,7 @@ def _mk_query_args(self, query, query_params: Dict[str, Union[str, int, float]])
             raise ValueError(f"Bad query type {type(query)}")
 
         args += query.get_args()
-        if query_params is not None:
-            args += self.get_params_args(query_params)
+        args += self.get_params_args(query_params)
 
         return args, query
 
@@ -480,8 +483,7 @@ def aggregate(
             cmd = [CURSOR_CMD, "READ", self.index_name] + query.build_args()
         else:
             raise ValueError("Bad query", query)
-        if query_params is not None:
-            cmd += self.get_params_args(query_params)
+        cmd += self.get_params_args(query_params)
 
         raw = self.execute_command(*cmd)
         return self._get_aggregate_result(raw, query, has_cursor)
@@ -506,16 +508,17 @@ def _get_aggregate_result(self, raw, query, has_cursor):
 
         return AggregateResult(rows, cursor, schema)
 
-    def profile(self, query, query_params=None, limited=False):
+    def profile(self, query, limited=False, query_params=None):
         """
         Performs a search or aggregate command and collects performance
         information.
 
         ### Parameters
 
-        **query**: This can be either an `AggregateRequest`, `Query` or
-        string.
+        **query**: This can be either an `AggregateRequest`, `Query` or string.
         **limited**: If set to True, removes details of reader iterator.
+        **query_params**: Define one or more value parameters.
+        Each parameter has a name and a value.
 
         """
         st = time.time()
@@ -530,8 +533,7 @@ def profile(self, query, query_params=None, limited=False):
         elif isinstance(query, Query):
             cmd[2] = "SEARCH"
             cmd += query.get_args()
-            if query_params is not None:
-                cmd += self.get_params_args(query_params)
+            cmd += self.get_params_args(query_params)
         else:
             raise ValueError("Must provide AggregateRequest object or " "Query object.")
 
@@ -930,8 +932,7 @@ async def aggregate(
             cmd = [CURSOR_CMD, "READ", self.index_name] + query.build_args()
         else:
             raise ValueError("Bad query", query)
-        if query_params is not None:
-            cmd += self.get_params_args(query_params)
+        cmd += self.get_params_args(query_params)
 
         raw = await self.execute_command(*cmd)
         return self._get_aggregate_result(raw, query, has_cursor)