Revert "gh-57: added support for paginating ListForAccount and ListForTeam (#58)" (#61)

This reverts commit 02078dc.

Author: ktrysmt

.gitignore

@@ -1,2 +1 @@
 vendor
-.idea

Gopkg.lock

@@ -1,7 +1,7 @@
 # go-bitbucket
 
-<a class="repo-badge" href="https://godoc.org/github.com/davidji99/go-bitbucket"><img src="https://godoc.org/github.com/davidji99/go-bitbucket?status.svg" alt="go-bitbucket?status"></a>
-<a href="https://goreportcard.com/report/github.com/davidji99/go-bitbucket"><img class="badge" tag="github.com/davidji99/go-bitbucket" src="https://goreportcard.com/badge/github.com/davidji99/go-bitbucket"></a>
+<a class="repo-badge" href="https://godoc.org/github.com/ktrysmt/go-bitbucket"><img src="https://godoc.org/github.com/ktrysmt/go-bitbucket?status.svg" alt="go-bitbucket?status"></a>
+<a href="https://goreportcard.com/report/github.com/ktrysmt/go-bitbucket"><img class="badge" tag="github.com/ktrysmt/go-bitbucket" src="https://goreportcard.com/badge/github.com/ktrysmt/go-bitbucket"></a>
 
 > Bitbucket-API library for golang.
 
@@ -15,7 +15,7 @@ And the response type is json format defined Bitbucket API.
 ## Install
 
 ```sh
-go get github.com/davidji99/go-bitbucket
+go get github.com/ktrysmt/go-bitbucket
 ```
 
 ## Usage
@@ -26,7 +26,7 @@ package main
 import (
         "fmt"
 
-        "github.com/davidji99/go-bitbucket"
+        "github.com/ktrysmt/go-bitbucket" 
 )
 
 func main() {
@@ -71,7 +71,7 @@ It's using dep.
 
 ```sh
 go get github.com/golang/dep/...
-git clone https://github.com/davidji99/go-bitbucket 
+git clone https://github.com/ktrysmt/go-bitbucket 
 cd ./go-bitbucket
 dep ensure 
 ```

README.md

@@ -1,15 +1,5 @@
 package bitbucket
 
-import (
-	"encoding/json"
-	"fmt"
-	"io/ioutil"
-	"net/http"
-	"net/url"
-	"sort"
-	"strings"
-)
-
 var apiBaseURL = "https://api.bitbucket.org/2.0"
 
 func GetApiBaseURL() string {
@@ -33,17 +23,17 @@ type user interface {
 }
 
 type pullrequests interface {
-	Create(opt CreatePullRequestOpts) (interface{}, error)
-	Update(opt CreatePullRequestOpts) (interface{}, error)
-	List(opt CreatePullRequestOpts) (interface{}, error)
-	Get(opt CreatePullRequestOpts) (interface{}, error)
-	Activities(opt CreatePullRequestOpts) (interface{}, error)
-	Activity(opt CreatePullRequestOpts) (interface{}, error)
-	Commits(opt CreatePullRequestOpts) (interface{}, error)
-	Patch(opt CreatePullRequestOpts) (interface{}, error)
-	Diff(opt CreatePullRequestOpts) (interface{}, error)
-	Merge(opt CreatePullRequestOpts) (interface{}, error)
-	Decline(opt CreatePullRequestOpts) (interface{}, error)
+	Create(opt PullRequestsOptions) (interface{}, error)
+	Update(opt PullRequestsOptions) (interface{}, error)
+	List(opt PullRequestsOptions) (interface{}, error)
+	Get(opt PullRequestsOptions) (interface{}, error)
+	Activities(opt PullRequestsOptions) (interface{}, error)
+	Activity(opt PullRequestsOptions) (interface{}, error)
+	Commits(opt PullRequestsOptions) (interface{}, error)
+	Patch(opt PullRequestsOptions) (interface{}, error)
+	Diff(opt PullRequestsOptions) (interface{}, error)
+	Merge(opt PullRequestsOptions) (interface{}, error)
+	Decline(opt PullRequestsOptions) (interface{}, error)
 }
 
 type repository interface {
@@ -105,15 +95,9 @@ type teams interface {
 	Projects(teamname string) (interface{}, error)
 }
 
-type ListOptions struct {
-	Page    uint64 `json:"page"`
-	PageLen uint64 `json:"pagelen"`
-}
-
 type RepositoriesOptions struct {
-	ListOptions *ListOptions `json:"list_options"`
-	Owner       string       `json:"owner"`
-	Role        string       `json:"role"` // role=[owner|admin|contributor|member]
+	Owner string `json:"owner"`
+	Role  string `json:"role"` // role=[owner|admin|contributor|member]
 }
 
 type RepositoryOptions struct {
@@ -130,6 +114,22 @@ type RepositoryOptions struct {
 	Project     string `json:"project"`
 }
 
+type PullRequestsOptions struct {
+	ID                string   `json:"id"`
+	CommentID         string   `json:"comment_id"`
+	Owner             string   `json:"owner"`
+	RepoSlug          string   `json:"repo_slug"`
+	Title             string   `json:"title"`
+	Description       string   `json:"description"`
+	CloseSourceBranch bool     `json:"close_source_branch"`
+	SourceBranch      string   `json:"source_branch"`
+	SourceRepository  string   `json:"source_repository"`
+	DestinationBranch string   `json:"destination_branch"`
+	DestinationCommit string   `json:"destination_repository"`
+	Message           string   `json:"message"`
+	Reviewers         []string `json:"reviewers"`
+}
+
 type CommitsOptions struct {
 	Owner       string `json:"owner"`
 	RepoSlug    string `json:"repo_slug"`
@@ -141,11 +141,12 @@ 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 {
@@ -200,78 +201,8 @@ type RepositoryPipelineKeyPairOptions struct {
 }
 
 type DownloadsOptions struct {
-	Owner    string `json:"owner"`
-	RepoSlug string `json:"repo_slug"`
-	FilePath string `json:"filepath"`
-	FileName string `json:"filename"`
-}
-
-type Response struct {
-	*http.Response
-}
-
-// newResponse creates a new Response for the provided http.Response.
-func newResponse(r *http.Response) *Response {
-	response := &Response{Response: r}
-	return response
-}
-
-type ErrorResponse struct {
-	Body     []byte
-	Response *http.Response
-	Message  string
-}
-
-func (e *ErrorResponse) Error() string {
-	path, _ := url.QueryUnescape(e.Response.Request.URL.Path)
-	u := fmt.Sprintf("%s://%s%s", e.Response.Request.URL.Scheme, e.Response.Request.URL.Host, path)
-	return fmt.Sprintf("%s %s: %d %s", e.Response.Request.Method, u, e.Response.StatusCode, e.Message)
-}
-
-// CheckResponse checks the API response for errors, and returns them if present.
-func CheckResponse(r *http.Response) error {
-	switch r.StatusCode {
-	case 200, 201, 202, 204, 304:
-		return nil
-	}
-
-	errorResponse := &ErrorResponse{Response: r}
-	data, err := ioutil.ReadAll(r.Body)
-	if err == nil && data != nil {
-		errorResponse.Body = data
-
-		var raw interface{}
-		if err := json.Unmarshal(data, &raw); err != nil {
-			errorResponse.Message = "failed to parse unknown error format"
-		} else {
-			errorResponse.Message = parseError(raw)
-		}
-	}
-
-	return errorResponse
-}
-
-func parseError(raw interface{}) string {
-	switch raw := raw.(type) {
-	case string:
-		return raw
-
-	case []interface{}:
-		var errs []string
-		for _, v := range raw {
-			errs = append(errs, parseError(v))
-		}
-		return fmt.Sprintf("[%s]", strings.Join(errs, ", "))
-
-	case map[string]interface{}:
-		var errs []string
-		for k, v := range raw {
-			errs = append(errs, fmt.Sprintf("{%s: %s}", k, parseError(v)))
-		}
-		sort.Strings(errs)
-		return strings.Join(errs, ", ")
-
-	default:
-		return fmt.Sprintf("failed to parse unexpected error type: %T", raw)
-	}
-}
+	Owner       string `json:"owner"`
+	RepoSlug    string `json:"repo_slug"`
+	FilePath    string `json:"filepath"`
+	FileName    string `json:"filename"`
+}

bitbucket.go

@@ -2,8 +2,9 @@ package bitbucket
 
 import (
 	"encoding/json"
-	"fmt"
 	"os"
+
+	"github.com/k0kubun/pp"
 )
 
 type BranchRestrictions struct {
@@ -12,29 +13,29 @@ type BranchRestrictions struct {
 
 func (b *BranchRestrictions) Gets(bo *BranchRestrictionsOptions) (interface{}, error) {
 	urlStr := b.c.requestUrl("/repositories/%s/%s/branch-restrictions", bo.Owner, bo.RepoSlug)
-	return b.c.execute("GET", urlStr, "", "")
+	return b.c.execute("GET", urlStr, "")
 }
 
 func (b *BranchRestrictions) Create(bo *BranchRestrictionsOptions) (interface{}, error) {
 	data := b.buildBranchRestrictionsBody(bo)
 	urlStr := b.c.requestUrl("/repositories/%s/%s/branch-restrictions", bo.Owner, bo.RepoSlug)
-	return b.c.execute("POST", urlStr, data, "")
+	return b.c.execute("POST", urlStr, data)
 }
 
 func (b *BranchRestrictions) Get(bo *BranchRestrictionsOptions) (interface{}, error) {
 	urlStr := b.c.requestUrl("/repositories/%s/%s/branch-restrictions/%s", bo.Owner, bo.RepoSlug, bo.ID)
-	return b.c.execute("GET", urlStr, "", "")
+	return b.c.execute("GET", urlStr, "")
 }
 
 func (b *BranchRestrictions) Update(bo *BranchRestrictionsOptions) (interface{}, error) {
 	data := b.buildBranchRestrictionsBody(bo)
 	urlStr := b.c.requestUrl("/repositories/%s/%s/branch-restrictions/%s", bo.Owner, bo.RepoSlug, bo.ID)
-	return b.c.execute("PUT", urlStr, data, "")
+	return b.c.execute("PUT", urlStr, data)
 }
 
 func (b *BranchRestrictions) Delete(bo *BranchRestrictionsOptions) (interface{}, error) {
 	urlStr := b.c.requestUrl("/repositories/%s/%s/branch-restrictions/%s", bo.Owner, bo.RepoSlug, bo.ID)
-	return b.c.execute("DELETE", urlStr, "", "")
+	return b.c.execute("DELETE", urlStr, "")
 }
 
 type branchRestrictionsBody struct {
@@ -121,7 +122,7 @@ func (b *BranchRestrictions) buildBranchRestrictionsBody(bo *BranchRestrictionsO
 
 	data, err := json.Marshal(body)
 	if err != nil {
-		fmt.Println(err)
+		pp.Println(err)
 		os.Exit(9)
 	}