Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Browse files

newforms: Got form_for_instance() to select initial ManyToManyField v…

…alues properly

git-svn-id: http://code.djangoproject.com/svn/django/trunk@4261 bcc190cf-cafb-0310-a4f2-bffc1f526a37
  • Loading branch information...
commit b9fdf9abb841f1eda11fb4d8094372dbbbeb58cb 1 parent 0919920
@adrianholovaty adrianholovaty authored
View
4 django/db/models/fields/related.py
@@ -726,6 +726,10 @@ def value_from_object(self, obj):
return getattr(obj, self.attname).all()
def formfield(self, initial=None):
+ # If initial is passed in, it's a list of related objects, but the
+ # MultipleChoiceField takes a list of IDs.
+ if initial is not None:
+ initial = [i._get_pk_val() for i in initial]
return forms.MultipleChoiceField(choices=self.get_choices_default(), required=not self.blank, label=capfirst(self.verbose_name), initial=initial)
class ManyToOneRel(object):
View
29 tests/modeltests/model_forms/models.py
@@ -28,7 +28,7 @@ def __str__(self):
class Article(models.Model):
headline = models.CharField(maxlength=50)
- pub_date = models.DateTimeField()
+ pub_date = models.DateField()
writer = models.ForeignKey(Writer)
categories = models.ManyToManyField(Category, blank=True)
@@ -80,6 +80,8 @@ def __str__(self):
>>> Category.objects.all()
[<Category: Entertainment>, <Category: It's a test>]
+If you call create() with save=False, then it will return an object that hasn't
+yet been saved. In this case, it's up to you to save it.
>>> f = CategoryForm({'name': 'Third test', 'url': 'third'})
>>> f.errors
{}
@@ -94,6 +96,7 @@ def __str__(self):
>>> Category.objects.all()
[<Category: Entertainment>, <Category: It's a test>, <Category: Third test>]
+If you call create() with invalid data, you'll get a ValueError.
>>> f = CategoryForm({'name': '', 'url': 'foo'})
>>> f.errors
{'name': [u'This field is required.']}
@@ -102,7 +105,6 @@ def __str__(self):
Traceback (most recent call last):
...
ValueError: The Category could not be created because the data didn't validate.
-
>>> f = CategoryForm({'name': '', 'url': 'foo'})
>>> f.create()
Traceback (most recent call last):
@@ -181,4 +183,27 @@ def __str__(self):
>>> new_art = Article.objects.get(id=1)
>>> new_art.headline
'New headline'
+
+Add some categories and test the many-to-many form output.
+>>> new_art.categories.all()
+[]
+>>> new_art.categories.add(Category.objects.get(name='Entertainment'))
+>>> new_art.categories.all()
+[<Category: Entertainment>]
+>>> TestArticleForm = form_for_instance(new_art)
+>>> f = TestArticleForm(auto_id=False)
+>>> print f.as_ul()
+<li>Headline: <input type="text" name="headline" value="New headline" maxlength="50" /></li>
+<li>Pub date: <input type="text" name="pub_date" value="1988-01-04" /></li>
+<li>Writer: <select name="writer">
+<option value="">---------</option>
+<option value="1" selected="selected">Mike Royko</option>
+<option value="2">Bob Woodward</option>
+</select></li>
+<li>Categories: <select multiple="multiple" name="categories">
+<option value="1" selected="selected">Entertainment</option>
+<option value="2">It&#39;s a test</option>
+<option value="3">Third test</option>
+</select></li>
+
"""}
Please sign in to comment.
Something went wrong with that request. Please try again.