Ошибка целостности при сохранении запросов из представления - PullRequest
Новая
       5

Ошибка целостности при сохранении запросов из представления

0 голосов
/ 27 февраля 2019

Я получаю сообщение об ошибке целостности.Данные поступают в базу данных.Затем после того, как это дает ошибку.Почему это происходит Может кто-нибудь сказать мне, как это сделать

IntegrityError в / trustyrequest / update / 4 /

(1062, «Дублирующая запись» для ключа «электронная почта» ») 

class TrustyRequestUpdate(generic.UpdateView):
    template_name = "trustyrequest/update.html"
    model = TrustyRequest
    form_class = TrustyRequestForm
    success_url = reverse_lazy('trustyrequest_list')


    def form_valid(self, form):

        if form.cleaned_data['status'] == 'Accepted':
            print('PK: ', form.instance.id)
            random_password = User.objects.make_random_password() # generate password here 
            user = User(
                first_name=form.cleaned_data['first_name'], 
                last_name=form.cleaned_data['last_name'], 
                email=form.cleaned_data['email'], 
                phone_number=form.cleaned_data['phone_number'], 
            )
            user.set_password(random_password)
            user.save()

        elif form.cleaned_data['status'] == 'Pending':
            print('Pending')
            print('PK: ', form.instance.id)

        elif form.cleaned_data['status'] == 'Rejected':
            print('Rejected')
            print('PK: ', form.instance.id)

        return super().form_valid(form)

Есть хороший способ сделать код выше.Я использую AbstractBaseUser для пользовательской модели

Models.py 

from __future__ import unicode_literals

from django.db import models
from django.core.mail import send_mail
from django.contrib.auth.models import PermissionsMixin
from django.contrib.auth.base_user import AbstractBaseUser
from django.utils.translation import ugettext_lazy as _

from .managers import UserManager

from apps.roles.models import Role


class User(AbstractBaseUser, PermissionsMixin):
    email = models.EmailField(_('email address'), unique=True)
    first_name = models.CharField(_('first name'), max_length=30, blank=True)
    last_name = models.CharField(_('last name'), max_length=30, blank=True)
    date_joined = models.DateTimeField(_('date joined'), auto_now_add=True)
    is_active = models.BooleanField(_('active'), default=True)
    avatar = models.ImageField(upload_to='avatars/', null=True, blank=True)
    is_staff = models.BooleanField(_('staff'), default=True)

    role = models.ForeignKey(Role, on_delete=models.CASCADE, blank=True, null=True)
    phone_number = models.CharField(max_length=30, blank=True, null=True)
    address = models.CharField(max_length=100, blank=True, null=True)

    objects = UserManager()

    USERNAME_FIELD = 'email'
    REQUIRED_FIELDS = []

    class Meta:
        verbose_name = _('user')
        verbose_name_plural = _('users')

    def get_full_name(self):
        '''
        Returns the first_name plus the last_name, with a space in between.
        '''
        full_name = '%s %s' % (self.first_name, self.last_name)
        return full_name.strip()

    def get_short_name(self):
        '''
        Returns the short name for the user.
        '''
        return self.first_name

    def email_user(self, subject, message, from_email=None, **kwargs):
        '''
        Sends an email to this User.
        '''
        send_mail(subject, message, from_email, [self.email], **kwargs)

    class Meta:
        db_table = "user"

Это мой Managers.py 

from django.contrib.auth.base_user import BaseUserManager

class UserManager(BaseUserManager):
    use_in_migrations = True

    def _create_user(self, email, password, **extra_fields):
        """
        Creates and saves 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):
        extra_fields.setdefault('is_superuser', False)
        return self._create_user(email, password, **extra_fields)

    def create_superuser(self, email, password, **extra_fields):
        extra_fields.setdefault('is_superuser', 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)

Form.py 

from django import forms
from .models import TrustyRequest

class TrustyRequestForm(forms.ModelForm):
    class Meta:
        model = TrustyRequest
        fields = [
            'name', 
            'address', 
            'first_name', 
            'last_name', 
            'email', 
            'phone_number',
            'status',
        ]

1 Ответ

0 голосов
/ 27 февраля 2019

Вы используете вид обновления для создания нового пользователя.При использовании вида обновления ожидается, что у вас уже есть пользователь в базе данных (у вас pk 4 в URL, нет?).

Вы должны использовать get () в представлении обновлений, чтобы получить пользователя, так как используемый вами синтаксис создает нового пользователя при каждом обновлении.Ошибка указывает на то, что вы пытаетесь продублировать уникальное поле - т.е. у нового пользователя тот же адрес электронной почты, что и у существующего пользователя

В form_valid () измените создание пользователя на: 

user = User.objects.get(pk=form.instance.id)
user.first_name=form.cleaned_data['first_name']
user.last_name = form.cleaned_data['last_name]
...
..
user.save()
...