diff --git a/src/backend/core/models.py b/src/backend/core/models.py index 4c24662..c3a8c29 100644 --- a/src/backend/core/models.py +++ b/src/backend/core/models.py @@ -814,7 +814,7 @@ class Team(MP_Node, BaseModel): except AttributeError: try: role = self.accesses.filter(user=user).values("role")[0]["role"] - except (TeamAccess.DoesNotExist, IndexError): + except TeamAccess.DoesNotExist, IndexError: role = None is_owner_or_admin = role in [RoleChoices.OWNER, RoleChoices.ADMIN] @@ -901,7 +901,7 @@ class TeamAccess(BaseModel): role = self._meta.model.objects.filter( team=self.team_id, user=user ).values("role")[0]["role"] - except (self._meta.model.DoesNotExist, IndexError): + except self._meta.model.DoesNotExist, IndexError: role = None is_team_owner_or_admin = role in [RoleChoices.OWNER, RoleChoices.ADMIN] @@ -1100,7 +1100,7 @@ class Invitation(BaseInvitation): role = self.team.accesses.filter(user=user).values("role")[0][ "role" ] - except (self._meta.model.DoesNotExist, IndexError): + except self._meta.model.DoesNotExist, IndexError: role = None can_delete = role in [RoleChoices.OWNER, RoleChoices.ADMIN] diff --git a/src/backend/core/plugins/urls.py b/src/backend/core/plugins/urls.py index ad25cd0..0a640be 100644 --- a/src/backend/core/plugins/urls.py +++ b/src/backend/core/plugins/urls.py @@ -9,7 +9,7 @@ plugins_urlpatterns = [] for app in settings.INSTALLED_PLUGINS: try: plugins_urlpatterns.append(path("", include(f"{app}.urls"))) - except (ImportError, AttributeError): + except ImportError, AttributeError: # Skip if app doesn't have urls.py continue diff --git a/src/backend/mailbox_manager/models.py b/src/backend/mailbox_manager/models.py index 4999141..a4a7cd9 100644 --- a/src/backend/mailbox_manager/models.py +++ b/src/backend/mailbox_manager/models.py @@ -111,7 +111,7 @@ class MailDomain(BaseModel): if user.is_authenticated: try: role = self.accesses.filter(user=user).values("role")[0]["role"] - except (MailDomainAccess.DoesNotExist, IndexError): + except MailDomainAccess.DoesNotExist, IndexError: role = None is_owner_or_admin = role in [ @@ -224,7 +224,7 @@ class MailDomainAccess(BaseModel): authenticated_user_role = user.mail_domain_accesses.get( domain=self.domain ).role - except (MailDomainAccess.DoesNotExist, IndexError): + except MailDomainAccess.DoesNotExist, IndexError: return [] # only an owner can set an owner role @@ -252,7 +252,7 @@ class MailDomainAccess(BaseModel): if user.is_authenticated: try: role = user.mail_domain_accesses.filter(domain=self.domain).get().role - except (MailDomainAccess.DoesNotExist, IndexError): + except MailDomainAccess.DoesNotExist, IndexError: role = None is_owner_or_admin = role in [ @@ -459,7 +459,7 @@ class MailDomainInvitation(BaseInvitation): role = self.domain.accesses.filter(user=user).values("role")[0][ "role" ] - except (self._meta.model.DoesNotExist, IndexError): + except self._meta.model.DoesNotExist, IndexError: role = None can_delete = role in [ @@ -515,7 +515,7 @@ class Alias(BaseModel): edit aliases, but also viewer if the alias points to their email.""" try: role = user.mail_domain_accesses.get(domain=self.domain).role - except (MailDomainAccess.DoesNotExist, IndexError): + except MailDomainAccess.DoesNotExist, IndexError: role = None is_owner_or_admin = role in [