Mirror
/
zulip
mirror of https://github.com/zulip/zulip.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

backends: Extract useful is_user_active function. 

This logic can be useful elsewhere, for checking whether user_profile is
active.
This commit is contained in:
Mateusz Mandera 2019-09-21 01:32:59 +02:00 committed by Tim Abbott
parent c19b7ec89d
commit 371a7768ad
1 changed files with 17 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
zproject/backends.py
View File

@ -109,6 +109,21 @@ def require_email_format_usernames(realm: Optional[Realm]=None) -> bool:
return False
return True
def is_user_active(user_profile: UserProfile, return_data: Optional[Dict[str, Any]]=None) -> bool:
if not user_profile.is_active:
if return_data is not None:
if user_profile.is_mirror_dummy:
# Record whether it's a mirror dummy account
return_data['is_mirror_dummy'] = True
return_data['inactive_user'] = True
return False
if user_profile.realm.deactivated:
if return_data is not None:
return_data['inactive_realm'] = True
return False
return True
def common_get_active_user(email: str, realm: Realm,
return_data: Optional[Dict[str, Any]]=None) -> Optional[UserProfile]:
"""This is the core common function used by essentially all
@ -128,17 +143,9 @@ def common_get_active_user(email: str, realm: Realm,
if return_data is not None:
return_data['invalid_subdomain'] = True
return None
if not user_profile.is_active:
if return_data is not None:
if user_profile.is_mirror_dummy:
# Record whether it's a mirror dummy account
return_data['is_mirror_dummy'] = True
return_data['inactive_user'] = True
return None
if user_profile.realm.deactivated:
if return_data is not None:
return_data['inactive_realm'] = True
if not is_user_active(user_profile, return_data):
return None
return user_profile
class ZulipAuthMixin: