Opened 4 years ago

Closed 4 years ago

Last modified 3 years ago

#14414 closed (fixed)

Additional check in content-type view to prevent 500

Reported by: subsume Owned by: nobody
Component: Contrib apps Version: 1.2
Severity: Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

In the event that your ContentType doesn't have a model_class associated with it anymore, it would be nice if the resulting url didn't chokedie 500 style.

Attachments (1)

ctviews.patch (731 bytes) - added by subsume 4 years ago.

Download all attachments as: .zip

Change History (6)

Changed 4 years ago by subsume

comment:1 Changed 4 years ago by subsume

  • Has patch set
  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset

comment:2 Changed 4 years ago by subsume

  • Triage Stage changed from Unreviewed to Ready for checkin

Marking RFC. Hope this is non-controversial enough.

comment:3 Changed 4 years ago by adrian

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

(In [13994]) Fixed #14414 -- Improved contenttypes shortcut() view to check that the ContentType has a model_class(). Thanks, subsume

comment:4 Changed 4 years ago by russellm

(In [14385]) [1.2.X] Fixed #14414 -- Improved contenttypes shortcut() view to check that the ContentType has a model_class(). Thanks, subsume

Backport of r13994 from trunk.

comment:5 Changed 3 years ago by jacob

  • milestone 1.3 deleted

Milestone 1.3 deleted

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