From 8d82b7e85bc6101a387d34fd88877196f3939cf7 Mon Sep 17 00:00:00 2001 From: Tanner Collin Date: Sun, 27 Sep 2020 04:07:42 +0000 Subject: [PATCH] Don't change ldap group membership if it's not configured --- apiserver/apiserver/api/views.py | 18 ++++++++++-------- 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/apiserver/apiserver/api/views.py b/apiserver/apiserver/api/views.py index 4e7ccbe..a9ba890 100644 --- a/apiserver/apiserver/api/views.py +++ b/apiserver/apiserver/api/views.py @@ -217,17 +217,19 @@ class TrainingViewSet(Base, Retrieve, Create, Update): elif session.course.id == 247: member.rabbit_cert_date = utils.today_alberta_tz() if status == 'Attended' else None - if status == 'Attended': - utils_ldap.add_to_group(member, 'Laser Users') - else: - utils_ldap.remove_from_group(member, 'Laser Users') + if utils_ldap.is_configured(): + if status == 'Attended': + utils_ldap.add_to_group(member, 'Laser Users') + else: + utils_ldap.remove_from_group(member, 'Laser Users') elif session.course.id == 321: member.trotec_cert_date = utils.today_alberta_tz() if status == 'Attended' else None - if status == 'Attended': - utils_ldap.add_to_group(member, 'Trotec Users') - else: - utils_ldap.remove_from_group(member, 'Trotec Users') + if utils_ldap.is_configured(): + if status == 'Attended': + utils_ldap.add_to_group(member, 'Trotec Users') + else: + utils_ldap.remove_from_group(member, 'Trotec Users') member.save() # TODO: turn these into @actions