diff options
| author | Hasan Ramezani <hasan.r67@gmail.com> | 2017-09-12 23:17:07 +0430 |
|---|---|---|
| committer | Tim Graham <timograham@gmail.com> | 2017-10-09 21:49:35 -0400 |
| commit | 6aec130a4c19ed17a61835af15b4fb2186dfacad (patch) | |
| tree | ac501febb66d7cfece1d2edce91a9d5d8ce2222f /django | |
| parent | 22ff4f81b1859ed7af7999ef479a10b5d31a2164 (diff) | |
Fixed #28591 -- Added an error message for createsuperuser --username= (blank).
Diffstat (limited to 'django')
| -rw-r--r-- | django/contrib/auth/management/commands/createsuperuser.py | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/django/contrib/auth/management/commands/createsuperuser.py b/django/contrib/auth/management/commands/createsuperuser.py index f9a198f0d4..7e19ef9955 100644 --- a/django/contrib/auth/management/commands/createsuperuser.py +++ b/django/contrib/auth/management/commands/createsuperuser.py @@ -68,6 +68,7 @@ class Command(BaseCommand): # Same as user_data but with foreign keys as fake model instances # instead of raw IDs. fake_user_data = {} + verbose_field_name = self.username_field.verbose_name # Do quick and dirty validation if --noinput if not options['interactive']: @@ -96,7 +97,6 @@ class Command(BaseCommand): raise NotRunningInTTYException("Not running in a TTY") # Get a username - verbose_field_name = self.username_field.verbose_name while username is None: input_msg = capfirst(verbose_field_name) if default_username: @@ -121,6 +121,9 @@ class Command(BaseCommand): self.stderr.write("Error: That %s is already taken." % verbose_field_name) username = None + if not username: + raise CommandError('%s cannot be blank.' % capfirst(verbose_field_name)) + for field_name in self.UserModel.REQUIRED_FIELDS: field = self.UserModel._meta.get_field(field_name) user_data[field_name] = options[field_name] |