Skip to content

Commit cf0061c

Browse files
committed
Fixed logic of gathering which personal information needs consent confirmation in the code that requests consent on login.
- Legacy-Id: 15218
1 parent c3041d7 commit cf0061c

1 file changed

Lines changed: 6 additions & 7 deletions

File tree

ietf/ietfauth/views.py

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -590,16 +590,15 @@ def login(request, extra_context=None):
590590
person = user.person
591591
if person.name != person.name_from_draft:
592592
require_consent.append("full name")
593-
elif person.ascii != person.name_from_draft:
593+
if person.ascii != person.name_from_draft:
594594
require_consent.append("ascii name")
595-
elif person.biography:
595+
if person.biography:
596596
require_consent.append("biography")
597-
elif user.communitylist_set.exists():
597+
if user.communitylist_set.exists():
598598
require_consent.append("draft notification subscription(s)")
599-
else:
600-
for email in person.email_set.all():
601-
if not email.origin.split(':')[0] in ['author', 'role', 'reviewer', 'liaison', 'shepherd', ]:
602-
require_consent.append("email address(es)")
599+
for email in person.email_set.all():
600+
if not email.origin.split(':')[0] in ['author', 'role', 'reviewer', 'liaison', 'shepherd', ]:
601+
require_consent.append("email address(es)")
603602
if user:
604603
try:
605604
identify_hasher(user.password)

0 commit comments

Comments
 (0)