chore: Refactor unnecessary else / elif when if block has a return statement (#7331)

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

Author: deepsource-autofix[bot]

app/api/auth.py

@@ -247,7 +247,7 @@ def login_user(provider):
             200,
         )
 
-    elif provider == 'google':
+    if provider == 'google':
         provider_class = GoogleOAuth()
         payload = {
             'client_id': provider_class.get_client_id(),

app/api/celery_tasks.py

@@ -32,9 +32,8 @@ def celery_task(task_id):
             # check if is error
             if '__error' in info:
                 return info['result']
-        # return normal
+            # return normal
         return jsonify(state='SUCCESS', result=info)
-    elif state == 'FAILURE':
-        return jsonify(state=state)
-    else:
+    if state == 'FAILURE':
         return jsonify(state=state)
+    return jsonify(state=state)

app/api/custom/orders.py

@@ -105,11 +105,10 @@ def resend_emails():
                     order_identifier
                 ),
             )
-        else:
-            raise UnprocessableEntityError(
-                {'source': 'data/order'},
-                "Only placed and completed orders have confirmation",
-            )
+        raise UnprocessableEntityError(
+            {'source': 'data/order'},
+            "Only placed and completed orders have confirmation",
+        )
     else:
         raise ForbiddenError({'source': ''}, "Co-Organizer Access Required")
 
@@ -190,7 +189,7 @@ def complete_order(order_id):
                 jsonify(status='Access Forbidden', error='You can only cancel an order.'),
                 403,
             )
-        elif data['status'] == 'cancelled':
+        if data['status'] == 'cancelled':
             order.status = 'cancelled'
             db.session.add(order)
             attendees = (
@@ -221,12 +220,11 @@ def complete_order(order_id):
             ),
             422,
         )
-    else:
-        attendees = (
-            db.session.query(TicketHolder)
-            .filter_by(order_id=order_id, deleted_at=None)
-            .all()
-        )
+    attendees = (
+        db.session.query(TicketHolder)
+        .filter_by(order_id=order_id, deleted_at=None)
+        .all()
+    )
     form_fields = (
         db.session.query(CustomForms)
         .filter_by(event_id=order.event_id, form='attendee', is_included=True)

app/api/event_invoices.py

@@ -166,8 +166,7 @@ def create_paypal_payment_invoice(invoice_identifier):
 
     if status:
         return jsonify(status=True, payment_id=response)
-    else:
-        return jsonify(status=False, error=response)
+    return jsonify(status=False, error=response)
 
 
 @order_misc_routes.route(
@@ -206,6 +205,5 @@ def charge_paypal_payment_invoice(invoice_identifier):
         save_to_db(event_invoice)
 
         return jsonify(status="Charge Successful", payment_id=paypal_payment_id)
-    else:
-        # return the error message from Paypal
-        return jsonify(status="Charge Unsuccessful", error=error)
+    # return the error message from Paypal
+    return jsonify(status="Charge Unsuccessful", error=error)

app/api/helpers/db.py

@@ -106,12 +106,11 @@ def get_or_create(model, **kwargs):
     instance = db.session.query(model).filter_by(**kwargs).first()
     if instance:
         return instance, was_created
-    else:
-        instance = model(**kwargs)
-        db.session.add(instance)
-        db.session.commit()
-        was_created = True
-        return instance, was_created
+    instance = model(**kwargs)
+    db.session.add(instance)
+    db.session.commit()
+    was_created = True
+    return instance, was_created
 
 
 def get_count(query):
@@ -141,8 +140,7 @@ def get_new_slug(model, name):
     count = get_count(model.query.filter_by(slug=slug))
     if count == 0:
         return slug
-    else:
-        return f'{slug}-{uuid.uuid4().hex}'
+    return f'{slug}-{uuid.uuid4().hex}'
 
 
 def get_new_identifier(model, length=None):
@@ -153,5 +151,4 @@ def get_new_identifier(model, length=None):
     count = get_count(model.query.filter_by(identifier=identifier))
     if not identifier.isdigit() and count == 0:
         return identifier
-    else:
-        return get_new_identifier(model)
+    return get_new_identifier(model)

app/api/helpers/export_helpers.py

@@ -270,8 +270,7 @@ def export_event_json(event_id, settings):
 def get_current_user():
     if current_user:
         return current_user
-    else:
-        return current_logged_user
+    return current_logged_user
 
 
 # HELPERS

app/api/helpers/jwt.py

@@ -20,8 +20,7 @@ def jwt_authenticate(email, password):
     auth_ok = user.facebook_login_hash == password or user.is_correct_password(password)
     if auth_ok:
         return user
-    else:
-        return None
+    return None
 
 
 def jwt_user_loader(identity):

app/api/helpers/payment.py

@@ -49,13 +49,12 @@ def get_credentials(event=None):
                     'SECRET_KEY': settings['stripe_test_secret_key'],
                     'PUBLISHABLE_KEY': settings["stripe_test_publishable_key"],
                 }
-            elif settings['stripe_secret_key'] and settings["stripe_publishable_key"]:
+            if settings['stripe_secret_key'] and settings["stripe_publishable_key"]:
                 return {
                     'SECRET_KEY': settings['stripe_secret_key'],
                     'PUBLISHABLE_KEY': settings["stripe_publishable_key"],
                 }
-            else:
-                return None
+            return None
         else:
             if represents_int(event):
                 authorization = StripeAuthorization.query.filter_by(
@@ -68,8 +67,7 @@ def get_credentials(event=None):
                     'SECRET_KEY': authorization.stripe_secret_key,
                     'PUBLISHABLE_KEY': authorization.stripe_publishable_key,
                 }
-            else:
-                return None
+            return None
 
     @staticmethod
     def get_event_organizer_credentials_from_stripe(stripe_auth_code):
@@ -225,8 +223,7 @@ def create_payment(order, return_url, cancel_url, payee_email=None):
 
         if payment.create():
             return True, payment.id
-        else:
-            return False, payment.error
+        return False, payment.error
 
     @staticmethod
     def verify_payment(payment_id, order):
@@ -252,14 +249,13 @@ def verify_payment(payment_id, order):
 
             if float(amount_server) != order.amount:
                 return False, 'Payment amount does not match order'
-            elif currency_server != order.event.payment_currency:
+            if currency_server != order.event.payment_currency:
                 return False, 'Payment currency does not match order'
-            elif sale_state != 'completed':
+            if sale_state != 'completed':
                 return False, 'Sale not completed'
-            elif PayPalPaymentsManager.used_payment(payment_id, order):
+            if PayPalPaymentsManager.used_payment(payment_id, order):
                 return False, 'Payment already been verified'
-            else:
-                return True, None
+            return True, None
         except paypalrestsdk.ResourceNotFound:
             return False, 'Payment Not Found'
 
@@ -272,8 +268,7 @@ def used_payment(payment_id, order):
             order.paypal_token = payment_id
             save_to_db(order)
             return False
-        else:
-            return True
+        return True
 
     @staticmethod
     def execute_payment(paypal_payer_id, paypal_payment_id):
@@ -288,8 +283,7 @@ def execute_payment(paypal_payer_id, paypal_payment_id):
 
         if payment.execute({"payer_id": paypal_payer_id}):
             return True, 'Successfully Executed'
-        else:
-            return False, payment.error
+        return False, payment.error
 
 
 class AliPayPaymentsManager:

app/api/helpers/speaker.py

@@ -23,7 +23,6 @@ def can_edit_after_cfs_ends(event_id):
                 or has_access('is_coorganizer', event_id=event_id)
             )
         )
-    else:
-        raise ForbiddenError(
-            {'source': '/data/event-id'}, f'Speaker Calls for event {event_id} not found',
-        )
+    raise ForbiddenError(
+        {'source': '/data/event-id'}, f'Speaker Calls for event {event_id} not found',
+    )

app/api/helpers/storage.py

@@ -159,12 +159,11 @@ def upload(uploaded_file, key, upload_dir='static/media/', **kwargs):
         return upload_to_aws(
             aws_bucket_name, aws_region, aws_key, aws_secret, uploaded_file, key, **kwargs
         )
-    elif gs_bucket_name and gs_key and gs_secret and storage_place == 'gs':
+    if gs_bucket_name and gs_key and gs_secret and storage_place == 'gs':
         return upload_to_gs(
             gs_bucket_name, gs_key, gs_secret, uploaded_file, key, **kwargs
         )
-    else:
-        return upload_local(uploaded_file, key, upload_dir, **kwargs)
+    return upload_local(uploaded_file, key, upload_dir, **kwargs)
 
 
 def upload_local(uploaded_file, key, upload_dir='static/media/', **kwargs):