From b0d2f304d65e126f2cd1ba48acb17faa06dce421 Mon Sep 17 00:00:00 2001 From: "deepsource-autofix[bot]" <62050782+deepsource-autofix[bot]@users.noreply.github.com> Date: Tue, 5 Oct 2021 08:56:54 +0530 Subject: [PATCH] Refactor unnecessary `else` / `elif` when `if` block has a `raise` statement (#1) Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com> --- drfaddons/auth.py | 2 +- drfaddons/utils.py | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drfaddons/auth.py b/drfaddons/auth.py index 52d6b8d..009282f 100644 --- a/drfaddons/auth.py +++ b/drfaddons/auth.py @@ -91,7 +91,7 @@ def get_jwt_value(self, request): msg = _("Invalid Authorization header. No credentials provided.") raise exceptions.AuthenticationFailed(msg) - elif len(auth) > 2: + if len(auth) > 2: msg = _( "Invalid Authorization header. Credentials string " "should not contain spaces." diff --git a/drfaddons/utils.py b/drfaddons/utils.py index dba4e01..41ccd5b 100644 --- a/drfaddons/utils.py +++ b/drfaddons/utils.py @@ -237,7 +237,7 @@ def send_message( if not len(recip) > 0: raise ValueError("No recipient to send message.") # Check if the value of recipient is valid (min length: a@b.c) - elif len(recip[0]) < 5: + if len(recip[0]) < 5: raise ValueError("Invalid recipient.") # Check if all recipient in list are of same type @@ -245,7 +245,7 @@ def send_message( for ind in range(len(recip)): if validate_email(recip[ind]) is not is_email: raise ValueError("All recipient should be of same type.") - elif not is_email: + if not is_email: recip[ind] = get_mobile_number(recip[ind]) # Check if fallback email is indeed an email