Django:用户外键 -> 表单未验证,因为字段是必需的



我目前正在创建一个包含两部分的注册页面

  • 其中一部分是关于用户名和密码的
  • 第二部分是关于选择自己的PC配置

定义完所有内容后,用户可以注册进入主页。

因此,我得到了一个名为">"的模型;配置PC_带有一堆指向处理器/图形卡等不同数据库模型的外键:

class PC_Configuration(models.Model):
user = models.ForeignKey(User, related_name='user_id', on_delete=models.DO_NOTHING)
processor = models.ForeignKey(Processors, related_name='processor_id', on_delete=models.DO_NOTHING)
graphicscard = models.ForeignKey(Graphicscard, related_name='graphicscard_id', on_delete=models.DO_NOTHING)
os = models.ForeignKey(OS, related_name='os_id', on_delete=models.DO_NOTHING)
ram = models.ForeignKey(RAM, related_name='ram_id', on_delete=models.DO_NOTHING)
harddrive = models.ForeignKey(Harddrive, related_name='harddrive_id', on_delete=models.DO_NOTHING)

此外,用户有一个ForeignKey,用于将配置连接到相应的User-ID。

views.py内部,我一直在为所有下拉字段创建一个下拉表单,用户可以自己选择:

class DropdownForm(forms.ModelForm):
class Meta:
model = models.PC_Configuration
exclude = []
def __init__(self, *args, **kwargs):
super(DropdownForm, self).__init__(*args, **kwargs)
self.fields['processors'].queryset = DropdownForm.objects.all()
self.fields['processors'].label_from_instance = lambda obj: "%s" % obj.name
self.fields['graphicscard'].queryset = DropdownForm.objects.all()
self.fields['graphicscard'].label_from_instance = lambda obj: "%s" % obj.name
self.fields['os'].queryset = DropdownForm.objects.all()
self.fields['os'].label_from_instance = lambda obj: "%s" % obj.name
self.fields['ram'].queryset = DropdownForm.objects.all()
self.fields['ram'].label_from_instance = lambda obj: "%s" % obj.name
self.fields['harddrive'].queryset = DropdownForm.objects.all()
self.fields['harddrive'].label_from_instance = lambda obj: "%s" % obj.name

但关于用户ID应自动分配给配置的事实,这里没有相应的字段。

它在register_view(请求(中定义-方法:

def register_view(request):
form = DropdownForm()
if request.method == "POST":
form = DropdownForm(request.POST)
username = request.POST.get('username')
password = request.POST.get('password')
myuser = User.objects.create_user(username, None, password)
myuser.save()
auth.login(request, myuser)
#form.user = request.user
print(form.errors)
if form.is_valid():
instance = form.save(commit=False)
instance.user = request.user
instance.save()
messages.success(request, "Account has been created successfully")
return redirect(reverse('gamesearch_view'))
else:
print('Failed')
form = DropdownForm()
render(request, 'register.html', dict(form=form))
return render(request, 'register.html', dict(form=form))

我想,在这里,我们遇到了问题。在测试注册时,测试帐户保持创建和登录成功。但问题是,由于表单未进行验证,因此没有创建PC配置。

print(form.errors)

我一直在想确切的原因,上面写着

<ul class="errorlist"><li>user<ul class="errorlist"><li>This field is required.</li></ul></li></ul>

因此,似乎有必要定义";用户"-字段,如果表单随后正在实例中验证和定义用户。

这就是为什么我尝试这样做:

form.user = request.user

但它仍然不起作用,我也不知道到底是什么问题,因为";用户";不应该是表单验证的一部分。

你能帮我吗?

提前感谢!

使用这样的东西会更简单。。。

  • 您的related_name有些虚假;它们应该是与";"观点";其他型号的。(此外,在Django中,您永远不需要手动将_id添加到字段中。(如果删除related_name,它们将隐式地为pc_configuration_set
  • on_delete=DO_NOTHING可能不是一个好主意。PROTECT是一个很好的默认值
  • 将用户名和密码作为表单中的字段处理更容易
  • 您缺少exclude = ["user"],所以如果您的模板没有为user呈现字段,那么它当然会丢失。但是,您也不希望表单的POSTer提交任何旧的用户id
  • 使用FormView可以删除管理表单所需的大部分样板文件
  • 我们使用transaction.atomic()来确保如果保存PC配置失败,用户不会最终保存到数据库中
  • 我们将创建的用户分配给form.instance,这是新的但尚未保存的PC配置

(当然,想象一下这些都在单独的文件中。(

from django import forms
from django.db import models, transaction
from django.views.generic import FormView

class PC_Configuration(models.Model):
user = models.ForeignKey(User, on_delete=models.PROTECT)
processor = models.ForeignKey(Processors, on_delete=models.PROTECT)
graphicscard = models.ForeignKey(Graphicscard, on_delete=models.PROTECT)
os = models.ForeignKey(OS, on_delete=models.PROTECT)
ram = models.ForeignKey(RAM, on_delete=models.PROTECT)
harddrive = models.ForeignKey(Harddrive, on_delete=models.PROTECT)

class RegisterAndConfigurePCForm(forms.ModelForm):
username = forms.CharField(required=True)
password = forms.CharField(required=True, widget=forms.PasswordInput())
class Meta:
model = PC_Configuration
exclude = ["user"]  # we'll assign this by hand

class RegisterAndConfigureView(FormView):
form_class = RegisterAndConfigurePCForm
template_name = "register.html"
def form_valid(self, form):
with transaction.atomic():
user = User.objects.create_user(form.cleaned_data["username"], None, form.cleaned_data["password"])
form.instance.user = user  # assign user to the to-be-created PC configuration
form.save()
return redirect(reverse("gamesearch_view"))

相关内容

最新更新