mirror of
https://github.com/cisagov/manage.get.gov.git
synced 2025-07-24 03:30:50 +02:00
Go back to using get_or_create
This commit is contained in:
parent
63979afe32
commit
57f3428025
1 changed files with 4 additions and 9 deletions
|
@ -62,15 +62,10 @@ class DomainInvitation(TimeStampedModel):
|
|||
)
|
||||
|
||||
# and create a role for that user on this domain
|
||||
try:
|
||||
UserDomainRole.objects.get(
|
||||
user=user, domain=self.domain, role=UserDomainRole.Roles.ADMIN
|
||||
)
|
||||
except UserDomainRole.DoesNotExist:
|
||||
UserDomainRole.objects.create(
|
||||
user=user, domain=self.domain, role=UserDomainRole.Roles.ADMIN
|
||||
)
|
||||
else:
|
||||
_, created = UserDomainRole.objects.get_or_create(
|
||||
user=user, domain=self.domain, role=UserDomainRole.Roles.ADMIN
|
||||
)
|
||||
if not created:
|
||||
# something strange happened and this role already existed when
|
||||
# the invitation was retrieved. Log that this occurred.
|
||||
logger.warn(
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue