From 8224f03a23751fb83d4d57058925dac694e8753c Mon Sep 17 00:00:00 2001 From: Jimmy Praet Date: Wed, 6 Jan 2021 06:07:58 +0100 Subject: [PATCH] Check permission for the appropriate unit type (#14261) * Check permission for the appropriate unit type Fixes false positives for "User created Issues in Repository which they no longer have access to" errors * change log level to Debug Co-authored-by: Lunny Xiao --- routers/user/home.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/routers/user/home.go b/routers/user/home.go index 952f5cdb3..2b59b971a 100644 --- a/routers/user/home.go +++ b/routers/user/home.go @@ -551,8 +551,8 @@ func Issues(ctx *context.Context) { ctx.ServerError("GetUserRepoPermission", fmt.Errorf("[%d]%v", repoID, err)) return } - if !perm.CanRead(models.UnitTypeIssues) { - log.Error("User created Issues in Repository which they no longer have access to: [%d]", repoID) + if !perm.CanRead(unitType) { + log.Debug("User created Issues in Repository which they no longer have access to: [%d]", repoID) } } }