From 8e4353c92575afedc4cd9e358a40d6a4a20d5587 Mon Sep 17 00:00:00 2001 From: Adam Goldsmith Date: Fri, 5 Jan 2024 14:50:15 -0500 Subject: [PATCH] membershipworks/ucsAccounts: Don't set email if empty in MembershipWorks --- membershipworks/tasks/ucsAccounts.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/membershipworks/tasks/ucsAccounts.py b/membershipworks/tasks/ucsAccounts.py index 14e642b..2e1f788 100644 --- a/membershipworks/tasks/ucsAccounts.py +++ b/membershipworks/tasks/ucsAccounts.py @@ -90,13 +90,15 @@ async def sync_member(user_mod, member: Member): # "userexpiry": member["Renewal Date"], # "pwdChangeNextLogin": "1", # User has to change password on next login # "sambaLogonHours": "", # Permitted times for Windows logins - "e-mail": [member.email], # ([]) E-mail address "phone": [member.phone], # Telephone number # "PasswordRecoveryMobile": member["Phone"], # Mobile phone number - "PasswordRecoveryEmail": member.email, } ) + if member.email: + user.props["e-mail"] = [member.email] # ([]) E-mail address + user.props["PasswordRecoveryEmail"] = member.email + new_groups = [ f"cn={sanitize_group_name(flag.name)},{GROUP_BASE}" async for flag in member.flags.filter(