After the review we decided it would be better to follow a more strict interface for our messages then a more open interface. A strict interface implies that the user should look at the interface and figure out all the dependencies for a particular object, while the open interface implies that the user must know/figure out the objects that the mail message requires.

To facilitate this change new methods are going to be added to the MailMessageFactory. For each different type of message we will have a new create method for that type of message that takes all that messages object requirements as parameters.

for example:

public MailMessage createAnnotationAddedMessage(Annotation annotation);

public MailMessage createNewUserWelcomeMessage(User user);