newforms form_for_* methods don't automatically validate against unique_together

[Jungle]

method is_valid don't return error for unique fields
I'm using this model

class Account(models.Model):
    provider = models.ForeignKey(Provider, verbose_name = 'Provider')
    percent = models.PositiveSmallIntegerField(u'Percent', null = True, blank = True)
    
    aid = models.PositiveIntegerField(u'Affiliate ID')
    skey = models.CharField(u'Secret key', maxlength = 256, blank = True)
    
    class Meta:
        unique_together = (('provider', 'aid'),)

When I try create instance of Account with exists fields in DB it raise IntegrityError
P.S. sorry for my English

[Chris Beaven]

You're right, they don't automatically validate against unique_together currently.

Whether they should or not is up for debate.

[Malcolm Tredinnick]

This gets fixed once Jacob has finished the model-aware validation stuff we started working on at OSCON. Things like this must be validated in the model, not the form, but for the form_for_* cases, the model's errors should then be fed back to the form. So the problem is real, but we are fixing it in a more holistic fashion than just tweaking form_for_*.

[Brian Rosner]

Marked #5736 as a duplicate. unique and unique_together will be fixed with some model aware validation that Malcolm mentions above.

[nix]

What is the ETA on the "model aware validation" changes? (I am trying to decide how much effort to spend working around this bug)

[ipse <ipse.reg@…>]

Any progress on this?
I'm still getting this exception when UNIQUE is violated.

[David Larlet]

form_for_* are now deprecated and replaced by ModelForm so the bug should be tested with it I guess.

[ipse <ipse.reg@…>]

It is still here with ModelForms (I'm using them, not form_for_*).

[Favo Yang <favo@…>]

also missing field's unique=True support.

[mrts]

See ​http://www.djangosnippets.org/snippets/649/ for a workaround that can be used until model validation gets integrated.

[anonymous]

Replying to mrts:

See ​http://www.djangosnippets.org/snippets/649/ for a workaround that can be used until model validation gets integrated.

doesn't handle i18n

[mrts]

Duplicate of #6845.