Ticket #2569: adminforeign.3.diff

django/db/models/fields/related.py

@@ -566 +566 @@
 
         self.db_index = True
 
+    def get_related_pk(self):
+        """ Returning the primary key of the object we are related to
+        """
+        return self.get_choices()[1][0]
+
     def get_attname(self):
         return '%s_id' % self.name
 

django/contrib/admin/media/js/admin/RelatedObjectLookups.js

@@ -55 +55 @@
     }
     win.close();
 }
+
+function showEditPopup(triggeringLink) {
+    var name = triggeringLink.id.replace(/^edit_/,'');
+    var elem = document.getElementById(name);
+    if (elem) {
+        if (elem.nodeName == 'SELECT') {
+            var id = elem.options[elem.selectedIndex].value
+            name = name.replace(/\./g,'___');
+            var win = window.open(triggeringLink.href + id + '/?_popup=1',name, 'height=500,width=800,resizable=yes,scrollbars=yes');
+            return false;
+        }
+    }
+    return false;
+}
+
+function dismissEditPopup(win, newId, newRepr) {
+    var name = win.name.replace(/___/g, '.');
+    var elem = document.getElementById(name);
+    if(elem) {
+        if( elem.nodeName == 'SELECT') {
+            var o = new Option(newRepr, newId);
+            elem.options[elem.selectedIndex] = o;
+            o.selected = true;
+        }
+    }
+    win.close();
+}

django/contrib/admin/views/main.py

@@ -130 +130 @@
 
         if field.rel:
             self.related_url = '../../../%s/%s/' % (field.rel.to._meta.app_label, field.rel.to._meta.object_name.lower())
+            try:
+                self.rel_to_pk = field.get_related_pk() # for one-to-one fields
+            except:
+                pass
 
     def original_value(self):
         if self.original:
@@ -343 +347 @@
             LogEntry.objects.log_action(request.user.id, ContentType.objects.get_for_model(model).id, pk_value, str(new_object), CHANGE, change_message)
 
             msg = _('The %(name)s "%(obj)s" was changed successfully.') % {'name': opts.verbose_name, 'obj': new_object}
+            if request.REQUEST.has_key("_popup"):
+                http_response = '<script type="text/javascript">opener.dismissEditPopup(window, %s, "%s");</script>' % \
+                    (pk_value, str(new_object).replace('"', '\\"'))
+                return HttpResponse(http_response)
             if request.POST.has_key("_continue"):
                 request.user.message_set.create(message=msg + ' ' + _("You may edit it again below."))
                 if request.REQUEST.has_key('_popup'):

django/contrib/admin/templates/widget/one_to_one.html

@@ +1 @@
+{% load admin_modify adminmedia %}
 {% if add %}{% include "widget/foreign.html" %}{% endif %}
-{% if change %}{% if bound_field.existing_display %}&nbsp;<strong>{{ bound_field.existing_display|truncatewords:"14"|escape }}</strong>{% endif %}{% endif %}
+{% if change %}
+{% if bound_field.rel_to_pk %}&nbsp;<a href="{{ bound_field.related_url }}{{ bound_field.rel_to_pk}}"><strong>{{ bound_field.existing_display|truncatewords:"14"|escape }}</strong></a>{% endif %}{% endif %}

django/contrib/admin/templates/widget/foreign.html

@@ -8 +8 @@
     {% 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 %}
+    <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> 
+    <a href="{{ bound_field.related_url }}" class="related-lookup" id="edit_{{ bound_field.element_id }}" onclick="return showEditPopup(this);"> <img src="{% admin_media_prefix %}img/admin/selector-search.gif" width="16" height="16" alt="Lookup"></a>
+ 
+{% endif %}
+{% endif %}
 {% if change %}
     {% if bound_field.field.primary_key %}
         {{ bound_field.original_value }}