Redmine

The current approach to logging in MailHandler is a bit funny. Instead of checking "if logger" every time, just make sure logger returns non-nil.

Also, extra-checking with "if logger && logger.info" doesn't make much sense. It prints extra blank lines when logger is set and accepts info level, but has no point when level setting is different. It's perfectly safe to just call "logger.info message".