From a13a6b14ec4ab8e9660fd59d7abe00e3d3666f9f Mon Sep 17 00:00:00 2001 From: "toby.zxj" Date: Wed, 11 Jun 2014 16:54:25 +0800 Subject: [PATCH] Using strings.HasPrefix(...) will misjudgement `strings.HasPrefix(access.RepoName, uname)` can't handle the situation which like following in access table. user_name | repo_name ----------+------------- toby | toby/blog toby | toby/test toby | tobyzxj/blog toby | tobyzxj/test --- models/repo.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/models/repo.go b/models/repo.go index 3eca78c4b9..7eab83c217 100644 --- a/models/repo.go +++ b/models/repo.go @@ -831,11 +831,11 @@ func GetCollaborativeRepos(uname string) ([]*Repository, error) { repos := make([]*Repository, 0, 10) for _, access := range accesses { - if strings.HasPrefix(access.RepoName, uname) { + infos := strings.Split(access.RepoName, "/") + if infos[0] == uname { continue } - - infos := strings.Split(access.RepoName, "/") + u, err := GetUserByName(infos[0]) if err != nil { return nil, err