Problemas al generar un ValidationError en un formulario de Django

Question

Estoy tratando de validar que una URL enviada no existe en la base de datos.

Las partes pertinentes de la clase Form aspecto como éste:

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 

El problema es que, independientemente de lo que el valor a mi juicio, no puede elevar el ValidationError. Y si hago algo como esto en el método clean_url():

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

cuando me siento un error DoesNotExist, incluso para las direcciones URL que ya existen en la base de datos. ¿Algunas ideas?

Answer 1

canal de Django en IRC me salvó aquí. El problema era que el URLField.clean() hace dos cosas que no esperaba:

1. Si ningún esquema de URL está presente (por ejemplo, http: //) antepone los métodos 'http: //' a la url
2. el método también agrega una barra inclinada.

Los resultados se devuelven y almacenan en clean_data del formulario. Así que estaba revisando cleaned_data['url'] esperando algo así como example.com y obteniendo realmente http://example.com/. Baste decir, el cambio de mi método clean_url() a las siguientes obras:

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

Lo hago de esta manera. Es un poco mas 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

Creo que puede devolver '' y rellenar _errores.

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

o

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

Bueno, yo registran en la causa que encontré esto a través de Google con un problema similar y quería añadir un comentario a Carl Meyers mensaje señalando que el uso de self._errors es totalmente válida según la documentación de Django:

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