inform participants on UI too

models/issue.go

@@ -1277,13 +1277,24 @@ func GetParticipantsIDsByIssueID(issueID int64) ([]int64, error) {
 
 // GetParticipantsByIssueID returns all users who are participated in comments of an issue.
 func GetParticipantsByIssueID(issueID int64) ([]*User, error) {
-	return getParticipantsByIssueID(x, issueID)
+	issue, err := getIssueByID(x, issueID)
+	if err != nil {
+		return nil, err
+	}
+	userIDs, err := getParticipantsByIssueID(x, issue)
+	if err != nil {
+		return nil, err
+	}
+	if len(userIDs) == 0 {
+		return nil, nil
+	}
+	return GetUsersByIDs(userIDs)
 }
 
-func getParticipantsByIssueID(e Engine, issueID int64) ([]*User, error) {
+func getParticipantsByIssueID(e Engine, issue *Issue) ([]int64, error) {
 	userIDs := make([]int64, 0, 5)
 	if err := e.Table("comment").Cols("poster_id").
-		Where("`comment`.issue_id = ?", issueID).
+		Where("`comment`.issue_id = ?", issue.ID).
 		And("`comment`.type in (?,?,?)", CommentTypeComment, CommentTypeCode, CommentTypeReview).
 		And("`user`.is_active = ?", true).
 		And("`user`.prohibit_login = ?", false).
@@ -1292,12 +1303,20 @@ func getParticipantsByIssueID(e Engine, issueID int64) ([]*User, error) {
 		Find(&userIDs); err != nil {
 		return nil, fmt.Errorf("get poster IDs: %v", err)
 	}
-	if len(userIDs) == 0 {
-		return nil, nil
+	if !util.IsInt64InSlice(issue.PosterID, userIDs) {
+		return append(userIDs, issue.PosterID), nil
 	}
+	return userIDs, nil
+}
 
-	users := make([]*User, 0, len(userIDs))
-	return users, e.In("id", userIDs).Find(&users)
+// IsUserParticipantsOfIssue return true if user is participants of an issue
+func IsUserParticipantsOfIssue(user *User, issue *Issue) bool {
+	userIDs, err := getParticipantsByIssueID(x, issue)
+	if err != nil {
+		log.Error(err.Error())
+		return false
+	}
+	return util.IsInt64InSlice(user.ID, userIDs)
 }
 
 // UpdateIssueMentions updates issue-user relations for mentioned users.

models/issue_test.go

@@ -81,7 +81,7 @@ func TestGetParticipantsByIssueID(t *testing.T) {
 	// User 2 only labeled issue1 (see fixtures/comment.yml)
 	// Users 3 and 5 made actual comments (see fixtures/comment.yml)
 	// User 3 is inactive, thus not active participant
-	checkParticipants(1, []int{5})
+	checkParticipants(1, []int{1, 5})
 }
 
 func TestIssue_ClearLabels(t *testing.T) {

models/notification.go

@@ -160,6 +160,14 @@ func createOrUpdateIssueNotifications(e Engine, issueID, commentID int64, notifi
 		toNotify[id] = struct{}{}
 	}
 
+	issueParticipants, err := getParticipantsByIssueID(e, issue)
+	if err != nil {
+		return err
+	}
+	for _, id := range issueParticipants {
+		toNotify[id] = struct{}{}
+	}
+
 	// dont notify user who cause notification
 	delete(toNotify, notificationAuthorID)
 	// explicit unwatch on issue

modules/util/compare.go

@@ -45,6 +45,16 @@ func IsStringInSlice(target string, slice []string) bool {
 	return false
 }
 
+// IsInt64InSlice sequential searches if int64 exists in slice.
+func IsInt64InSlice(target int64, slice []int64) bool {
+	for i := 0; i < len(slice); i++ {
+		if slice[i] == target {
+			return true
+		}
+	}
+	return false
+}
+
 // IsEqualSlice returns true if slices are equal.
 func IsEqualSlice(target []string, source []string) bool {
 	if len(target) != len(source) {

routers/repo/issue.go

@@ -704,7 +704,7 @@ func ViewIssue(ctx *context.Context) {
 			iw = &models.IssueWatch{
 				UserID:     ctx.User.ID,
 				IssueID:    issue.ID,
-				IsWatching: models.IsWatching(ctx.User.ID, ctx.Repo.Repository.ID),
+				IsWatching: models.IsWatching(ctx.User.ID, ctx.Repo.Repository.ID) || models.IsUserParticipantsOfIssue(ctx.User, issue),
 			}
 		}
 	}