Refactor RepoSearch to be easy usable (#346)

Allow EdgeCases

Transform Uggly API to Prity Search Function

Update & add description of SearchRepoOptions

Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>
Co-authored-by: 6543 <6543@obermui.de>
Reviewed-on: https://gitea.com/gitea/go-sdk/pulls/346
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com>
Reviewed-by: lafriks <lafriks@noreply.gitea.io>

gitea/repo.go

@@ -58,6 +58,20 @@ type Repository struct {
 	AvatarURL                 string      `json:"avatar_url"`
 }
 
+// RepoType represent repo type
+type RepoType string
+
+const (
+	// RepoTypeNone dont specify a type
+	RepoTypeNone RepoType = ""
+	// RepoTypeSource is the default repo type
+	RepoTypeSource RepoType = "source"
+	// RepoTypeFork is a repo witch was forked from an other one
+	RepoTypeFork RepoType = "fork"
+	// RepoTypeMirror represents an mirror repo
+	RepoTypeMirror RepoType = "mirror"
+)
+
 // ListReposOptions options for listing repositories
 type ListReposOptions struct {
 	ListOptions
@@ -92,17 +106,52 @@ func (c *Client) ListOrgRepos(org string, opt ListOrgReposOptions) ([]*Repositor
 // SearchRepoOptions options for searching repositories
 type SearchRepoOptions struct {
 	ListOptions
-	Keyword         string
+
-	Topic           bool
+	// The keyword to query
-	IncludeDesc     bool
+	Keyword string
-	UID             int64
+	// Limit search to repositories with keyword as topic
-	PriorityOwnerID int64
+	KeywordIsTopic bool
-	StarredBy       int64
+	// Include search of keyword within repository description
-	Private         bool
+	KeywordInDescription bool
-	Template        bool
+
-	Mode            string
+	/*
-	Exclusive       bool
+		User Filter
-	Sort            string
+	*/
+
+	// Repo Owner
+	OwnerID int64
+	// Stared By UserID
+	StarredByUserID int64
+
+	/*
+		Repo Attributes
+	*/
+
+	// pubic, private or all repositories (defaults to all)
+	IsPrivate *bool
+	// archived, non-archived or all repositories (defaults to all)
+	IsArchived *bool
+	// Exclude template repos from search
+	ExcludeTemplate bool
+	// Filter by "fork", "source", "mirror"
+	Type RepoType
+
+	/*
+		Sort Filters
+	*/
+
+	// sort repos by attribute. Supported values are "alpha", "created", "updated", "size", and "id". Default is "alpha"
+	Sort string
+	// sort order, either "asc" (ascending) or "desc" (descending). Default is "asc", ignored if "sort" is not specified.
+	Order string
+	// Repo owner to prioritize in the results
+	PrioritizedByOwnerID int64
+
+	/*
+		Cover EdgeCases
+	*/
+	// if set all other options are ignored and this string is used as query
+	RawQuery string
 }
 
 // QueryEncode turns options into querystring argument
@@ -111,34 +160,46 @@ func (opt *SearchRepoOptions) QueryEncode() string {
 	if opt.Keyword != "" {
 		query.Add("q", opt.Keyword)
 	}
-
+	if opt.KeywordIsTopic {
-	query.Add("topic", fmt.Sprintf("%t", opt.Topic))
+		query.Add("topic", "true")
-	query.Add("includeDesc", fmt.Sprintf("%t", opt.IncludeDesc))
+	}
-
+	if opt.KeywordInDescription {
-	if opt.UID > 0 {
+		query.Add("includeDesc", "true")
-		query.Add("uid", fmt.Sprintf("%d", opt.UID))
 	}
 
-	if opt.PriorityOwnerID > 0 {
+	// User Filter
-		query.Add("priority_owner_id", fmt.Sprintf("%d", opt.PriorityOwnerID))
+	if opt.OwnerID > 0 {
+		query.Add("uid", fmt.Sprintf("%d", opt.OwnerID))
+		query.Add("exclusive", "true")
+	}
+	if opt.StarredByUserID > 0 {
+		query.Add("starredBy", fmt.Sprintf("%d", opt.StarredByUserID))
 	}
 
-	if opt.StarredBy > 0 {
+	// Repo Attributes
-		query.Add("starredBy", fmt.Sprintf("%d", opt.StarredBy))
+	if opt.IsPrivate != nil {
+		query.Add("is_private", fmt.Sprintf("%v", opt.IsPrivate))
+	}
+	if opt.IsArchived != nil {
+		query.Add("archived", fmt.Sprintf("%v", opt.IsArchived))
+	}
+	if opt.ExcludeTemplate {
+		query.Add("template", "false")
+	}
+	if len(opt.Type) != 0 {
+		query.Add("mode", string(opt.Type))
 	}
 
-	query.Add("private", fmt.Sprintf("%t", opt.Private))
+	// Sort Filters
-	query.Add("template", fmt.Sprintf("%t", opt.Template))
-
-	if opt.Mode != "" {
-		query.Add("mode", opt.Mode)
-	}
-
-	query.Add("exclusive", fmt.Sprintf("%t", opt.Exclusive))
-
 	if opt.Sort != "" {
 		query.Add("sort", opt.Sort)
 	}
+	if opt.PrioritizedByOwnerID > 0 {
+		query.Add("priority_owner_id", fmt.Sprintf("%d", opt.PrioritizedByOwnerID))
+	}
+	if opt.Order != "" {
+		query.Add("order", opt.Order)
+	}
 
 	return query.Encode()
 }
@@ -153,7 +214,20 @@ func (c *Client) SearchRepos(opt SearchRepoOptions) ([]*Repository, error) {
 	resp := new(searchRepoResponse)
 
 	link, _ := url.Parse("/repos/search")
-	link.RawQuery = opt.QueryEncode()
+
+	if len(opt.RawQuery) != 0 {
+		link.RawQuery = opt.RawQuery
+	} else {
+		link.RawQuery = opt.QueryEncode()
+		// IsPrivate only works on gitea >= 1.12.0
+		if err := c.CheckServerVersionConstraint(">=1.12.0"); err != nil && opt.IsPrivate != nil {
+			if *opt.IsPrivate {
+				// private repos only not supported on gitea <= 1.11.x
+				return nil, err
+			}
+			link.Query().Add("private", "false")
+		}
+	}
 
 	err := c.getParsedResponse("GET", link.String(), nil, nil, &resp)
 	return resp.Repos, err

gitea/repo_test.go

@@ -45,32 +45,32 @@ func TestSearchRepo(t *testing.T) {
 	assert.NoError(t, c.AddRepoTopic(repo.Owner.UserName, repo.Name, "TestTopic1"))
 
 	repos, err := c.SearchRepos(SearchRepoOptions{
-		Keyword:     "Search1",
+		Keyword:              "Search1",
-		IncludeDesc: true,
+		KeywordInDescription: true,
 	})
 	assert.NoError(t, err)
 	assert.NotNil(t, repos)
 	assert.Len(t, repos, 1)
 
 	repos, err = c.SearchRepos(SearchRepoOptions{
-		Keyword:     "Search",
+		Keyword:              "Search",
-		IncludeDesc: true,
+		KeywordInDescription: true,
 	})
 	assert.NoError(t, err)
 	assert.NotNil(t, repos)
 	assert.Len(t, repos, 2)
 
 	repos, err = c.SearchRepos(SearchRepoOptions{
-		Keyword: "TestTopic1",
+		Keyword:              "TestTopic1",
-		Topic:   true,
+		KeywordInDescription: true,
 	})
 	assert.NoError(t, err)
 	assert.NotNil(t, repos)
 	assert.Len(t, repos, 2)
 
 	repos, err = c.SearchRepos(SearchRepoOptions{
-		Keyword: "TestTopic2",
+		Keyword:              "TestTopic2",
-		Topic:   true,
+		KeywordInDescription: true,
 	})
 	assert.NoError(t, err)
 	assert.NotNil(t, repos)