API: add owner filter to repo.ListPullRequests

Author: 6543

models/issues/pull_list.go

@@ -26,6 +26,7 @@ type PullRequestsOptions struct {
 	SortType    string
 	Labels      []int64
 	MilestoneID int64
+	OwnerID     int64
 }
 
 func listPullRequestStatement(ctx context.Context, baseRepoID int64, opts *PullRequestsOptions) *xorm.Session {
@@ -46,6 +47,10 @@ func listPullRequestStatement(ctx context.Context, baseRepoID int64, opts *PullR
 		sess.And("issue.milestone_id=?", opts.MilestoneID)
 	}
 
+	if opts.OwnerID > 0 {
+		sess.And("issue.owner_id=?", opts.OwnerID)
+	}
+
 	return sess
 }
 

routers/api/v1/repo/pull.go

@@ -83,6 +83,10 @@ func ListPullRequests(ctx *context.APIContext) {
 	//   items:
 	//     type: integer
 	//     format: int64
+	// - name: owner
+	//   in: query
+	//   description: filter by owner
+	//   type: string
 	// - name: page
 	//   in: query
 	//   description: page number of results to return (1-based)
@@ -102,13 +106,27 @@ func ListPullRequests(ctx *context.APIContext) {
 		ctx.Error(http.StatusInternalServerError, "PullRequests", err)
 		return
 	}
+	var ownerID int64
+	if ctx.FormString("owner") != "" {
+		owner, err := user_model.GetUserByName(ctx, ctx.FormString("owner"))
+		if err != nil {
+			if user_model.IsErrUserNotExist(err) {
+				ctx.Error(http.StatusBadRequest, "Owner not found", err)
+			} else {
+				ctx.Error(http.StatusInternalServerError, "GetUserByName", err)
+			}
+			return
+		}
+		ownerID = owner.ID
+	}
 	listOptions := utils.GetListOptions(ctx)
 	prs, maxResults, err := issues_model.PullRequests(ctx, ctx.Repo.Repository.ID, &issues_model.PullRequestsOptions{
 		ListOptions: listOptions,
 		State:       ctx.FormTrim("state"),
 		SortType:    ctx.FormTrim("sort"),
 		Labels:      labelIDs,
 		MilestoneID: ctx.FormInt64("milestone"),
+		OwnerID:     ownerID,
 	})
 	if err != nil {
 		ctx.Error(http.StatusInternalServerError, "PullRequests", err)