No need to specify the owner to list repositories (#163)

chmouel · web-flow · commit aed0f1f90b92 · 2021-10-02T22:27:47.000+09:00
We can list repositories to /repositories for an owner if we define a
scope so we let allow it.

Signed-off-by: Chmouel Boudjnah &lt;chmouel@chmouel.com&gt;
diff --git a/repositories.go b/repositories.go
@@ -2,6 +2,7 @@ package bitbucket
 
 import (
 	"errors"
+  "fmt"
 
 	"github.com/mitchellh/mapstructure"
 )
@@ -31,7 +32,11 @@ type RepositoriesRes struct {
 }
 
 func (r *Repositories) ListForAccount(ro *RepositoriesOptions) (*RepositoriesRes, error) {
-	urlStr := r.c.requestUrl("/repositories/%s", ro.Owner)
+	url := "/repositories"
+	if ro.Owner != "" {
+		url += fmt.Sprintf("/%s", ro.Owner)
+	}
+	urlStr := r.c.requestUrl(url)
 	if ro.Role != "" {
 		urlStr += "?role=" + ro.Role
 	}

Original file line number	Diff line number	Diff line change
`@@ -2,6 +2,7 @@ package bitbucket`
`2`	`2`
`3`	`3`	`import (`
`4`	`4`	`"errors"`
	`5`	`+ "fmt"`
`5`	`6`
`6`	`7`	`"github.com/mitchellh/mapstructure"`
`7`	`8`	`)`
`@@ -31,7 +32,11 @@ type RepositoriesRes struct {`
`31`	`32`	`}`
`32`	`33`
`33`	`34`	`func (r Repositories) ListForAccount(ro RepositoriesOptions) (*RepositoriesRes, error) {`
`34`		`- urlStr := r.c.requestUrl("/repositories/%s", ro.Owner)`
	`35`	`+ url := "/repositories"`
	`36`	`+ if ro.Owner != "" {`
	`37`	`+ url += fmt.Sprintf("/%s", ro.Owner)`
	`38`	`+ }`
	`39`	`+ urlStr := r.c.requestUrl(url)`
`35`	`40`	`if ro.Role != "" {`
`36`	`41`	`urlStr += "?role=" + ro.Role`
`37`	`42`	`}`