Skip to content
This repository was archived by the owner on Apr 14, 2026. It is now read-only.

Commit ab662a9

Browse files
committed
Try getting member by kth account earlier.
1 parent a6b29c3 commit ab662a9

1 file changed

Lines changed: 9 additions & 8 deletions

File tree

app/finger/models.py

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -148,6 +148,15 @@ def update_data(self, data):
148148
# If the above causes conflicts, add and fingerweb_identifier key to the member.
149149
if fields["identifier"]:
150150
member, _ = self.update_or_create(identifier__exact=fields["identifier"], defaults=fields)
151+
152+
elif fields["kth_account"]:
153+
try:
154+
member, _ = self.update_or_create(kth_account=fields["kth_account"], defaults=fields)
155+
except Member.DoesNotExist:
156+
pass
157+
except Member.MultipleObjectsReturned:
158+
raise Exception("Multiple members have kth account %r" % fields["kth_account"])
159+
151160
else:
152161
user_from_db = None
153162
if user.get("användarnamn"):
@@ -158,14 +167,6 @@ def update_data(self, data):
158167
except User.MultipleObjectsReturned:
159168
raise Exception("Multiple users have username %r" % user.get("användarnamn"))
160169

161-
if user_from_db is None and fields["kth_account"]:
162-
try:
163-
user_from_db = User.objects.get(member__kth_account=fields["kth_account"])
164-
except User.DoesNotExist:
165-
pass
166-
except User.MultipleObjectsReturned:
167-
raise Exception("Multiple users have kth account %r" % fields["kth_account"])
168-
169170
if self.is_valid_user(user) and user_from_db:
170171
member, _ = self.update_or_create(id=user_from_db.id, defaults=fields)
171172
elif "@" in fields.get("email", ""):

0 commit comments

Comments
 (0)