Wrong number of arguments for geosearch command (#2464)

dvora-h · chayim · web-flow · commit f4d07dddba55 · 2022-12-01T13:16:14.000+02:00
Co-authored-by: Chayim <chayim@users.noreply.github.com> Fixes #2462
diff --git a/redis/commands/core.py b/redis/commands/core.py
@@ -5581,7 +5581,7 @@ def _geosearchgeneric(
                     "GEOSEARCH member and longitude or latitude" " cant be set together"
                 )
             pieces.extend([b"FROMMEMBER", kwargs["member"]])
-        if kwargs["longitude"] and kwargs["latitude"]:
+        if kwargs["longitude"] is not None and kwargs["latitude"] is not None:
             pieces.extend([b"FROMLONLAT", kwargs["longitude"], kwargs["latitude"]])
 
         # BYRADIUS or BYBOX

Original file line number	Diff line number	Diff line change
`@@ -5581,7 +5581,7 @@ def _geosearchgeneric(`
`5581`	`5581`	`"GEOSEARCH member and longitude or latitude" " cant be set together"`
`5582`	`5582`	`)`
`5583`	`5583`	`pieces.extend([b"FROMMEMBER", kwargs["member"]])`
`5584`		`- if kwargs["longitude"] and kwargs["latitude"]:`
	`5584`	`+ if kwargs["longitude"] is not None and kwargs["latitude"] is not None:`
`5585`	`5585`	`pieces.extend([b"FROMLONLAT", kwargs["longitude"], kwargs["latitude"]])`
`5586`	`5586`
`5587`	`5587`	`# BYRADIUS or BYBOX`