Other people's code
I'm currently working on a 10+ year old code base and often come across some code that I am initially baffled by. def using_custom_theme? !custom_theme_id.nil? end My first thoughts were - why ? This is the sort of logic kungfu that really hurts my