Skip to content

Commit

Permalink
Merge pull request #809 from wklken/fix_profile_celery_task
Browse files Browse the repository at this point in the history
fix(profile/create): fail when got no settings of expired_after_days
  • Loading branch information
wklken authored Nov 22, 2022
2 parents 1b8a27b + cb57bff commit 57f49a3
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion src/api/bkuser_core/api/web/profile/views.py
Original file line number Diff line number Diff line change
Expand Up @@ -292,7 +292,12 @@ def create(self, request, *args, **kwargs):
meta = SettingMeta.objects.filter(
key="expired_after_days", namespace=SettingsEnableNamespaces.ACCOUNT.value
).first()
expired_after_days = Setting.objects.filter(category_id=category_id, meta=meta).first().value

# NOTE: maybe None here if the meta is not set
expired_after_days = -1
expired_after_days_setting = Setting.objects.filter(category_id=category_id, meta=meta).first()
if expired_after_days_setting:
expired_after_days = expired_after_days_setting.value

# 账户有效期,不传,默认设置为目录设置项
if expired_after_days == -1:
Expand Down

0 comments on commit 57f49a3

Please sign in to comment.