Skip to content

Commit e30a8a7

Browse files
authored
Merge pull request #369 from VOKO-Utrecht/368-fout-bij-anonimiseren-account
368 fout bij anonimiseren account
2 parents 73998f1 + 374e5a3 commit e30a8a7

File tree

1 file changed

+13
-11
lines changed

1 file changed

+13
-11
lines changed

webapp/accounts/admin.py

Lines changed: 13 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ def anonymize_user(modeladmin, request, queryset):
7878
""" Anonymize user to comply with GDPR regulations"""
7979
for user in queryset:
8080

81-
balance = user.balance.credit() or -user.balance.debit()
81+
balance = user.balance and (user.balance.credit() or -user.balance.debit())
8282
if balance:
8383
messages.add_message(
8484
request, messages.WARNING,
@@ -100,16 +100,18 @@ def anonymize_user(modeladmin, request, queryset):
100100
try:
101101
# Anonymize user profile
102102
profile = user.userprofile
103-
profile.phone_number = ''
104-
profile.notes = ''
105-
profile.save()
106-
107-
# Anonymize address
108-
address = profile.address
109-
address.street_and_number = ''
110-
address.zip_code = '0000'
111-
address.city = ''
112-
address.save()
103+
if profile:
104+
profile.phone_number = ''
105+
profile.notes = ''
106+
profile.save()
107+
108+
# Anonymize address
109+
address = profile.address
110+
if address:
111+
address.street_and_number = ''
112+
address.zip_code = '0000'
113+
address.city = ''
114+
address.save()
113115

114116
except (UserProfile.DoesNotExist, Address.DoesNotExist):
115117
# Can happen with inactive users

0 commit comments

Comments
 (0)