Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#14672 closed (fixed)

Admin doesn't handle new on_delete exceptions gracefully

Reported by: jtiai Owned by: carljm
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:


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 5 years ago by carljm

  • Needs documentation unset
  • Needs tests unset
  • Owner changed from nobody to carljm
  • Patch needs improvement unset
  • Status changed from new to assigned
  • Triage Stage changed from Unreviewed to Accepted

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 5 years ago by carljm

  • Resolution set to fixed
  • Status changed from assigned to closed

Fixed in [14672].

comment:3 Changed 5 years ago by carljm

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