#14672 closed (fixed)
Admin doesn't handle new on_delete exceptions gracefully
Reported by: | Jani Tiainen | Owned by: | Carl Meyer |
---|---|---|---|
Component: | contrib.admin | Version: | dev |
Severity: | Keywords: | ||
Cc: | Triage Stage: | Accepted | |
Has patch: | no | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
Changes made in #7539 causes now admin to throw HTTP 500 when integrity exception happens leaving end user pretty much clueless that something "normal" has happened.
Change History (3)
comment:1 by , 14 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
Triage Stage: | Unreviewed → Accepted |
comment:3 by , 14 years ago
Ack - I mean [15249], of course. What's up with Trac's commit-message parser anyway?
Note:
See TracTickets
for help on using tickets.
Yes, something different should happen here, though it's not yet clear to me precisely what. Perhaps just a user-message informing that X cannot be deleted because it is referenced by Y? Though in the admin-actions case, it could be many Xs and many Ys, so it may need to be a separate page, like the delete-confirm page...