Add skip and limit to git.GetTags (#16897)

* Make GetTags() api similar to GetBranches()
* Use it for Tag/Release page

2 files changed, 15 insertions, 3 deletions

diff --git a/modules/git/repo_tag_gogit.go b/modules/git/repo_tag_gogit.go
index 3022fe96f7..ff8a6d53ee 100644
--- a/modules/git/repo_tag_gogit.go
+++ b/modules/git/repo_tag_gogit.go
@@ -21,7 +21,8 @@ func (repo *Repository) IsTagExist(name string) bool {
 }
 
 // GetTags returns all tags of the repository.
-func (repo *Repository) GetTags() ([]string, error) {
+// returning at most limit tags, or all if limit is 0.
+func (repo *Repository) GetTags(skip, limit int) ([]string, error) {
 	var tagNames []string
 
 	tags, err := repo.gogitRepo.Tags()
@@ -40,5 +41,15 @@ func (repo *Repository) GetTags() ([]string, error) {
 		tagNames[i], tagNames[j] = tagNames[j], tagNames[i]
 	}
 
+	// since we have to reverse order we can paginate only afterwards
+	if len(tagNames) < skip {
+		tagNames = []string{}
+	} else {
+		tagNames = tagNames[skip:]
+	}
+	if limit != 0 && len(tagNames) > limit {
+		tagNames = tagNames[:limit]
+	}
+
 	return tagNames, nil
 }
diff --git a/modules/git/repo_tag_nogogit.go b/modules/git/repo_tag_nogogit.go
index 0170f0cc76..172b6fd66c 100644
--- a/modules/git/repo_tag_nogogit.go
+++ b/modules/git/repo_tag_nogogit.go
@@ -18,7 +18,8 @@ func (repo *Repository) IsTagExist(name string) bool {
 }
 
 // GetTags returns all tags of the repository.
-func (repo *Repository) GetTags() (tags []string, err error) {
-	tags, _, err = callShowRef(repo.Path, TagPrefix, "--tags", 0, 0)
+// returning at most limit tags, or all if limit is 0.
+func (repo *Repository) GetTags(skip, limit int) (tags []string, err error) {
+	tags, _, err = callShowRef(repo.Path, TagPrefix, "--tags", skip, limit)
 	return
 }