From ec5a967937a8dd4fb5287591e8dedcdedfa2b32d Mon Sep 17 00:00:00 2001
From: "Renato P. de Aquino" <renato.aquino@gmail.com>
Date: Mon, 30 Jan 2017 11:35:12 -0200
Subject: Issue #3814 (#3903)

---
 models/user.go | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

(limited to 'models')

diff --git a/models/user.go b/models/user.go
index 18221756..e69a6798 100644
--- a/models/user.go
+++ b/models/user.go
@@ -469,6 +469,12 @@ func (u *User) ShortName(length int) string {
 	return base.EllipsisString(u.Name, length)
 }
 
+// IsMailable checks if a user is elegible
+// to receive emails.
+func (u *User) IsMailable() bool {
+	return u.IsActive
+}
+
 // IsUserExist checks if given user name exist,
 // the user name should be noncased unique.
 // If uid is presented, then check will rule out that one,
@@ -929,7 +935,9 @@ func GetUserEmailsByNames(names []string) []string {
 		if err != nil {
 			continue
 		}
-		mails = append(mails, u.Email)
+		if u.IsMailable() {
+			mails = append(mails, u.Email)
+		}
 	}
 	return mails
 }
-- 
cgit v1.2.3