[PATCH v2] Don't require optional model fields in forms
Dirk Wallenstein
halsmit at t-online.de
Fri Feb 11 20:41:23 EST 2011
On Fri, Feb 11, 2011 at 09:13:26AM +0800, Jeremy Kerr wrote:
> Hi Dirk,
>
> > The keyword 'blank' concerns only validation and does not change what
> > will be stored in the database.
>
> mostly applied:
>
> > - hash = HashField(null = True, db_index = True)
> > + hash = HashField(db_index=True ,null=True, blank=True)
>
> Trying to sneak an index in, eh? :)
Yeah, except this pesky change-history in the django admin interface ;)
> Although this is probably a good idea, do you have a particular requirement
> (or data) for this change?
No, just when saving a pull request through the admin interface I would
have to fake a hash.
If it's ok, and you apply, maybe fix the comma-space order in the new
line.
--
Greetings,
Dirk
More information about the Patchwork
mailing list