Skip to content

Commit

Permalink
Fix arg shadowing in get_or_create_klant_for_new_user
Browse files Browse the repository at this point in the history
  • Loading branch information
swrichards committed Jan 7, 2025
1 parent 235d0f0 commit 0da54c4
Showing 1 changed file with 9 additions and 4 deletions.
13 changes: 9 additions & 4 deletions src/open_inwoner/accounts/views/signals.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@ def get_or_create_klant_for_new_user(
sender: type, instance: User, created: bool, **kwargs
) -> None:
if not created:
logger.info("No klanten sync performed because user has just been created")
return

user = instance
Expand All @@ -34,13 +35,14 @@ def get_or_create_klant_for_new_user(
):
return

partij, created = service.get_or_create_partij_for_user(
partij, partij_created = service.get_or_create_partij_for_user(
fetch_params=fetch_params, user=user
)
if not partij:
logger.error("Failed to create partij for new user %s", user)
return
elif not created:

if not partij_created:
service.update_user_from_partij(partij_uuid=partij["uuid"], user=user)

logger.info("Created partij %s for new user %s", partij, user)
Expand All @@ -59,11 +61,14 @@ def get_or_create_klant_for_new_user(
):
return

klant, created = service.get_or_create_klant(fetch_params=fetch_params, user=user)
klant, klant_created = service.get_or_create_klant(
fetch_params=fetch_params, user=user
)
if not klant:
logger.error("Failed to create klant for new user %s", user)
return
elif not created:

if not klant_created:
service.update_user_from_klant(klant, user)

logger.info("Created klant %s for new user %s", klant, user)

0 comments on commit 0da54c4

Please sign in to comment.