Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#14672 closed (fixed)

Admin doesn't handle new on_delete exceptions gracefully

Reported by: Jani Tiainen Owned by: Carl Meyer
Component: contrib.admin Version: master
Severity: Keywords:
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

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 Changed 6 years ago by Carl Meyer

Needs documentation: unset
Needs tests: unset
Owner: changed from nobody to Carl Meyer
Patch needs improvement: unset
Status: newassigned
Triage Stage: UnreviewedAccepted

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...

comment:2 Changed 6 years ago by Carl Meyer

Resolution: fixed
Status: assignedclosed

Fixed in [14672].

comment:3 Changed 6 years ago by Carl Meyer

Ack - I mean [15249], of course. What's up with Trac's commit-message parser anyway?

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