From 7dc7b1653cd1f2ca7fb8cc80197231d98afd144a Mon Sep 17 00:00:00 2001 From: Rishi Gupta Date: Mon, 13 Mar 2017 11:52:38 -0700 Subject: [PATCH] actions: Remove unnecessary domain in validate_user_access_to_subscribers. validate_user_access_to_subscribers_helper never uses stream_dict['realm__domain']. I imagine it was there originally to do the is_zephyr_mirror_realm check. --- zerver/lib/actions.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/zerver/lib/actions.py b/zerver/lib/actions.py index 5c5c577a2d..e9d591b673 100644 --- a/zerver/lib/actions.py +++ b/zerver/lib/actions.py @@ -1388,8 +1388,7 @@ def validate_user_access_to_subscribers(user_profile, stream): """ validate_user_access_to_subscribers_helper( user_profile, - {"realm__domain": stream.realm.domain, - "realm_id": stream.realm_id, + {"realm_id": stream.realm_id, "invite_only": stream.invite_only}, # We use a lambda here so that we only compute whether the # user is subscribed if we have to