Possible solution for #38 (#62)

* Added states query params to the pullrequests API call as per https://developer.atlassian.com/bitbucket/api/2/reference/resource/repositories/%7Busername%7D/%7Brepo_slug%7D/pullrequests#get.

* Fixed atypo and removed some debug.

* Added a gaurd to ensure we have states to process.

* Added API querying support as per https://developer.atlassian.com/bitbucket/api/2/reference/meta/filtering.

* Fixed the mangling of the query string.

* Added sort symantics.

Author: charl

bitbucket.go

@@ -128,6 +128,9 @@ type PullRequestsOptions struct {
 	DestinationCommit string   `json:"destination_repository"`
 	Message           string   `json:"message"`
 	Reviewers         []string `json:"reviewers"`
+	States            []string `json:"states"`
+	Query             string   `json:"query"`
+	Sort              string   `json:"sort"`
 }
 
 type CommitsOptions struct {
@@ -141,12 +144,11 @@ type CommitsOptions struct {
 }
 
 type CommitStatusOptions struct {
-	Key 		string `json:"key"`
-	Url 		string `json:"url"`
-	State 		string `json:"state"`
-	Name 		string `json:"name"`
+	Key         string `json:"key"`
+	Url         string `json:"url"`
+	State       string `json:"state"`
+	Name        string `json:"name"`
 	Description string `json:"description"`
-
 }
 
 type BranchRestrictionsOptions struct {
@@ -201,8 +203,8 @@ type RepositoryPipelineKeyPairOptions struct {
 }
 
 type DownloadsOptions struct {
-	Owner       string `json:"owner"`
-	RepoSlug    string `json:"repo_slug"`
-	FilePath    string `json:"filepath"`
-	FileName    string `json:"filename"`
-}
+	Owner    string `json:"owner"`
+	RepoSlug string `json:"repo_slug"`
+	FilePath string `json:"filepath"`
+	FileName string `json:"filename"`
+}

pullrequests.go

@@ -2,6 +2,7 @@ package bitbucket
 
 import (
 	"encoding/json"
+	"net/url"
 	"os"
 
 	"github.com/k0kubun/pp"
@@ -25,6 +26,42 @@ func (p *PullRequests) Update(po *PullRequestsOptions) (interface{}, error) {
 
 func (p *PullRequests) Gets(po *PullRequestsOptions) (interface{}, error) {
 	urlStr := GetApiBaseURL() + "/repositories/" + po.Owner + "/" + po.RepoSlug + "/pullrequests/"
+
+	if po.States != nil && len(po.States) != 0 {
+		parsed, err := url.Parse(urlStr)
+		if err != nil {
+			return nil, err
+		}
+		query := parsed.Query()
+		for _, state := range po.States {
+			query.Set("state", state)
+		}
+		parsed.RawQuery = query.Encode()
+		urlStr = parsed.String()
+	}
+
+	if po.Query != "" {
+		parsed, err := url.Parse(urlStr)
+		if err != nil {
+			return nil, err
+		}
+		query := parsed.Query()
+		query.Set("q", po.Query)
+		parsed.RawQuery = query.Encode()
+		urlStr = parsed.String()
+	}
+
+	if po.Sort != "" {
+		parsed, err := url.Parse(urlStr)
+		if err != nil {
+			return nil, err
+		}
+		query := parsed.Query()
+		query.Set("sort", po.Sort)
+		parsed.RawQuery = query.Encode()
+		urlStr = parsed.String()
+	}
+
 	return p.c.execute("GET", urlStr, "")
 }