Skip to content

Commit f1edb33

Browse files
fix: redact pending primary email before retirement deletion
1 parent 4eb27b5 commit f1edb33

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

  • common/djangoapps/student/models

common/djangoapps/student/models/user.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -923,10 +923,10 @@ def redact_pending_email_by_user_value(cls, value, field):
923923
Returns True if redacted, and False if no matching records found.
924924
"""
925925
filter_kwargs = {field: value}
926-
records_matching_user_value = cls.objects.filter(**filter_kwargs)
927-
if not records_matching_user_value.exists():
926+
records = list(cls.objects.filter(**filter_kwargs))
927+
if not records:
928928
return False
929-
for record in records_matching_user_value:
929+
for record in records:
930930
record.new_email = get_retired_email_by_email(record.new_email)
931931
record.save(update_fields=['new_email'])
932932
return True

0 commit comments

Comments
 (0)