#30386: Admin foreign key widgets don't quote keys.
--------------------------------+------------------------------------
     Reporter:  Joshua Goodwin  |                    Owner:  zeynel
         Type:  Bug             |                   Status:  assigned
    Component:  contrib.admin   |                  Version:  dev
     Severity:  Normal          |               Resolution:
     Keywords:                  |             Triage Stage:  Accepted
    Has patch:  1               |      Needs documentation:  0
  Needs tests:  0               |  Patch needs improvement:  1
Easy pickings:  0               |                    UI/UX:  0
--------------------------------+------------------------------------

Comment (by Alexander Pervakov):

 I'm dive deeper into this and found that `quote(obj.id)` maybe not the
 best option and should be debated but the presence of a warning message
 when `not isinstance(obj.pk, int)` would be great and will lead people to
 this thread. Because stackoverflow is full of questions about quoting the
 link in admin but no one provide refs to this ticket that I found after
 couple hours of code investigating and testing.

-- 
Ticket URL: <https://code.djangoproject.com/ticket/30386#comment:6>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/068.a0a2ea3692ef8b80221440e6d68b7d1f%40djangoproject.com.

Reply via email to