Changeset 7930 for django/branches/newforms-admin/django/newforms/models.py

Timestamp:

07/15/08 17:41:17 (6 months ago)

Author:

brosner

Message:

newforms-admin: Fixed #7230 -- Added a save_m2m method to BaseModelFormSet? when commit=False is passed to save. Thanks Books Travis for the original report.

Files:

• django/branches/newforms-admin/django/newforms/models.py (modified) (3 diffs)

django/branches/newforms-admin/django/newforms/models.py

@@ -332 +332 @@
-        as necessary, and returns the list of instances.
-        """
+        if not commit:
+            self.saved_forms = []
+            def save_m2m():
+                for form in self.saved_forms:
+                    form.save_m2m()
+            self.save_m2m = save_m2m
-        return self.save_existing_objects(commit) + self.save_new_objects(commit)
-
@@ -354 +360 @@
-                    self.changed_objects.append((obj, form.changed_data))
-                    saved_instances.append(self.save_existing(form, obj, commit=commit))
+                    if not commit:
+                        self.saved_forms.append(form)
-        return saved_instances
-
@@ -366 +374 @@
-                continue
-            self.new_objects.append(self.save_new(form, commit=commit))
+            if not commit:
+                self.saved_forms.append(form)
-        return self.new_objects
-