Problèmes soulevés par ValidationError sur un formulaire Django

Question

J'essaie de valider qu'une URL soumise n'existe pas déjà dans la base de données.

Les parties pertinentes du look classe Form comme celui-ci:

from django.contrib.sites.models import Site 
class SignUpForm(forms.Form): 
    # ... Other fields ... 
    url = forms.URLField(label='URL for new site, eg: example.com') 

    def clean_url(self): 
     url = self.cleaned_data['url'] 
     try: 
      a = Site.objects.get(domain=url) 

     except Site.DoesNotExist: 
      return url 

     else: 
      raise forms.ValidationError("That URL is already in the database. Please submit a unique URL.") 

    def clean(self): 
     # Other form cleaning stuff. I don't *think* this is causing the grief 

Le problème est, quelle que soit la valeur que je soumets, je ne peux pas soulever la ValidationError. Et si je fais quelque chose comme ça dans la méthode clean_url():

if Site.objects.get(domain=url): 
    raise forms.ValidationError("That URL is already in the database. Please submit a unique URL.") 

j'obtiens une erreur DoesNotExist, même pour les URL qui existent déjà dans la base de données. Des idées?

Answer 1

canal django IRC m'a sauvé ici. Le problème était que le URLField.clean() fait deux choses je ne m'y attendais:

1. Si aucun schéma d'URL est présent (par exemple, http: //) la méthode précèder « http: // » au url
2. la méthode ajoute également un slash final.

Les résultats sont retournés et stockés dans le formulaire clean_data. Donc, je vérifiais cleaned_data['url'] s'attendant à quelque chose comme example.com et effectivement obtenir http://example.com/. Qu'il suffise de dire, de changer ma méthode clean_url() aux travaux suivants:

def clean_url(self): 
     url = self.cleaned_data['url']   
     bits = urlparse(url) 
     dom = bits[1] 
     try: 
      site=Site.objects.get(domain__iexact=dom) 
     except Site.DoesNotExist: 
      return dom 
     raise forms.ValidationError(u'That domain is already taken. Please choose another') 

Answer 2

Je le fais de cette façon. C'est un peu plus simple.

try: 
    a = Site.objects.get(domain=url) 
    raise forms.ValidationError("That URL is already in the database. Please submit a unique URL.") 
except Site.DoesNotExist: 
    pass 
return url 

Answer 3

Je pense que vous pouvez retourner '' et remplir _errors.

msg = u"That URL is already in the database. Please submit a unique URL." 
self._errors["url"]=ErrorList([msg]) 
return '' 

ou

from django.contrib.sites.models import Site 
class SignUpForm(forms.Form): 
    # ... Other fields ... 

url = forms.URLField(label='URL for new site, eg: example.com') 

def clean_url(self): 
    url = self.cleaned_data['url'] 
    try: 
     a = Site.objects.get(domain=url) 
     raise forms.ValidationError("That URL is already in the database. Please submit a unique URL.") 
    except Site.DoesNotExist: 
     return url 
    return '' 

def clean(self): 
    # Other form cleaning stuff. I don't *think* this is causing the grief 

Answer 4

Eh bien je me suis connecté parce que je trouve cela via Google avec un problème similaire et je voulais ajouter un commentaire à Carl Meyers après noter que l'utilisation self._errors est tout à fait valable selon les Django docs:

http://docs.djangoproject.com/en/1.2/ref/forms/validation/#cleaning-and-validating-fields-that-depend-on-each-other