Do not queue emails with no recipients (issue 200)

James Moger

2013-03-28 4aeec822d0fe9306c4ca64f1a1d9683f77684198

Do not queue emails with no recipients (issue-200)

 releases.moxie

@@ -10,6 +10,7 @@
     - Fix NPE when getting user's fork without repository list caching (issue 182)

     - Fix internal error on folder history links (issue 192)

     - Fixed incorrect icon file name for .doc files (issue 200)

     - Do not queue emails with no recipients (issue 201)



    additions: 

     - Option to force client-side basic authentication instead of form-based authentication if web.authenticateViewPages=true (issue 222)


 src/main/java/com/gitblit/GitBlit.java

@@ -2903,6 +2903,10 @@
     * @param toAddresses

     */

    public void sendMail(String subject, String message, String... toAddresses) {

        if (toAddresses == null || toAddresses.length == 0) {

            logger.debug(MessageFormat.format("Dropping message {0} because there are no recipients", subject));

            return;

        }

        try {

            Message mail = mailExecutor.createMessage(toAddresses);

            if (mail != null) {

@@ -2934,6 +2938,10 @@
     * @param toAddresses

     */

    public void sendHtmlMail(String subject, String message, String... toAddresses) {

        if (toAddresses == null || toAddresses.length == 0) {

            logger.debug(MessageFormat.format("Dropping message {0} because there are no recipients", subject));

            return;

        }

        try {

            Message mail = mailExecutor.createMessage(toAddresses);

            if (mail != null) {