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

management: Clean up a variable name in knight.

Browse Source
This commit is contained in:
Tim Abbott
2019-07-08 13:42:05 -07:00
parent 3cc790420a
commit 9e9ddf464e
1 changed files with 7 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
zerver/management/commands/knight.py
View File

@@ -38,25 +38,25 @@ ONLY perform this on customer request from an authorized person.
email = options['email'] email = options['email']
realm = self.get_realm(options) realm = self.get_realm(options)
profile = self.get_user(email, realm) user = self.get_user(email, realm)
if options['grant']: if options['grant']:
if profile.has_perm(options['permission'], profile.realm): if user.has_perm(options['permission'], user.realm):
raise CommandError("User already has permission for this realm.") raise CommandError("User already has permission for this realm.")
else: else:
if options['ack']: if options['ack']:
do_change_is_admin(profile, True, permission=options['permission']) do_change_is_admin(user, True, permission=options['permission'])
print("Done!") print("Done!")
else: else:
print("Would have granted %s %s rights for %s" % ( print("Would have granted %s %s rights for %s" % (
email, options['permission'], profile.realm.string_id)) email, options['permission'], user.realm.string_id))
else: else:
if profile.has_perm(options['permission'], profile.realm): if user.has_perm(options['permission'], user.realm):
if options['ack']: if options['ack']:
do_change_is_admin(profile, False, permission=options['permission']) do_change_is_admin(user, False, permission=options['permission'])
print("Done!") print("Done!")
else: else:
print("Would have removed %s's %s rights on %s" % (email, options['permission'], print("Would have removed %s's %s rights on %s" % (email, options['permission'],
profile.realm.string_id)) user.realm.string_id))
else: else:
raise CommandError("User did not have permission for this realm!") raise CommandError("User did not have permission for this realm!")