Use cascading deletes on models more sparingly

[BenMusch]

Right now, basically all foreign keys cascade when deleted. This isn't intrinsically necessary, but because so many places in the front-end assume certain relationships exist, cascade is used to prevent 500 errors when rendering the templates. We should revise it so that relationships which can be nullified don't cause unnecessary errors