I don't know if I'm ready to join the anti-if campaign, but the point about not handling failure cases and not writing a conditional without a test to motivate it makes sense. I can't think of a situation where the `#destroy` will fail besides a database connection error of sorts, so for the time being the conditional has been removed (3dc5165783875fa1b6fe482eb5c03b2b2cc6b17e).

Is a failed database connection something I should be testing for and handling? It seems to be a bit of an edge case and would thus be sufficiently covered by a 500 internal error.