Shoudn't this be in an 'if' block?

Also dodgy `</` at end of the line