2009-10-30 70 views
0

排除我有一个模型菜单:unique_together验证失败,如果场中的一个从形式

class Menu(models.Model): 
    loja = models.ForeignKey(Loja, related_name='menus') 
    nome = models.CharField(max_length=30) 
    ordenacao = models.IntegerField(blank=True, null=True) 

    class Meta: 
     ordering = ('ordenacao',) 
     #prevent equally named menus within a store(loja) 
     unique_together = ('loja', 'nome') 

    def __unicode__(self): 
     return self.nome 

菜单形式:

class MenuForm(ModelForm): 
    class Meta: 
     model = Menu 
     exclude =('loja', 'ordenacao',) 

添加菜单视图:

def addmenu(request, s_loja): 
    loja = get_object_or_404(Loja, slug=s_loja) 
    if not loja.activo: 
     return render_to_response('painelcontrolo/loja_detail.html', { 
      'notificacoes': ['Loja está definida como inactivo.', \ 
          'Alterações não permitidas']}) 
    if request.method == 'POST': 
     menu = Menu(loja=loja) 
     form = MenuForm(request.POST, instance=menu) 
     if form.is_valid(): 
      menu_existe = Menu.objects.\ 
          filter(nome=form.cleaned_data['nome']).\ 
          filter(loja=loja) 
      if menu_existe: 
       return render_to_response(\ 
        'painelcontrolo/loja_detail.html', { 
         'notificacoes': ['Já existe um menu com esse nome'] 
         }) 
      form.save() 
      return render_to_response(\ 
        'painelcontrolo/loja_detail.html', { 
         'notificacoes': ['Menu adicionado'] 
         }) 
    else: 
     form = MenuForm() 
    return render_to_response('form_as_p.html', { 
     'form': form 
    }) 

错误:(不再在添加菜单视图中添加验证时发生)

Request Method:  POST 
Request URL: http://127.0.0.1:8000/painel/bispos/addmenu/ 
Exception Type:  IntegrityError 
Exception Value: columns loja_id, nome are not unique 

问题: 表单是有效的,但是模型不是如果nome + loja_id已经存在于数据库中。我是否需要在其他地方添加此验证?哪里?

编辑: 我写了对视图的验证,它将通知传递给模板,这很好,但并不完美。我想重新显示带有用户输入的表单,以便在不丢失该信息的情况下提供解决问题的机会。有没有办法做到这一点?

回答

1

好吧,这是我能拿出最好的。它给出了窗体上的错误,似乎工作正常。

@login_required 
def addmenu(request, s_loja): 
    loja = get_object_or_404(Loja, slug=s_loja) 
    if not loja.activo: 
     return render_to_response('painelcontrolo/loja_detail.html', { 
      'notificacoes': ['Loja está definida como inactivo.', \ 
          'Alterações não permitidas']}) 
    if request.method == 'POST': 
     menu = Menu(loja=loja) 
     form = MenuForm(request.POST, instance=menu) 
     if form.is_valid(): 
      menu_existe = Menu.objects.\ 
          filter(nome=form.cleaned_data['nome']).\ 
          filter(loja=loja) 
      if not menu_existe: 
       form.save() 
       return render_to_response('painelcontrolo/loja_detail.html', { 
         'notificacoes': ['Menu adicionado'] 
         }) 
      else: 
       form._errors['nome'] = ErrorList(\ 
        ['Um menu com esse nome já existe']) 
    else: 
     form = MenuForm() 
    return render_to_response('form_as_p.html', { 
     'form': form, 
    }) 
0

我不确定表单如何知道数据库中存在哪些唯一值,它只是验证填充模型所需的字段类型,据我所知。

换句话说,“form.is_valid()”不会保存到数据库,这是您可以查看保存是否有效的唯一方法。

你必须自己去理解这一点,我很确定,但“unique_together”已经在做它的工作。

2

我会离开loja的形式,但使其成为一个隐藏的领域。

class MenuForm(ModelForm): 
    loja = models.ModelChoiceField(Loja.objects.all(), widget=forms.HiddenInput) 

    class Meta: 
     model = Menu 
     exclude =('ordenacao',) 

你可能将不得不改变你的看法打电话给你getloja()不管请求是post或get。你从来没有解释如何getloja()决定什么是正确的情况下...

@login_required 
def addmenu(request, s_loja): 
    if request.method == 'POST': 
     form = MenuForm(request.POST) 
     if form.is_valid(): 
      form.save() 
      return HttpResponseRedirect('/painel/profile/') 
    else: 
     loja = getloja(request, s_loja) #simply retrieves the correct loja instance 
     menu = Menu(loja=loja) 
     form = MenuForm(instance=menu) 
    return render_to_response('form_as_p.html', { 
      'form': form,}) 
+0

但是,我的表单无法提交,因为该字段不能为空。这似乎是一个更好的解决方案,但我不知道如何使其工作。我试过用loja = forms.ModelChoiceField(label =“”, queryset = Loja.objects.all(), widget = forms.HiddenInput()) – 2009-10-31 04:20:25

+0

Ricardo,getloja(request,s_loja)在做什么?你可以发布吗? – cethegeek 2009-10-31 04:43:15

+0

哦,我刚刚意识到......'s_loja'是传递给视图的url参数,对吗? getloja可能只是在做一个'Loja.objects.get(...)'(我只是不知道为什么你将请求传递给它)。只需将getloja()移到form = MenuForm()之前即可。我会编辑我的答案... – cethegeek 2009-10-31 04:58:06