🚨(lint) fix linter warnings
fix linter warnings after ruff upgrade
This commit is contained in:
committed by
Marie
parent
b1e0a24626
commit
b9c69e7c08
@@ -814,7 +814,7 @@ class Team(MP_Node, BaseModel):
|
|||||||
except AttributeError:
|
except AttributeError:
|
||||||
try:
|
try:
|
||||||
role = self.accesses.filter(user=user).values("role")[0]["role"]
|
role = self.accesses.filter(user=user).values("role")[0]["role"]
|
||||||
except (TeamAccess.DoesNotExist, IndexError):
|
except TeamAccess.DoesNotExist, IndexError:
|
||||||
role = None
|
role = None
|
||||||
|
|
||||||
is_owner_or_admin = role in [RoleChoices.OWNER, RoleChoices.ADMIN]
|
is_owner_or_admin = role in [RoleChoices.OWNER, RoleChoices.ADMIN]
|
||||||
@@ -901,7 +901,7 @@ class TeamAccess(BaseModel):
|
|||||||
role = self._meta.model.objects.filter(
|
role = self._meta.model.objects.filter(
|
||||||
team=self.team_id, user=user
|
team=self.team_id, user=user
|
||||||
).values("role")[0]["role"]
|
).values("role")[0]["role"]
|
||||||
except (self._meta.model.DoesNotExist, IndexError):
|
except self._meta.model.DoesNotExist, IndexError:
|
||||||
role = None
|
role = None
|
||||||
|
|
||||||
is_team_owner_or_admin = role in [RoleChoices.OWNER, RoleChoices.ADMIN]
|
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 = self.team.accesses.filter(user=user).values("role")[0][
|
||||||
"role"
|
"role"
|
||||||
]
|
]
|
||||||
except (self._meta.model.DoesNotExist, IndexError):
|
except self._meta.model.DoesNotExist, IndexError:
|
||||||
role = None
|
role = None
|
||||||
|
|
||||||
can_delete = role in [RoleChoices.OWNER, RoleChoices.ADMIN]
|
can_delete = role in [RoleChoices.OWNER, RoleChoices.ADMIN]
|
||||||
|
|||||||
@@ -9,7 +9,7 @@ plugins_urlpatterns = []
|
|||||||
for app in settings.INSTALLED_PLUGINS:
|
for app in settings.INSTALLED_PLUGINS:
|
||||||
try:
|
try:
|
||||||
plugins_urlpatterns.append(path("", include(f"{app}.urls")))
|
plugins_urlpatterns.append(path("", include(f"{app}.urls")))
|
||||||
except (ImportError, AttributeError):
|
except ImportError, AttributeError:
|
||||||
# Skip if app doesn't have urls.py
|
# Skip if app doesn't have urls.py
|
||||||
continue
|
continue
|
||||||
|
|
||||||
|
|||||||
@@ -111,7 +111,7 @@ class MailDomain(BaseModel):
|
|||||||
if user.is_authenticated:
|
if user.is_authenticated:
|
||||||
try:
|
try:
|
||||||
role = self.accesses.filter(user=user).values("role")[0]["role"]
|
role = self.accesses.filter(user=user).values("role")[0]["role"]
|
||||||
except (MailDomainAccess.DoesNotExist, IndexError):
|
except MailDomainAccess.DoesNotExist, IndexError:
|
||||||
role = None
|
role = None
|
||||||
|
|
||||||
is_owner_or_admin = role in [
|
is_owner_or_admin = role in [
|
||||||
@@ -224,7 +224,7 @@ class MailDomainAccess(BaseModel):
|
|||||||
authenticated_user_role = user.mail_domain_accesses.get(
|
authenticated_user_role = user.mail_domain_accesses.get(
|
||||||
domain=self.domain
|
domain=self.domain
|
||||||
).role
|
).role
|
||||||
except (MailDomainAccess.DoesNotExist, IndexError):
|
except MailDomainAccess.DoesNotExist, IndexError:
|
||||||
return []
|
return []
|
||||||
|
|
||||||
# only an owner can set an owner role
|
# only an owner can set an owner role
|
||||||
@@ -252,7 +252,7 @@ class MailDomainAccess(BaseModel):
|
|||||||
if user.is_authenticated:
|
if user.is_authenticated:
|
||||||
try:
|
try:
|
||||||
role = user.mail_domain_accesses.filter(domain=self.domain).get().role
|
role = user.mail_domain_accesses.filter(domain=self.domain).get().role
|
||||||
except (MailDomainAccess.DoesNotExist, IndexError):
|
except MailDomainAccess.DoesNotExist, IndexError:
|
||||||
role = None
|
role = None
|
||||||
|
|
||||||
is_owner_or_admin = role in [
|
is_owner_or_admin = role in [
|
||||||
@@ -459,7 +459,7 @@ class MailDomainInvitation(BaseInvitation):
|
|||||||
role = self.domain.accesses.filter(user=user).values("role")[0][
|
role = self.domain.accesses.filter(user=user).values("role")[0][
|
||||||
"role"
|
"role"
|
||||||
]
|
]
|
||||||
except (self._meta.model.DoesNotExist, IndexError):
|
except self._meta.model.DoesNotExist, IndexError:
|
||||||
role = None
|
role = None
|
||||||
|
|
||||||
can_delete = role in [
|
can_delete = role in [
|
||||||
@@ -515,7 +515,7 @@ class Alias(BaseModel):
|
|||||||
edit aliases, but also viewer if the alias points to their email."""
|
edit aliases, but also viewer if the alias points to their email."""
|
||||||
try:
|
try:
|
||||||
role = user.mail_domain_accesses.get(domain=self.domain).role
|
role = user.mail_domain_accesses.get(domain=self.domain).role
|
||||||
except (MailDomainAccess.DoesNotExist, IndexError):
|
except MailDomainAccess.DoesNotExist, IndexError:
|
||||||
role = None
|
role = None
|
||||||
|
|
||||||
is_owner_or_admin = role in [
|
is_owner_or_admin = role in [
|
||||||
|
|||||||
Reference in New Issue
Block a user