(Minor) You call `subject_text` three or four times in this method. This actually invokes the HTML cleaner every time... that's kind of heavyweight. You should create a local variable with the subject and operate on that later.