Commit 6dea941

authored

Fix bug with social account adapter name override, in very specific conditions (cookiecutter#4650)

* Fix bug with social account adapter in specific condition * Slight refactoring of fix

1 parent befc571 commit 6dea941Copy full SHA for 6dea941

File tree

-8

lines changed

-8

lines changed

Lines changed: 9 additions & 8 deletions

Original file line number	Diff line number	Diff line change
`@@ -27,11 +27,12 @@ def populate_user(self, request: HttpRequest, sociallogin: SocialLogin, data: di`
`27`	`27`
`28`	`28`	`See: https://django-allauth.readthedocs.io/en/latest/advanced.html?#creating-and-populating-user-instances`
`29`	`29`	`"""`
`30`		`- user = sociallogin.user`
`31`		`- if name := data.get("name"):`
`32`		`- user.name = name`
`33`		`- elif first_name := data.get("first_name"):`
`34`		`- user.name = first_name`
`35`		`- if last_name := data.get("last_name"):`
`36`		`- user.name += f" {last_name}"`
`37`		`- return super().populate_user(request, sociallogin, data)`
	`30`	`+ user = super().populate_user(request, sociallogin, data)`
	`31`	`+ if not user.name:`
	`32`	`+ if name := data.get("name"):`
	`33`	`+ user.name = name`
	`34`	`+ elif first_name := data.get("first_name"):`
	`35`	`+ user.name = first_name`
	`36`	`+ if last_name := data.get("last_name"):`
	`37`	`+ user.name += f" {last_name}"`
	`38`	`+ return user`

Comments

(0)