Fix: ensure SASTs see only explicit returns happening

OpenTTD · Oct 2, 2020 · b63707a · b63707a
1 parent b7cd6f1
commit b63707a
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/webclient/api.py b/webclient/api.py
@@ -45,11 +45,11 @@ def api_call(method, path, params=None, json=None, session=None, return_errors=F
         # have to the caller.
         pass
 
-    if return_errors:
-        return (None, "API call failed")
-    else:
+    if not return_errors:
         flask.abort(error_response)
 
+    return (None, "API call failed")
+
 
 def api_get(*args, **kwargs):
     return api_call(requests.get, *args, **kwargs)