Ticket #5563: boolean_no_null_validation.patch

django/core/management/validation.py

@@ -51 +51 @@
                     from PIL import Image
                 except ImportError:
                     e.add(opts, '"%s": To use ImageFields, you need to install the Python Imaging Library. Get it at http://www.pythonware.com/products/pil/ .' % f.name)
+            if isinstance(f, models.BooleanField) and getattr(f, 'null', False):
+                e.add(opts, '"%s": BooleanFields cannot allow null values. Use NullBooleanField instead.' % f.name)
             if f.choices:
                 if isinstance(f.choices, basestring) or not is_iterable(f.choices):
                     e.add(opts, '"%s": "choices" should be iterable (e.g., a tuple or list).' % f.name)

tests/modeltests/invalid_models/models.py

@@ -14 +14 @@
     choices2 = models.CharField(max_length=10, choices=[(1,2,3),(1,2,3)])
     index = models.CharField(max_length=10, db_index='bad')
     field_ = models.CharField(max_length=10)
+    nullbool = models.BooleanField(null=True)
 
 class Target(models.Model):
     tgt_safe = models.CharField(max_length=10)
@@ -190 +191 @@
 invalid_models.fielderrors: "choices2": "choices" should be a sequence of two-tuples.
 invalid_models.fielderrors: "index": "db_index" should be either None, True or False.
 invalid_models.fielderrors: "field_": Field names cannot end with underscores, because this would lead to ambiguous queryset filters.
+invalid_models.fielderrors: "nullbool": BooleanFields cannot allow null values. Use NullBooleanField instead.
 invalid_models.clash1: Accessor for field 'foreign' clashes with field 'Target.clash1_set'. Add a related_name argument to the definition for 'foreign'.
 invalid_models.clash1: Accessor for field 'foreign' clashes with related m2m field 'Target.clash1_set'. Add a related_name argument to the definition for 'foreign'.
 invalid_models.clash1: Reverse query name for field 'foreign' clashes with field 'Target.clash1'. Add a related_name argument to the definition for 'foreign'.

tests/regressiontests/serializers_regress/models.py

@@ -11 +11 @@
 from django.contrib.localflavor.us.models import USStateField, PhoneNumberField
 
 # The following classes are for testing basic data
-# marshalling, including NULL values.
+# marshalling, including NULL values, where allowed.
 
 class BooleanData(models.Model):
-    data = models.BooleanField(null=True)
+    data = models.BooleanField()
 
 class CharData(models.Model):
     data = models.CharField(max_length=30, null=True)