#15779: admin cannot edit records with value 'add' as primary key
-------------------------------------+-------------------------------------
Reporter: Marwan | Owner: nobody
Alsabbagh <marwan.alsabbagh@…> | Status: new
Type: Bug | Component: contrib.admin
Milestone: | Severity: Normal
Version: 1.3 | Keywords:
Resolution: | Has patch: 0
Triage Stage: Accepted | Needs tests: 0
Needs documentation: 0 |
Patch needs improvement: 0 |
-------------------------------------+-------------------------------------
Changes (by lukeplant):
* stage: Design decision needed => Accepted
Comment:
I agree with julien that the only sensible way to do this is to fix the
URL structure.
Technically I don't believe our backwards compatibility guarantee ever
included the URL structure of admin apps. Since Django 1.1 we've had a
fully capable URL reverser for the admin, and this has been the documented
way to reverse admin views. So I'm going to accept on that basis - since
we've documented the correct way to do it, if people have hard coded URLs
then that is like using private methods instead of the exposed public
interface. We should nonetheless include a note in the backwards
compatibilities section of the release note.
The patch would be quite simple, but there are some redirects which also
assume the current structure and would also need to be fixed.
(/me goes off to fix those hard-coded admin URLs in a really old Django
app...)
--
Ticket URL: <http://code.djangoproject.com/ticket/15779#comment:2>
Django <http://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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/django-updates?hl=en.
