Нужно ли переопределять поле username в Django?

Question

[PenguinVasily]

Расширил модель пользователя с помощью AbstractUser. Но при регистрации пользователя, Django ругается, что поле username не уникально. Мне нужно было переопределить это поле? Как-то так: username=None? Или же у меня ошибка при описании модели?

Модель:

class User(AbstractUser):
    phone_number = models.CharField(max_length=12, unique=True, verbose_name='Телефон')
    birthdate = models.DateField(verbose_name='Дата рождения', blank=True)

    USERNAME_FIELD = 'phone_number'

    def __str__(self):
        return getattr(self, self.USERNAME_FIELD)

Comments

[Александр Нестеров]

Но при регистрации пользователя, Django ругается, что поле username не уникально.

По формулировке, предположу, что в базе уже запись в которой есть поле phone_number/username со значением которое вы вводите.
Но здесь не битва экстрасенсов - советую привести полный текст ошибки.

[PenguinVasily]

Александр Нестеров,
Environment:

Request Method: POST
Request URL: 127.0.0.1:8000/sign-up

Django Version: 4.0.5
Python Version: 3.10.7
Installed Applications:
['django.contrib.admin',
'django.contrib.auth',
'django.contrib.contenttypes',
'django.contrib.sessions',
'django.contrib.messages',
'django.contrib.staticfiles',
'core.apps.CoreConfig']
Installed Middleware:
['django.middleware.security.SecurityMiddleware',
'django.contrib.sessions.middleware.SessionMiddleware',
'django.middleware.common.CommonMiddleware',
'django.middleware.csrf.CsrfViewMiddleware',
'django.contrib.auth.middleware.AuthenticationMiddleware',
'django.contrib.messages.middleware.MessageMiddleware',
'django.middleware.clickjacking.XFrameOptionsMiddleware']

Traceback (most recent call last):
File "C:\Users\semen\AppData\Local\Programs\Python\Python310\lib\site-packages\django\db\backends\utils.py", line 89, in _execute
return self.cursor.execute(sql, params)
File "C:\Users\semen\AppData\Local\Programs\Python\Python310\lib\site-packages\django\db\backends\sqlite3\base.py", line 477, in execute
return Database.Cursor.execute(self, query, params)

The above exception (UNIQUE constraint failed: core_user.username) was the direct cause of the following exception:
File "C:\Users\semen\AppData\Local\Programs\Python\Python310\lib\site-packages\django\core\handlers\exception.py", line 55, in inner
response = get_response(request)
File "C:\Users\semen\AppData\Local\Programs\Python\Python310\lib\site-packages\django\core\handlers\base.py", line 197, in _get_response
response = wrapped_callback(request, *callback_args, **callback_kwargs)
File "D:\DjangoApps\CustomUsers\core\views.py", line 11, in sign_up
user.save()
File "C:\Users\semen\AppData\Local\Programs\Python\Python310\lib\site-packages\django\contrib\auth\base_user.py", line 68, in save
super().save(*args, **kwargs)
File "C:\Users\semen\AppData\Local\Programs\Python\Python310\lib\site-packages\django\db\models\base.py", line 806, in save
self.save_base(
File "C:\Users\semen\AppData\Local\Programs\Python\Python310\lib\site-packages\django\db\models\base.py", line 857, in save_base
updated = self._save_table(
File "C:\Users\semen\AppData\Local\Programs\Python\Python310\lib\site-packages\django\db\models\base.py", line 1000, in _save_table
results = self._do_insert(
File "C:\Users\semen\AppData\Local\Programs\Python\Python310\lib\site-packages\django\db\models\base.py", line 1041, in _do_insert
return manager._insert(
File "C:\Users\semen\AppData\Local\Programs\Python\Python310\lib\site-packages\django\db\models\manager.py", line 85, in manager_method
return getattr(self.get_queryset(), name)(*args, **kwargs)
File "C:\Users\semen\AppData\Local\Programs\Python\Python310\lib\site-packages\django\db\models\query.py", line 1434, in _insert
return query.get_compiler(using=using).execute_sql(returning_fields)
File "C:\Users\semen\AppData\Local\Programs\Python\Python310\lib\site-packages\django\db\models\sql\compiler.py", line 1621, in execute_sql
cursor.execute(sql, params)
File "C:\Users\semen\AppData\Local\Programs\Python\Python310\lib\site-packages\django\db\backends\utils.py", line 103, in execute
return super().execute(sql, params)
File "C:\Users\semen\AppData\Local\Programs\Python\Python310\lib\site-packages\django\db\backends\utils.py", line 67, in execute
return self._execute_with_wrappers(
File "C:\Users\semen\AppData\Local\Programs\Python\Python310\lib\site-packages\django\db\backends\utils.py", line 80, in _execute_with_wrappers
return executor(sql, params, many, context)
File "C:\Users\semen\AppData\Local\Programs\Python\Python310\lib\site-packages\django\db\backends\utils.py", line 84, in _execute
with self.db.wrap_database_errors:
File "C:\Users\semen\AppData\Local\Programs\Python\Python310\lib\site-packages\django\db\utils.py", line 91, in __exit__
raise dj_exc_value.with_traceback(traceback) from exc_value
File "C:\Users\semen\AppData\Local\Programs\Python\Python310\lib\site-packages\django\db\backends\utils.py", line 89, in _execute
return self.cursor.execute(sql, params)
File "C:\Users\semen\AppData\Local\Programs\Python\Python310\lib\site-packages\django\db\backends\sqlite3\base.py", line 477, in execute
return Database.Cursor.execute(self, query, params)

Exception Type: IntegrityError at /sign-up/
Exception Value: UNIQUE constraint failed: core_user.username

[PenguinVasily]

Александр Нестеров, я думал, что поле phone_number - заменит поле username. Однако, в бд, в поле username записывается пустая строка.

[Александр Нестеров]

PenguinVasily, ну, собственно - да, проблема с дублирующийся значениями. Самый простой вариант удалить базу и выполнить миграции по новой.

[PenguinVasily]

Александр Нестеров, изначально это сделал.

[PenguinVasily]

Dr. Bacon, не понимаю откуда такая уверенность :)

[PenguinVasily]

Dr. Bacon, ага, только ошибка возникает, потому что при сохранении нового пользователя сравнивается поле username, а не phone_number. Я могу сколько угодно раз удалять миграции и саму бд, но больше одного пользователя не сохраню, потому что в поле username хранится пустая строка.

[PenguinVasily]

Dr. Bacon, поэтому я и задаю вопрос сюда... По какой-то причине значение есть, я могу его получить/вывести, почему так - я не могу объяснить. Если есть какие-то варианты, то я готов их выслушать. Изначально, я думал, что неправильно описал модель, но таких ответов не было.

[PenguinVasily]

Dr. Bacon, тогда лучше использовать почту? Ну и если посмотреть, то сейчас везде используется привязка к номеру телефона. В том же телеграмме вход исключительно по номеру телефона и присылаемому коду.

Answer 1

[AlexandrBirukov]

Скриншот фрагмента кода удалён модератором.
username = None
и заново все миграции

Comments

[PenguinVasily]

Спасибо. Из этого решения вытекает вопрос: зачем тогда нужно поле "USERNAME_FIELD"?

[PenguinVasily]

Dr. Bacon, прочитал, не понял, задаю вопрос сюда :)))

[PenguinVasily]

Dr. Bacon, читал.

[AlexandrBirukov]

PenguinVasily,

Из этого решения вытекает вопрос: зачем тогда нужно поле "USERNAME_FIELD"?

чтобы не париться с менеджером для модели

[AlexandrBirukov]

Dr. Bacon,

зачем постить код картинкой, это какое-это извращение?

у тебя с картинками проблемы? медленный интернет?

[AlexandrBirukov]

Dr. Bacon,

правила сайта

тогда извиняюсь, я не внимательно видимо их прочитал

[AlexandrBirukov]

Dr. Bacon, по какому факту?! где оно существует? это описание таблицы в бд - там это поле не создается

[AlexandrBirukov]

Dr. Bacon,
извиняюсь за картинку

[AlexandrBirukov]

Dr. Bacon,

в коде джанго

при каких кейсах могут прилететь побочки?

[AlexandrBirukov]

Dr. Bacon,

class UserManager(DjangoBaseUserManager):
    """Define a model manager for User model with no username field."""

    use_in_migrations = True

    def _create_user(self, email, password, **extra_fields):
        """Create and save a User with the given email and password."""
        if not email:
            raise ValueError('The given email must be set')
        email = self.normalize_email(email)
        user = self.model(email=email, **extra_fields)
        user.set_password(password)
        user.save(using=self._db)
        return user

    def create_user(self, email, password=None, **extra_fields):
        """Create and save a regular User with the given email and password."""
        extra_fields.setdefault('is_staff', False)
        extra_fields.setdefault('is_superuser', False)
        return self._create_user(email, password, **extra_fields)

    def create_superuser(self, email, password=None, **extra_fields):
        """Create and save a SuperUser with the given email and password."""
        extra_fields.setdefault('is_staff', True)
        extra_fields.setdefault('is_superuser', True)

        if extra_fields.get('is_staff') is not True:
            raise ValueError('Superuser must have is_staff=True.')
        if extra_fields.get('is_superuser') is not True:
            raise ValueError('Superuser must have is_superuser=True.')

        return self._create_user(email, password, **extra_fields)

я обычно менеджер переписываю, больше нигде не видел, может еще в админке

[AlexandrBirukov]

PenguinVasily, и так на всякий, надо еще в settings типа такого написать AUTH_USER_MODEL = 'accounts.User'

[PenguinVasily]

AlexandrBirukov, да, я изначально так сделал. Я же правильно понимаю: теперь, чтобы создать супер-пользователя необходимо создать кастомного менеджера и там переопределить этот метод?

[AlexandrBirukov]

PenguinVasily, ну смотря что надо, к примеру если поле email будет, то можно в менеджере сразу его к нижнему регистру приводить, а то очень часто с мобил вставляют свой email с заглавной первой буквой и это будет уже другой пользователь

[Модератор]

Замените скриншот фрагмента кода текстом.