Refactored users.forms to make the code more readeable
from django.contrib.auth import forms as admin_forms
from django.contrib.auth import get_user_model
-from django.core.exceptions import ValidationError
from django.utils.translation import gettext_lazy as _
User = get_user_model()
@@ -12,20 +11,9 @@ class UserChangeForm(admin_forms.UserChangeForm):
class UserCreationForm(admin_forms.UserCreationForm):
-
- error_message = admin_forms.UserCreationForm.error_messages.update(
- {"duplicate_username": _("This username has already been taken.")}
- )
-
class Meta(admin_forms.UserCreationForm.Meta):
model = User
- def clean_username(self):
- username = self.cleaned_data["username"]
-
- try:
- User.objects.get(username=username)
- except User.DoesNotExist:
- return username
-
- raise ValidationError(self.error_messages["duplicate_username"])
+ error_messages = {
+ "username": {"unique": _("This username has already been taken.")}
+ }
