I thought that too, however currently there is no way to find out if the user has changed the mail settings in the web-ui above or not, so you need to write the settings first and to send the email then. 
But as it is now is also incorrect, I need to move the line which sends the email a bit down in the controller within the `unless` condition.