Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Browse files

[1.0.X] Fixed #10792 -- Ensured that ModelChoiceFields don't provide …

…an empty option when the underlying field has blank=False and there is a default value available. Thanks to carljm for the report and patch.

Merge of r10729 from trunk.

git-svn-id: http://code.djangoproject.com/svn/django/branches/releases/1.0.X@10733 bcc190cf-cafb-0310-a4f2-bffc1f526a37
  • Loading branch information...
commit bc07a498fe789ed5c4057d56bac6ba50d7296ed1 1 parent 0d930ee
@freakboy3742 freakboy3742 authored
View
5 django/forms/models.py
@@ -869,7 +869,10 @@ class ModelChoiceField(ChoiceField):
def __init__(self, queryset, empty_label=u"---------", cache_choices=False,
required=True, widget=None, label=None, initial=None,
help_text=None, to_field_name=None, *args, **kwargs):
- self.empty_label = empty_label
+ if required and (initial is not None):
+ self.empty_label = None
+ else:
+ self.empty_label = empty_label
self.cache_choices = cache_choices
# Call Field instead of ChoiceField __init__() because we don't need
View
4 docs/ref/forms/fields.txt
@@ -752,6 +752,10 @@ example::
# No empty label
field2 = forms.ModelChoiceField(queryset=..., empty_label=None)
+ Note that if a ``ModelChoiceField`` is required and has a default
+ initial value, no empty choice is created (regardless of the value
+ of ``empty_label``).
+
``ModelMultipleChoiceField``
~~~~~~~~~~~~~~~~~~~~~~~~~~~~
View
26 tests/regressiontests/forms/models.py
@@ -18,6 +18,17 @@ class ChoiceModel(models.Model):
"""For ModelChoiceField and ModelMultipleChoiceField tests."""
name = models.CharField(max_length=10)
+class ChoiceOptionModel(models.Model):
+ """Destination for ChoiceFieldModel's ForeignKey.
+ Can't reuse ChoiceModel because error_message tests require that it have no instances."""
+ name = models.CharField(max_length=10)
+
+class ChoiceFieldModel(models.Model):
+ """Model with ForeignKey to another model, for testing ModelForm
+ generation with ModelChoiceField."""
+ choice = models.ForeignKey(ChoiceOptionModel, blank=False,
+ default=lambda: ChoiceOptionModel.objects.all()[0])
+
class FileModel(models.Model):
file = models.FileField(upload_to='/')
@@ -73,4 +84,19 @@ class FileForm(django_forms.Form):
datetime.date(1969, 4, 4)
>>> instance_form.initial['value']
12
+
+In a ModelForm with a ModelChoiceField, if the model's ForeignKey has blank=False and a default,
+no empty option is created (regression test for #10792).
+
+First we need at least one instance of ChoiceOptionModel:
+
+>>> ChoiceOptionModel.objects.create(name='default')
+<ChoiceOptionModel: ChoiceOptionModel object>
+
+>>> class ChoiceFieldForm(ModelForm):
+... class Meta:
+... model = ChoiceFieldModel
+>>> list(ChoiceFieldForm().fields['choice'].choices)
+[(1, u'ChoiceOptionModel object')]
+
"""}
Please sign in to comment.
Something went wrong with that request. Please try again.