Did you verify that they checked the newsetter box? Did you try printing the value of $newsletter before the if statement?