aboutsummaryrefslogtreecommitdiffstats
path: root/djangorestframework/mixins.py
diff options
context:
space:
mode:
authorGitHub Merge Button2011-07-08 14:59:57 -0700
committerGitHub Merge Button2011-07-08 14:59:57 -0700
commit3a4f39497be899218ad8d3638bbee38f455782c5 (patch)
treeebbe6a68174746a1e15727c5ef92d0e685b068f8 /djangorestframework/mixins.py
parent3b2e70dd3a3bbac0fb5318329483b4a4eb20410a (diff)
parent91b9d0b2a3fa55ff163f64bc689a59ca01cff8bb (diff)
downloaddjango-rest-framework-3a4f39497be899218ad8d3638bbee38f455782c5.tar.bz2
Merge 91b9d0b2a3fa55ff163f64bc689a59ca01cff8bb into 3b2e70dd3a3bbac0fb5318329483b4a4eb20410a
Diffstat (limited to 'djangorestframework/mixins.py')
-rw-r--r--djangorestframework/mixins.py42
1 files changed, 34 insertions, 8 deletions
diff --git a/djangorestframework/mixins.py b/djangorestframework/mixins.py
index b1ba0596..b1c0d815 100644
--- a/djangorestframework/mixins.py
+++ b/djangorestframework/mixins.py
@@ -5,7 +5,7 @@ classes that can be added to a `View`.
from django.contrib.auth.models import AnonymousUser
from django.db.models.query import QuerySet
-from django.db.models.fields.related import RelatedField
+from django.db.models.fields.related import ForeignKey
from django.http import HttpResponse
from django.http.multipartparser import LimitBytes
@@ -508,21 +508,47 @@ class CreateModelMixin(object):
"""
Behavior to create a `model` instance on POST requests
"""
- def post(self, request, *args, **kwargs):
+ def post(self, request, *args, **kwargs):
model = self.resource.model
- # translated 'related_field' kwargs into 'related_field_id'
- for related_name in [field.name for field in model._meta.fields if isinstance(field, RelatedField)]:
- if kwargs.has_key(related_name):
- kwargs[related_name + '_id'] = kwargs[related_name]
- del kwargs[related_name]
+ # Copy the dict to keep self.CONTENT intact
+ content = dict(self.CONTENT)
+ m2m_data = {}
+
+ for field in model._meta.fields:
+ if isinstance(field, ForeignKey) and kwargs.has_key(field.name):
+ # translate 'related_field' kwargs into 'related_field_id'
+ kwargs[field.name + '_id'] = kwargs[field.name]
+ del kwargs[field.name]
+
+ for field in model._meta.many_to_many:
+ if content.has_key(field.name):
+ m2m_data[field.name] = (
+ field.m2m_reverse_field_name(), content[field.name]
+ )
+ del content[field.name]
+
+ all_kw_args = dict(content.items() + kwargs.items())
- all_kw_args = dict(self.CONTENT.items() + kwargs.items())
if args:
instance = model(pk=args[-1], **all_kw_args)
else:
instance = model(**all_kw_args)
instance.save()
+
+ for fieldname in m2m_data:
+ manager = getattr(instance, fieldname)
+
+ if hasattr(manager, 'add'):
+ manager.add(*m2m_data[fieldname][1])
+ else:
+ data = {}
+ data[manager.source_field_name] = instance
+
+ for related_item in m2m_data[fieldname][1]:
+ data[m2m_data[fieldname][0]] = related_item
+ manager.through(**data).save()
+
headers = {}
if hasattr(instance, 'get_absolute_url'):
headers['Location'] = self.resource(self).url(instance)