Skip to content

Commit 7092402

Browse files
authored
Add missing repository type filter parameters to pager (#31832)
Fix #31807 ps: the newly added params's value will be changed. When the first time you selected the filter, the values of params will be `0` or `1` But in pager it will be `true` or `false`. So do we have `boolToInt` function?
1 parent b491b21 commit 7092402

File tree

4 files changed

+61
-0
lines changed

4 files changed

+61
-0
lines changed

routers/web/explore/repo.go

+15
Original file line numberDiff line numberDiff line change
@@ -144,6 +144,21 @@ func RenderRepoSearch(ctx *context.Context, opts *RepoSearchOptions) {
144144
pager.AddParamString("topic", fmt.Sprint(topicOnly))
145145
pager.AddParamString("language", language)
146146
pager.AddParamString(relevantReposOnlyParam, fmt.Sprint(opts.OnlyShowRelevant))
147+
if archived.Has() {
148+
pager.AddParamString("archived", fmt.Sprint(archived.Value()))
149+
}
150+
if fork.Has() {
151+
pager.AddParamString("fork", fmt.Sprint(fork.Value()))
152+
}
153+
if mirror.Has() {
154+
pager.AddParamString("mirror", fmt.Sprint(mirror.Value()))
155+
}
156+
if template.Has() {
157+
pager.AddParamString("template", fmt.Sprint(template.Value()))
158+
}
159+
if private.Has() {
160+
pager.AddParamString("private", fmt.Sprint(private.Value()))
161+
}
147162
ctx.Data["Page"] = pager
148163

149164
ctx.HTML(http.StatusOK, opts.TplName)

routers/web/org/home.go

+16
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
package org
55

66
import (
7+
"fmt"
78
"net/http"
89
"path"
910
"strings"
@@ -155,6 +156,21 @@ func Home(ctx *context.Context) {
155156
pager := context.NewPagination(int(count), setting.UI.User.RepoPagingNum, page, 5)
156157
pager.SetDefaultParams(ctx)
157158
pager.AddParamString("language", language)
159+
if archived.Has() {
160+
pager.AddParamString("archived", fmt.Sprint(archived.Value()))
161+
}
162+
if fork.Has() {
163+
pager.AddParamString("fork", fmt.Sprint(fork.Value()))
164+
}
165+
if mirror.Has() {
166+
pager.AddParamString("mirror", fmt.Sprint(mirror.Value()))
167+
}
168+
if template.Has() {
169+
pager.AddParamString("template", fmt.Sprint(template.Value()))
170+
}
171+
if private.Has() {
172+
pager.AddParamString("private", fmt.Sprint(private.Value()))
173+
}
158174
ctx.Data["Page"] = pager
159175

160176
ctx.Data["ShowMemberAndTeamTab"] = ctx.Org.IsMember || len(members) > 0

routers/web/user/notification.go

+15
Original file line numberDiff line numberDiff line change
@@ -446,6 +446,21 @@ func NotificationWatching(ctx *context.Context) {
446446
// redirect to last page if request page is more than total pages
447447
pager := context.NewPagination(total, setting.UI.User.RepoPagingNum, page, 5)
448448
pager.SetDefaultParams(ctx)
449+
if archived.Has() {
450+
pager.AddParamString("archived", fmt.Sprint(archived.Value()))
451+
}
452+
if fork.Has() {
453+
pager.AddParamString("fork", fmt.Sprint(fork.Value()))
454+
}
455+
if mirror.Has() {
456+
pager.AddParamString("mirror", fmt.Sprint(mirror.Value()))
457+
}
458+
if template.Has() {
459+
pager.AddParamString("template", fmt.Sprint(template.Value()))
460+
}
461+
if private.Has() {
462+
pager.AddParamString("private", fmt.Sprint(private.Value()))
463+
}
449464
ctx.Data["Page"] = pager
450465

451466
ctx.Data["Status"] = 2

routers/web/user/profile.go

+15
Original file line numberDiff line numberDiff line change
@@ -333,6 +333,21 @@ func prepareUserProfileTabData(ctx *context.Context, showPrivate bool, profileDb
333333
pager.AddParamString("date", fmt.Sprint(ctx.Data["Date"]))
334334
}
335335
}
336+
if archived.Has() {
337+
pager.AddParamString("archived", fmt.Sprint(archived.Value()))
338+
}
339+
if fork.Has() {
340+
pager.AddParamString("fork", fmt.Sprint(fork.Value()))
341+
}
342+
if mirror.Has() {
343+
pager.AddParamString("mirror", fmt.Sprint(mirror.Value()))
344+
}
345+
if template.Has() {
346+
pager.AddParamString("template", fmt.Sprint(template.Value()))
347+
}
348+
if private.Has() {
349+
pager.AddParamString("private", fmt.Sprint(private.Value()))
350+
}
336351
ctx.Data["Page"] = pager
337352
}
338353

0 commit comments

Comments
 (0)