Throw the exception only when an email is about to be sent

[jlaute]

Hey Guys,

first of all thank you for this plugin. It can be a life saver when someone has imported some staging/live database and forgets to reset the mailer config.

I do not have a real issue with the plugin, but instead I would suggest to not break the whole shop when a mailer config is not set correctly. Maybe it would be better to have some big warnings in front- and backend instead of throwing an exception globally. Throwing the exception just when some E-Mail is about to be sent is enough in my opinion.

[kleinmann]

Hi @jlaute, this seems like a reasonable change, thanks for suggesting 🙂
In this first iteration, a crash&burn handling was sufficient for us to remove any doubt of an invalid mail configuration. It can be disruptive with database imports, I agree 🙂

Are you interested in preparing a pull request for this? Otherwise it may be some time until we get around to this.
Something to keep in mind: It is still important to me, that any action in the Storefront, Administration and Console notifies the user about the wrong configuration.