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

show_admins: Rewrite to use management library.

Browse Source 
This makes this command more standardized, and helps avoid future bugs
like the one fixed in the last commit.
This commit is contained in:
Tim Abbott
2018-11-29 11:39:53 -08:00
parent a36cb9b247
commit 92ea9a0729
1 changed files with 9 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
zerver/management/commands/show_admins.py
View File

@@ -3,24 +3,20 @@ import sys
from argparse import ArgumentParser
from typing import Any
from django.core.management.base import BaseCommand, CommandError
from django.core.management.base import CommandError
from zerver.lib.management import ZulipBaseCommand
from zerver.models import get_realm
class Command(BaseCommand):
class Command(ZulipBaseCommand):
help = """Show the admins in a realm."""
def add_arguments(self, parser: ArgumentParser) -> None:
parser.add_argument('realm', metavar='<realm>', type=str,
help="realm to show admins for")
def handle(self, *args: Any, **options: str) -> None:
realm_name = options['realm']
realm = get_realm(realm_name)
if realm is None:
raise CommandError('There is no realm called %s.' % (realm_name,))
self.add_realm_args(parser, required=True)
def handle(self, *args: Any, **options: Any) -> None:
realm = self.get_realm(options)
assert realm is not None # True because of required=True above
users = realm.get_admin_users()
if users:
@@ -30,4 +26,5 @@ class Command(BaseCommand):
else:
print('There are no admins for this realm!')
print('\nYou can use the "knight" management command to knight admins.')
print('\nYou can use the "knight" management command to make more users admins.')
print('\nOr with the --revoke argument, remove admin status from users.')