Ticket #4667: newforms-admin-5519-generic-edit-inline-sprint14sep.patch

a/django/contrib/admin/options.py

@@ -689 +689 @@
-    template = None
-    verbose_name = None
-    verbose_name_plural = None
+    formset = None
-
-    def __init__(self, parent_model, admin_site):
-        self.admin_site = admin_site
@@ -706 +707 @@
-            fields = flatten_fieldsets(self.declared_fieldsets)
-        else:
-            fields = None
-
+
+
-
-    def formset_change(self, request, obj):
-        """Returns an InlineFormSet class for use in admin change views."""
@@ -714 +716 @@
-            fields = flatten_fieldsets(self.declared_fieldsets)
-        else:
-            fields = None
-
+
+
-
-    def fieldsets_add(self, request):
-        if self.declared_fieldsets:

a/django/contrib/contenttypes/generic.py

@@ -260 +260 @@
-        self.multiple = True
-        assert not (self.raw_id_admin and self.filter_interface), \
-            "Generic relations may not use both raw_id_admin and filter_interface"
+
+
+
+from django.newforms.models import InlineFormset, save_instance
+class GenericInlineFormset(InlineFormset):
+
+    @classmethod
+    def add_fk(cls, parent_model, model, fk_name):
+        from django.template.defaultfilters import slugify
+        assert ':' in fk_name, "Supply valid content_type:object_id as fk_name."
+
+        opts = model._meta
+        ct_field_name, id_field_name = fk_name.split(':')
+        ct = opts.get_field( ct_field_name )
+        obj_id = opts.get_field( id_field_name )
+        # HACK: remove the ForeignKey to the parent from every form
+        # This should be done a line above before we pass 'fields' to formset_for_model
+        # an 'omit' argument would be very handy here
+        try:
+            del cls.form_class.base_fields[ct.name]
+            del cls.form_class.base_fields[obj_id.name]
+        except KeyError:
+            pass
+
+        cls.parent_model = parent_model
+        cls.model = model
+        cls.ct_field_name = ct_field_name
+        cls.id_field_name = id_field_name
+        cls.ct = ct
+        cls.obj_id = obj_id
+        cls.rel_name = slugify(fk_name)
+
+    def get_inline_objects(self):
+        # This import is done here to avoid circular import importing this module
+        from django.contrib.contenttypes.models import ContentType
+        if self.instance is None:
+            return []
+        return self.model._default_manager.filter( **{ 
+                self.ct_field_name : ContentType.objects.get_for_model(self.instance),
+                self.id_field_name : self.instance._get_pk_val() 
+            })
+
+    def save_new(self, form, commit=True):
+        # This import is done here to avoid circular import importing this module
+        from django.contrib.contenttypes.models import ContentType
+        kwargs = {
+            self.ct.get_attname(): ContentType.objects.get_for_model(self.instance).id,
+            self.obj_id.get_attname(): self.instance._get_pk_val(),
+        }
+        new_obj = self.model(**kwargs)
+        return save_instance(form, new_obj, commit=commit)

a/django/newforms/models.py

@@ -289 +289 @@
-class InlineFormset(BaseModelFormSet):
-    """A formset for child objects related to a parent."""
-    def __init__(self, instance=None, data=None, files=None):
-        from django.db.models.fields.related import RelatedObject
-        self.instance = instance
-        # is there a better way to get the object descriptor?
-        self.rel_name = RelatedObject(self.fk.rel.to, self.model, self.fk).get_accessor_name()
-        super(InlineFormset, self).__init__(data, files, instances=self.get_inline_objects(), prefix=self.rel_name)
+ 
+    @classmethod
+    def add_fk(cls, parent_model, model, fk_name):
+        from django.db.models import ForeignKey 
+        from django.db.models.fields.related import RelatedObject
+        opts = model._meta
+        # figure out what the ForeignKey from model to parent_model is
+        if fk_name is None:
+            fks_to_parent = [f for f in opts.fields if isinstance(f, ForeignKey) and f.rel.to == parent_model]
+            if len(fks_to_parent) == 1:
+                fk = fks_to_parent[0]
+            elif len(fks_to_parent) == 0:
+                raise Exception("%s has no ForeignKey to %s" % (model, parent_model))
+            else:
+                raise Exception("%s has more than 1 ForeignKey to %s" % (model, parent_model))
+        else:
+            fk = opts.get_field( fk_name )
+        # HACK: remove the ForeignKey to the parent from every form
+        # This should be done a line above before we pass 'fields' to formset_for_model
+        # an 'omit' argument would be very handy here
+        try:
+            del cls.form_class.base_fields[fk.name]
+        except KeyError:
+            pass
+
+        cls.parent_model = parent_model
+        cls.fk_name = fk.name
+        cls.rel_name = RelatedObject(fk.rel.to, model, fk).get_accessor_name()
+        cls.fk = fk
-
-    def get_inline_objects(self):
-        if self.instance is None:
@@ -305 +332 @@
-        new_obj = self.model(**kwargs)
-        return save_instance(form, new_obj, commit=commit)
-
-
+
+
-    """
-    Returns an ``InlineFormset`` for the given kwargs.
-    
-    You must provide ``fk_name`` if ``model`` has more than one ``ForeignKey``
-    to ``parent_model``.
-    """
-    from django.db.models import ForeignKey
-    opts = model._meta
-    # figure out what the ForeignKey from model to parent_model is
-    if fk_name is None:
-        fks_to_parent = [f for f in opts.fields if isinstance(f, ForeignKey) and f.rel.to == parent_model]
-        if len(fks_to_parent) == 1:
-            fk = fks_to_parent[0]
-        elif len(fks_to_parent) == 0:
-            raise Exception("%s has no ForeignKey to %s" % (model, parent_model))
-        else:
-            raise Exception("%s has more than 1 ForeignKey to %s" % (model, parent_model))
-    # let the formset handle object deletion by default
-InlineFormset, fields=fields, 
- 
- 
+formset or InlineFormset, fields=fields,
+
+
-                                deletable=deletable)
-
-
-
-
-
-
-
-
-
-
+
-    return FormSet