The layout of your code makes it hard to read. Your indentation isn't consistent, and defining subs inside if constructs doesn't win any prices for readability either.

But I see your print statement is inside a couple of if/then/else constructs. Perhaps the conditions aren't what you think they are? Does the email get send?