Changeset 4357

Timestamp:

01/18/07 23:59:28 (2 years ago)

Author:

adrian

Message:

newforms-admin: Added fieldsets(), fieldsets_add() and fieldsets_change() methods to ModelAdmin?. These hooks aren't called from anywhere yet.

Files:

• django/branches/newforms-admin/django/contrib/admin/options.py (modified) (2 diffs)

django/branches/newforms-admin/django/contrib/admin/options.py

@@ -68 +68 @@
-        return len(self.fields)
-
+# New implementation of Fieldset
+class Fieldset(object):
+    def __init__(self, name, field_list, classes=None, description=None):
+        self.name, self.field_list = name, field_list
+        self.classes = classes or ()
+        self.description = description
+
-class ModelAdmin(object):
-    "Encapsulates all admin options and functionality for a given model."
@@ -126 +133 @@
-            new_fieldset_list.append(AdminFieldSet(name, classes, opts.get_field, options['fields'], description))
-        return new_fieldset_list
+
+    def fieldsets(self, request):
+        """
+        Generator that yields Fieldset objects for use on add and change admin
+        form pages.
+
+        This default implementation looks at self.fields, but subclasses can
+        override this implementation and do something special based on the
+        given HttpRequest object.
+        """
+        if self.fields is None:
+            default_fields = [f.name for f in self.opts.fields + self.opts.many_to_many if f.editable and not isinstance(f, models.AutoField)]
+            yield Fieldset(fields=default_fields)
+        else:
+            for name, options in self.fields:
+                yield Fieldset(name, options['fields'], classes=options.get('classes'), description=options.get('description'))
+
+    def fieldsets_add(self, request):
+        "Hook for specifying Fieldsets for the add form."
+        for fs in self.fieldsets(request):
+            yield fs
+
+    def fieldsets_change(self, request, object_id):
+        "Hook for specifying Fieldsets for the change form."
+        for fs in self.fieldsets(request):
+            yield fs
-
-    def has_add_permission(self, request):