Skip to content

Commit f042dba

Browse files
committed
Fixing unset intro
1 parent 38df698 commit f042dba

File tree

2 files changed

+8
-2
lines changed

2 files changed

+8
-2
lines changed

conditional/blueprints/slideshow.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
from conditional.util.auth import get_user
1313
from conditional.util.flask import render_template
1414
from conditional.util.ldap import ldap_is_eval_director, ldap_is_intromember, ldap_set_failed, ldap_set_bad_standing, \
15-
ldap_set_intro_member, ldap_set_inactive, ldap_get_member
15+
ldap_set_inactive, ldap_get_member, ldap_set_not_intro_member
1616

1717
logger = structlog.get_logger()
1818

@@ -135,7 +135,7 @@ def slideshow_spring_review(user_dict=None):
135135

136136
if status == "Passed":
137137
if ldap_is_intromember(account):
138-
ldap_set_intro_member(account)
138+
ldap_set_not_intro_member(account)
139139
elif status == "Failed":
140140
if ldap_is_intromember(account):
141141
ldap_set_failed(account)

conditional/util/ldap.py

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -132,6 +132,12 @@ def ldap_set_intro_member(account):
132132
ldap_get_member.cache_clear()
133133

134134

135+
def ldap_set_not_intro_member(account):
136+
_ldap_remove_member_from_group(account, 'intromembers')
137+
ldap_get_intro_members().cache_clear()
138+
ldap_get_member.cache_clear()
139+
140+
135141
def ldap_set_current_student(account):
136142
_ldap_add_member_to_group(account, 'current_student')
137143
ldap_get_current_students.cache_clear()

0 commit comments

Comments
 (0)