Opened 13 years ago

Closed 13 years ago

#16542 closed Bug (fixed)

Admin Raw ID widgets can show broken magnifying glass link

Reported by: Ramiro Morales Owned by: Ramiro Morales
Component: contrib.admin Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description (last modified by Ramiro Morales)

This can happen when the related model isn't registered with the same admin site.

Ideally these widgets should behave in a way consistent with the regular non-raw ID admin widgets in this regard.

Example:

# models.py

from django.db import models

class Honeycomb(models.Model):
    location = models.CharField(max_length=20)

class Bee(models.Model):
    honeycomb = models.ForeignKey(Honeycomb)
# admin.py

from django.contrib import admin

import models

class BeeAdmin(admin.ModelAdmin):                                                                                                                                            
    raw_id_fields = ('honeycomb',) 

admin.site.register(models.Bee)

Try to add a Bee instance, in the add view click in the magnifying glass besides the 'Honeycomb' field, this leads to a 404 error.

Attachments (1)

16542.2.diff (20.5 KB ) - added by Ramiro Morales 13 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 by Ramiro Morales, 13 years ago

Description: modified (diff)
Owner: changed from nobody to Ramiro Morales

by Ramiro Morales, 13 years ago

Attachment: 16542.2.diff added

comment:2 by Ramiro Morales, 13 years ago

Has patch: set

comment:3 by Aymeric Augustin, 13 years ago

Triage Stage: UnreviewedAccepted

comment:4 by anonymous, 13 years ago

I see no reason for

except NoReverseMatch: 
    raise 

Aside from that it looks good I guess ;)

in reply to:  4 comment:5 by Ramiro Morales, 13 years ago

Replying to anonymous:

I see no reason for

except NoReverseMatch: 
    raise 

Aside from that it looks good I guess ;)

Point taken. Thanks

comment:6 by Ramiro Morales, 13 years ago

Resolution: fixed
Status: newclosed

In [16578]:

Fixed #16542 -- Made Raw ID form widgets shipped with the admin app render the related object lookup tool only when the related model is effectively registered with the AdminSite.

Also, converted these widgets to reverse named URLs instead of hard-coded '../../...'-style links, refs #15294.
Thanks Florian Apolloner for the initial patch.

Note: See TracTickets for help on using tickets.
Back to Top