chore: Refactor unnecessary else / elif when if block has a `re…

…turn` statement (#7332)

Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>

app/api/helpers/payment.py

@@ -55,19 +55,18 @@ def get_credentials(event=None):
                     'PUBLISHABLE_KEY': settings["stripe_publishable_key"],
                 }
             return None
+        if represents_int(event):
+            authorization = StripeAuthorization.query.filter_by(
+                event_id=event
+            ).first()
         else:
-            if represents_int(event):
-                authorization = StripeAuthorization.query.filter_by(
-                    event_id=event
-                ).first()
-            else:
-                authorization = event.stripe_authorization
-            if authorization:
-                return {
-                    'SECRET_KEY': authorization.stripe_secret_key,
-                    'PUBLISHABLE_KEY': authorization.stripe_publishable_key,
-                }
-            return None
+            authorization = event.stripe_authorization
+        if authorization:
+            return {
+                'SECRET_KEY': authorization.stripe_secret_key,
+                'PUBLISHABLE_KEY': authorization.stripe_publishable_key,
+            }
+        return None
 
     @staticmethod
     def get_event_organizer_credentials_from_stripe(stripe_auth_code):

app/api/users.py

@@ -400,8 +400,7 @@ def is_email_available():
         if get_count(db.session.query(User).filter_by(email=email)):
             return jsonify(result="False")
         return jsonify(result="True")
-    else:
-        abort(make_response(jsonify(error="Email field missing"), 422))
+    abort(make_response(jsonify(error="Email field missing"), 422))
 
 
 def start_image_resizing_tasks(user, original_image_url):

app/views/healthcheck.py

@@ -88,5 +88,4 @@ def health_check_migrations():
             return True, result[1]
         # the exception will be caught in check_migrations function, so no need for sentry catching exception here
         return False, result[1]
-    else:
-        return False, 'The health_check_migration test is still running'
+    return False, 'The health_check_migration test is still running'