Changeset 4416

Timestamp:

01/24/07 11:18:21 (2 years ago)

Author:

adrian

Message:

newforms-admin: Implemented part of raw_id_admin and related-object popup window links for ForeignKey? and ManyToManyFields? in the admin

Files:

• django/branches/newforms-admin/django/contrib/admin/options.py (modified) (1 diff)
• django/branches/newforms-admin/django/contrib/admin/templates/widget/foreign.html (modified) (1 diff)
• django/branches/newforms-admin/django/contrib/admin/views/main.py (modified) (2 diffs)
• django/branches/newforms-admin/django/contrib/admin/widgets.py (modified) (1 diff)

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

@@ -208 +208 @@
-            widget = widgets.FilteredSelectMultiple(db_field.verbose_name, db_field.rel.filter_interface-1)
-            return db_field.formfield(widget=widget, **kwargs)
+
-        # For DateTimeFields, use a special field and widget.
-        if isinstance(db_field, models.DateTimeField):
-            return forms.SplitDateTimeField(required=not db_field.blank,
-                widget=widgets.AdminSplitDateTime(), label=capfirst(db_field.verbose_name), **kwargs)
+
-        # For DateFields, add a custom CSS class.
-        if isinstance(db_field, models.DateField):
-            return db_field.formfield(widget=forms.TextInput(attrs={'class': 'vDateField', 'size': '10'}))
+
-        # For TimeFields, add a custom CSS class.
-        if isinstance(db_field, models.TimeField):
-            return db_field.formfield(widget=forms.TextInput(attrs={'class': 'vTimeField', 'size': '8'}))
+
+        # For ForeignKey or ManyToManyFields, use a special widget.
+        if db_field.rel and isinstance(db_field.rel, (models.ManyToOneRel, models.ManyToManyRel)):
+            # Wrap the widget's render() method with a method that adds
+            # extra HTML to the end of the rendered output.
+            formfield = db_field.formfield()
+            formfield.widget.render = widgets.RelatedFieldWidgetWrapper(formfield.widget.render, db_field.rel)
+            return formfield
+
+        # For any other type of field, just call its formfield() method.
-        return db_field.formfield(**kwargs)
-

django/branches/newforms-admin/django/contrib/admin/templates/widget/foreign.html

@@ -1 +1 @@
-{% load admin_modify adminmedia %}
-{% output_all bound_field.form_fields %}
-{% if bound_field.raw_id_admin %}
-    {% if bound_field.field.rel.limit_choices_to %}
-        <a href="{{ bound_field.related_url }}?{% for limit_choice in bound_field.field.rel.limit_choices_to.items %}{% if not forloop.first %}&amp;{% endif %}{{ limit_choice|join:"=" }}{% endfor %}" class="related-lookup" id="lookup_{{ bound_field.element_id }}" onclick="return showRelatedObjectLookupPopup(this);"> <img src="{% admin_media_prefix %}img/admin/selector-search.gif" width="16" height="16" alt="Lookup"></a>
-    {% else %}
-        <a href="{{ bound_field.related_url }}" class="related-lookup" id="lookup_{{ bound_field.element_id }}" onclick="return showRelatedObjectLookupPopup(this);"> <img src="{% admin_media_prefix %}img/admin/selector-search.gif" width="16" height="16" alt="Lookup"></a>
-    {% endif %}
-{% else %}
-{% if bound_field.needs_add_label %}
-    <a href="{{ bound_field.related_url }}add/" class="add-another" id="add_{{ bound_field.element_id }}" onclick="return showAddAnotherPopup(this);"> <img src="{% admin_media_prefix %}img/admin/icon_addlink.gif" width="10" height="10" alt="Add Another"/></a>
-{% endif %}{% endif %}
-{% if change %}
-    {% if bound_field.field.primary_key %}

django/branches/newforms-admin/django/contrib/admin/views/main.py

@@ -64 +64 @@
-        self.original = original
-        self.form_fields = [field_mapping[name] for name in self.field.get_manipulator_field_names('')]
-        self.element_id = self.form_fields[0].get_id()
-        self.has_label_first = not isinstance(self.field, models.BooleanField)
-        self.raw_id_admin = use_raw_id_admin(field)
-        self.is_date_time = isinstance(field, models.DateTimeField)
-        self.is_file_field = isinstance(field, models.FileField)
-        self.needs_add_label = field.rel and (isinstance(field.rel, models.ManyToOneRel) or isinstance(field.rel, models.ManyToManyRel)) and field.rel.to._meta.admin
-        self.hidden = isinstance(self.field, models.AutoField)
-        self.first = False
@@ -81 +79 @@
-            self.cell_class_attribute = ' class="%s" ' % ' '.join(classes)
-        self._repr_filled = False
-
-        if field.rel:
-            self.related_url = '../../../%s/%s/' % (field.rel.to._meta.app_label, field.rel.to._meta.object_name.lower())
-
-    def original_value(self):

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

@@ -41 +41 @@
-        return u'<p class="datetime">%s %s<br />%s %s</p>' % \
-            (_('Date:'), rendered_widgets[0], _('Time:'), rendered_widgets[1])
+
+class RelatedFieldWidgetWrapper(object):
+    """
+    This class is a wrapper whose __call__() method mimics the interface of a
+    Widget's render() method.
+    """
+    def __init__(self, render_func, rel):
+        self.render_func, self.rel = render_func, rel
+
+    def __call__(self, name, value, *args, **kwargs):
+        from django.conf import settings
+        rel_to = self.rel.to
+        related_url = '../../../%s/%s/' % (rel_to._meta.app_label, rel_to._meta.object_name.lower())
+        output = [self.render_func(name, value, *args, **kwargs)]
+        if self.rel.raw_id_admin:
+            if self.rel.limit_choices_to:
+                url = '?' + '&amp;'.join(['%s=%s' % (k, v) for k, v in self.rel.limit_choices_to.items()])
+            else:
+                url = ''
+            # TODO: "id_" is hard-coded here. This should instead use the correct
+            # API to determine the ID dynamically.
+            output.append('<a href="%s%s" class="related-lookup" id="lookup_id_%s" onclick="return showRelatedObjectLookupPopup(this);"> ' % \
+                (related_url, url, name))
+            output.append('<img src="%simg/admin/selector-search.gif" width="16" height="16" alt="Lookup"></a>' % settings.ADMIN_MEDIA_PREFIX)
+            #if self.change: # TODO
+                #output.append('&nbsp;<strong>TODO</strong>')
+        elif rel_to._meta.admin: # If the related object has an admin interface:
+            # TODO: "id_" is hard-coded here. This should instead use the correct
+            # API to determine the ID dynamically.
+            output.append(u'<a href="%sadd/" class="add-another" id="add_id_%s" onclick="return showAddAnotherPopup(this);"> ' % \
+                (related_url, name))
+            output.append(u'<img src="%simg/admin/icon_addlink.gif" width="10" height="10" alt="Add Another"/></a>' % settings.ADMIN_MEDIA_PREFIX)
+        return u''.join(output)