From bedd7b2833349c151eeb21bec90d5cec0453613a Mon Sep 17 00:00:00 2001 From: guillep2k <18600385+guillep2k@users.noreply.github.com> Date: Fri, 3 Jan 2020 18:39:12 -0300 Subject: [PATCH] Fix error logged when repos qs is empty (#9591) * Fix error logged when repos qs is empty * Update routers/user/home.go Co-Authored-By: Lauris BH Co-authored-by: techknowlogick Co-authored-by: Lauris BH --- routers/user/home.go | 26 ++++++++++++++------------ 1 file changed, 14 insertions(+), 12 deletions(-) diff --git a/routers/user/home.go b/routers/user/home.go index ae5975a711..f5e74b2406 100644 --- a/routers/user/home.go +++ b/routers/user/home.go @@ -389,21 +389,23 @@ func Issues(ctx *context.Context) { reposQuery := ctx.Query("repos") var repoIDs []int64 - if issueReposQueryPattern.MatchString(reposQuery) { - // remove "[" and "]" from string - reposQuery = reposQuery[1 : len(reposQuery)-1] - //for each ID (delimiter ",") add to int to repoIDs - for _, rID := range strings.Split(reposQuery, ",") { - // Ensure nonempty string entries - if rID != "" && rID != "0" { - rIDint64, err := strconv.ParseInt(rID, 10, 64) - if err == nil { - repoIDs = append(repoIDs, rIDint64) + if len(reposQuery) != 0 { + if issueReposQueryPattern.MatchString(reposQuery) { + // remove "[" and "]" from string + reposQuery = reposQuery[1 : len(reposQuery)-1] + //for each ID (delimiter ",") add to int to repoIDs + for _, rID := range strings.Split(reposQuery, ",") { + // Ensure nonempty string entries + if rID != "" && rID != "0" { + rIDint64, err := strconv.ParseInt(rID, 10, 64) + if err == nil { + repoIDs = append(repoIDs, rIDint64) + } } } + } else { + log.Error("issueReposQueryPattern not match with query") } - } else { - log.Error("issueReposQueryPattern not match with query") } isShowClosed := ctx.Query("state") == "closed"