Skip to content

Commit f83ab64

Browse files
fix: 🐛 linter
1 parent 5beefe4 commit f83ab64

2 files changed

Lines changed: 24 additions & 4 deletions

File tree

app/features/orcid/routes.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,11 @@ def authorize():
8989
orcid_id = (user_info.get("sub") or "").strip()
9090
external_repo = ExternalIdentityRepository()
9191
external_repo.create(
92-
user_id=current_user.id, provider="orcid", provider_id=orcid_id, provider_username=orcid_id, email=connect_email
92+
user_id=current_user.id,
93+
provider="orcid",
94+
provider_id=orcid_id,
95+
provider_username=orcid_id,
96+
email=connect_email,
9397
)
9498
flash("ORCID account connected successfully", "success")
9599
return redirect(url_for("profile.edit_profile"))

app/features/orcid/services.py

Lines changed: 19 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -105,17 +105,33 @@ def get_or_create_user(self, user_info, email=None):
105105
if email:
106106
existing_by_email = self.user_repository.get_by_email(email)
107107
if existing_by_email:
108-
external_repo.create(commit=False, user_id=existing_by_email.id, provider="orcid", provider_id=orcid_id, provider_username=orcid_id, email=email)
108+
external_repo.create(
109+
commit=False,
110+
user_id=existing_by_email.id,
111+
provider="orcid",
112+
provider_id=orcid_id,
113+
provider_username=orcid_id,
114+
email=email,
115+
)
109116
self.repository.session.commit()
110117
return existing_by_email, None
111118

112119
# 3. Create new user
113-
user = self.user_repository.create(commit=False, email=email, password=secrets.token_urlsafe(24), active=True)
120+
user = self.user_repository.create(
121+
commit=False, email=email, password=secrets.token_urlsafe(24), active=True
122+
)
114123
profile = self.user_profile_repository.create(
115124
commit=False, user_id=user.id, name=given_name, surname=family_name, affiliation=affiliation
116125
)
117126
self.repository.create(commit=False, orcid_id=orcid_id, profile_id=profile.id)
118-
external_repo.create(commit=False, user_id=user.id, provider="orcid", provider_id=orcid_id, provider_username=orcid_id, email=email)
127+
external_repo.create(
128+
commit=False,
129+
user_id=user.id,
130+
provider="orcid",
131+
provider_id=orcid_id,
132+
provider_username=orcid_id,
133+
email=email,
134+
)
119135
self.repository.session.commit()
120136
return user, None
121137

0 commit comments

Comments
 (0)