Ticket #11163: 11163.patch.1

File 11163.patch.1, 2.3 KB (added by Antti Kaihola, 15 years ago)

Patch against r11173: imitating what RelatedFieldWidgetWrapper does

Line 
1commit 1eadf1be1bf986794961407a26cbe408bcf4af13
2Author: Antti Kaihola <akaihol+django@ambitone.com>
3Date: Fri Jul 3 14:07:14 2009 +0300
4
5 Fixed #11163 -- Editable raw ID field pop-up link now correct also in admin changelists
6
7diff --git a/django/contrib/admin/options.py b/django/contrib/admin/options.py
8index 8297eca..e1660a2 100644
9--- a/django/contrib/admin/options.py
10+++ b/django/contrib/admin/options.py
11@@ -139,7 +139,7 @@ class BaseModelAdmin(object):
12 Get a form Field for a ForeignKey.
13 """
14 if db_field.name in self.raw_id_fields:
15- kwargs['widget'] = widgets.ForeignKeyRawIdWidget(db_field.rel)
16+ kwargs['widget'] = widgets.ForeignKeyRawIdWidget(db_field.rel, self.admin_site)
17 elif db_field.name in self.radio_fields:
18 kwargs['widget'] = widgets.AdminRadioSelect(attrs={
19 'class': get_ul_class(self.radio_fields[db_field.name]),
20diff --git a/django/contrib/admin/widgets.py b/django/contrib/admin/widgets.py
21index 7ae5e64..960bb3c 100644
22--- a/django/contrib/admin/widgets.py
23+++ b/django/contrib/admin/widgets.py
24@@ -101,14 +101,23 @@ class ForeignKeyRawIdWidget(forms.TextInput):
25 A Widget for displaying ForeignKeys in the "raw_id" interface rather than
26 in a <select> box.
27 """
28- def __init__(self, rel, attrs=None):
29+ def __init__(self, rel, admin_site, attrs=None):
30 self.rel = rel
31+ self.admin_site = admin_site
32 super(ForeignKeyRawIdWidget, self).__init__(attrs)
33
34 def render(self, name, value, attrs=None):
35 if attrs is None:
36 attrs = {}
37- related_url = '../../../%s/%s/' % (self.rel.to._meta.app_label, self.rel.to._meta.object_name.lower())
38+ # TODO: DRY violation, copied code from
39+ # RelatedFieldWidgetWrapper.render() below to fix #11163
40+ rel_to = self.rel.to
41+ info = (rel_to._meta.app_label, rel_to._meta.object_name.lower())
42+ try:
43+ related_info = (self.admin_site.name,) + info
44+ related_url = reverse('%sadmin_%s_%s_changelist' % related_info)
45+ except NoReverseMatch:
46+ related_url = '../../../%s/%s/' % info
47 params = self.url_parameters()
48 if params:
49 url = '?' + '&amp;'.join(['%s=%s' % (k, v) for k, v in params.items()])
Back to Top