[PATCH v2] Don't require optional model fields in forms

Dirk Wallenstein halsmit at t-online.de
Mon Jan 17 20:46:48 EST 2011


The keyword 'blank' concerns only validation and does not change what
will be stored in the database.

Signed-off-by: Dirk Wallenstein <halsmit at t-online.de>
---
Initially I thought the 'content' and 'hash' fields should always be
non-empty, but they are in a pull request.  Now every null=True has a
blank=True.

 apps/patchwork/models.py |   12 ++++++------
 1 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/apps/patchwork/models.py b/apps/patchwork/models.py
index 5c0ca95..dd554a9 100644
--- a/apps/patchwork/models.py
+++ b/apps/patchwork/models.py
@@ -44,8 +44,8 @@ except ImportError:
 
 class Person(models.Model):
     email = models.CharField(max_length=255, unique = True)
-    name = models.CharField(max_length=255, null = True)
-    user = models.ForeignKey(User, null = True)
+    name = models.CharField(max_length=255, null=True, blank=True)
+    user = models.ForeignKey(User, null=True, blank=True)
 
     def __unicode__(self):
         if self.name:
@@ -71,7 +71,7 @@ class Project(models.Model):
 
 class UserProfile(models.Model):
     user = models.ForeignKey(User, unique = True)
-    primary_project = models.ForeignKey(Project, null = True)
+    primary_project = models.ForeignKey(Project, null=True, blank=True)
     maintainer_projects = models.ManyToManyField(Project,
             related_name = 'maintainer_project')
     send_email = models.BooleanField(default = False,
@@ -188,10 +188,10 @@ class Patch(models.Model):
     state = models.ForeignKey(State)
     archived = models.BooleanField(default = False)
     headers = models.TextField(blank = True)
-    content = models.TextField(null = True)
-    pull_url = models.CharField(max_length=255, null = True)
+    content = models.TextField(null=True, blank=True)
+    pull_url = models.CharField(max_length=255, null=True, blank=True)
     commit_ref = models.CharField(max_length=255, null = True, blank = True)
-    hash = HashField(null = True, db_index = True)
+    hash = HashField(db_index=True ,null=True, blank=True)
 
     def __unicode__(self):
         return self.name
-- 
1.7.3.2



More information about the Patchwork mailing list