DjangoBot: #django-es

alguien?

internéstor esta turulo en pycamp

ouch, wrong channel.. sorry

cramm: cómo va che

cramm: si tenés un rato en algún momento, querés mirar el parchecito en http://code.djangoproject.com/ticket/12627 ?

hola nicoechaniz

disculpame ayer no nos pudimos encontrar, estuve saliendo de casa y encima cortaron la electricidad

nicoechaniz_away: le vamos apedir su opinion a algun core developer, solo veo dos cositas que a lo mejor son bobadas mias

nicoechaniz_away: 1. a lo mejor la nuva logica en is_valid() haya que ejecutarla solo cuand is_bound es True. Creo recordar que si el form está unbound no tenemso que tener side effects en la colección de errores

nicoechaniz_away:y 2. no se porque esta es la primera vez quehace falta usar ugettext_lazy() en ese modulo. Me da miedo de que esten intentando mantener eso fuera del mismo

sí. yo pensé lo mismo

cramm: pero le dimos unas cuantas vueltas y no encontramos un lugar mejor

cramm: pero bueno, por eso la idea era que lo vea alguien que esté más adentro a ver si hay un mejor lugar donde hacer esto, o mejor forma de resolverlo

cramm: el otro lugar interesante era en clean() de BaseForm, pero ese método es un método que siempre se sobrecarga

nicoechaniz_away: Alex Gaynor dice en #django-dev: cramm: why not just have admin validation disallow ModelAdmin with no fields?

es admin-specific eso?

cramm: en realidad el ticket reporta 2 cosas

una es específica del admin (la de readonly) y la otra no (la de editable=False)

este parchecito soluciona los dos problemas

ah si, cierto que charlamos con alguien el Sabado aca en Cba de que a lo mejor habria que desdoblarlo

sí, acá pensamos lo mismo pero viéndolo nos encontramos con que el problema coincidía en que está vacío .fields

y por eso ese parchecito resuelve los 2 casoa

casos

genial

cramm: lo que no llegamos es a escribir tests para esto y yo ahora vuelvo a BsAs y al trabajo, así que eso quedó pendiente por ahora...

nicoechaniz_away: Alex_Gaynor cramm: perhaps that should be form level validation then? at the end of forms.Form.__init__ have assert self.fields, "There must be fields on a form

sí, de hecho tiene que ser form level

nicoechaniz_away: buenisimo, al menos ya está encarado

cramm: bueno, cuando vuelva si nadie lo siguió y me hago un rato miro de entrar por forms.Form.__init__

cramm: si no, como bien decís, al menos está encarado :)

ok, yo hoy estoy un poco complciado, pero voy a tratard e dar ua mano aunque sea escribiendo tests

cramm: joia

cramm: bueno... voy cerrando acá... a acomodar mis cosas, que ya está terminando el PyCamp.

nicoechaniz_away: Genial. Se ve que la pasaron bien, como es costubre :). Buen viaje!

cramm: muchas gracias. Sí, la pasamos muy bien. hablamos pronto

Estoy haciendo un form con unos campos ModelChoiceField pero quiero limitar el queryset de este campo alguien lo ha hecho?