From a13a6b14ec4ab8e9660fd59d7abe00e3d3666f9f Mon Sep 17 00:00:00 2001
From: "toby.zxj" <toby.zxj@gmail.com>
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