For a concise and more flexible API, completely replace Mailer constructors with MailerBuilder. Add support completely ignoring property defaults. · Issue #115 · bbottema/simple-java-mail · GitHub
You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
For a concise and more flexible API, completely replace Mailer constructors with MailerBuilder. Add support completely ignoring property defaults.
#115
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
Currently, the constructors are growing wild in the Mailer class. This is due to the fact that, programmaticcally, there is no invalid combination of parameters as any missing argument can be compensated with a property default.
As such, to keep the Mailer API simple and flexible at the same time, all constructors should be replaced in favor of a new MailerBuilder class.
The text was updated successfully, but these errors were encountered:
bbottema
changed the title
For a concise and more flexible API, completely replace all Mailer constructors with a new MailerBuilder
For a concise and more flexible API, completely replace Mailer constructors with MailerBuilder. Add support completely ignoring property defaults.
Dec 8, 2017
Currently, the constructors are growing wild in the Mailer class. This is due to the fact that, programmaticcally, there is no invalid combination of parameters as any missing argument can be compensated with a property default.
As such, to keep the Mailer API simple and flexible at the same time, all constructors should be replaced in favor of a new MailerBuilder class.
The text was updated successfully, but these errors were encountered: