Fix GetPosters to work with issues

I forgot to consider that this function was also being used for issue pages, so this now allows both issues and PRs to be handled.

models/repo/user_repo.go

@@ -171,13 +171,13 @@ func GetReviewers(ctx context.Context, repo *Repository, doerID, posterID int64)
 	return users, db.GetEngine(ctx).Where(cond).OrderBy(user_model.GetOrderByName()).Find(&users)
 }
 
-// GetPullRequestPosters returns all users that have authored a pull request for the given repository
-func GetPullRequestPosters(ctx context.Context, repo *Repository) ([]*user_model.User, error) {
+// GetIssuePosters returns all users that have authored a pull request for the given repository
+func GetIssuePosters(ctx context.Context, repo *Repository, isPull bool) ([]*user_model.User, error) {
 	users := make([]*user_model.User, 0, 8)
 	cond := builder.In("`user`.id",
 		builder.Select("poster_id").From("issue").Where(
 			builder.Eq{"repo_id": repo.ID}.
-				And(builder.Eq{"is_pull": true}),
+				And(builder.Eq{"is_pull": isPull}),
 		).GroupBy("poster_id"),
 	)
 	return users, db.GetEngine(ctx).Where(cond).OrderBy(user_model.GetOrderByName()).Find(&users)

routers/web/repo/issue.go

@@ -291,7 +291,7 @@ func issues(ctx *context.Context, milestoneID, projectID int64, isPullOption uti
 		return
 	}
 
-	ctx.Data["Posters"], err = repo_model.GetPullRequestPosters(ctx, repo)
+	ctx.Data["Posters"], err = repo_model.GetIssuePosters(ctx, repo, isPullOption.IsTrue())
 	if err != nil {
 		ctx.ServerError("GetPullRequestPosters", err)
 		return