Ticket #3258: save_instance.2.patch

django/newforms/models.py

@@ -26 +26 @@
     """
     Saves bound Form ``form``'s clean_data into model instance ``instance``.
 
-    Assumes ``form`` has a field for every non-AutoField database field in
-    ``instance``. If commit=True, then the changes to ``instance`` will be
-    saved to the database. Returns ``instance``.
+    If commit=True, then the changes to ``instance`` will be saved to the
+    database. Returns ``instance``.
     """
     from django.db import models
     opts = instance.__class__._meta
@@ -38 +37 @@
     for f in opts.fields:
         if not f.editable or isinstance(f, models.AutoField):
             continue
-        setattr(instance, f.name, clean_data[f.name])
+        if clean_data.has_key(f.name):
+            setattr(instance, f.name, clean_data[f.name])
     if commit:
         instance.save()
         for f in opts.many_to_many:
-            setattr(instance, f.attname, clean_data[f.name])
+            if clean_data.has_key(f.name):
+                setattr(instance, f.attname, clean_data[f.name])
     # GOTCHA: If many-to-many data is given and commit=False, the many-to-many
     # data will be lost. This happens because a many-to-many options cannot be
     # set on an object until after it's saved. Maybe we should raise an