paulmataruso/zulip
1
0
Fork 0
mirror of https://github.com/zulip/zulip.git synced 2025-11-16 03:41:58 +00:00
Code Issues Packages Projects Releases Wiki Activity

create_user: Remove user_profile_id option.

Browse Source 
This hasn't been used in years, had no test coverage, and doesn't have
a clear use case.
This commit is contained in:
Tim Abbott
2018-05-15 13:51:56 -07:00
parent e731aeda44
commit 191e6dc13a
2 changed files with 1 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
tools/test-backend
View File

@@ -54,7 +54,6 @@ not_yet_fully_covered = {
'zerver/lib/bugdown/__init__.py',
'zerver/lib/cache.py',
'zerver/lib/cache_helpers.py',
'zerver/lib/create_user.py',
'zerver/lib/exceptions.py',
'zerver/lib/i18n.py',
'zerver/lib/notifications.py',

6
zerver/lib/create_user.py
View File

@@ -60,8 +60,7 @@ def create_user(email: str, password: Optional[str], realm: Realm,
is_mirror_dummy: bool = False,
default_sending_stream: Optional[Stream] = None,
default_events_register_stream: Optional[Stream] = None,
default_all_public_streams: Optional[bool] = None,
user_profile_id: Optional[int] = None) -> UserProfile:
default_all_public_streams: Optional[bool] = None) -> UserProfile:
user_profile = create_user_profile(realm, email, password, active, bot_type,
full_name, short_name, bot_owner,
is_mirror_dummy, tos_version, timezone)
@@ -74,9 +73,6 @@ def create_user(email: str, password: Optional[str], realm: Realm,
if default_all_public_streams is not None:
user_profile.default_all_public_streams = default_all_public_streams
if user_profile_id is not None:
user_profile.id = user_profile_id
user_profile.save()
recipient = Recipient.objects.create(type_id=user_profile.id,
type=Recipient.PERSONAL)