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

refactor: Make acting_user a mandatory kwarg for do_set_realm_authentication_methods.

Browse Source
This commit is contained in:
shanukun
2021-03-07 10:30:29 +05:30
committed by Tim Abbott
parent c95061e9b9
commit 4dc62f962b
3 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
docs/tutorials/new-feature-tutorial.md
View File

@@ -321,7 +321,9 @@ field and send an event. For example:
# zerver/lib/actions.py
def do_set_realm_authentication_methods(realm: Realm, authentication_methods: Dict[str, bool]) -> None:
def do_set_realm_authentication_methods(
realm: Realm, authentication_methods: Dict[str, bool], *, acting_user: Optional[UserProfile]
) -> None:
for key, value in list(authentication_methods.items()):
index = getattr(realm.authentication_methods, key).number
realm.authentication_methods.set_bit(index, int(value))

2
zerver/lib/actions.py
View File

@@ -824,7 +824,7 @@ def do_set_realm_property(
def do_set_realm_authentication_methods(
realm: Realm, authentication_methods: Dict[str, bool], acting_user: Optional[UserProfile] = None
realm: Realm, authentication_methods: Dict[str, bool], *, acting_user: Optional[UserProfile]
) -> None:
old_value = realm.authentication_methods_dict()
for key, value in list(authentication_methods.items()):

2
zerver/tests/test_events.py
View File

@@ -1068,7 +1068,7 @@ class NormalActionsTest(BaseAction):
with fake_backends():
events = self.verify_action(
lambda: do_set_realm_authentication_methods(
self.user_profile.realm, auth_method_dict
self.user_profile.realm, auth_method_dict, acting_user=None
)
)