Remove redundant check

eljulians · eljulians · commit b328c3e2c00e · 2020-04-22T09:20:22.000+02:00
The payload object was already initialized with the `username` value,
so there's no need to check whether is specified or not.
diff --git a/jira/client.py b/jira/client.py
@@ -2845,9 +2845,7 @@ def search_users(
             "includeInactive": includeInactive,
         }
 
-        if user:
-            params["username"] = user
-        else:
+        if query:
             params["query"] = query
 
         return self._fetch_pages(User, None, "user/search", startAt, maxResults, params)

Original file line number	Diff line number	Diff line change
`@@ -2845,9 +2845,7 @@ def search_users(`
`2845`	`2845`	`"includeInactive": includeInactive,`
`2846`	`2846`	`}`
`2847`	`2847`
`2848`		`- if user:`
`2849`		`- params["username"] = user`
`2850`		`- else:`
	`2848`	`+ if query:`
`2851`	`2849`	`params["query"] = query`
`2852`	`2850`
`2853`	`2851`	`return self._fetch_pages(User, None, "user/search", startAt, maxResults, params)`