egtyl/forgejo
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Using strings.HasPrefix(...) will misjudgement

Browse source 
`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
This commit is contained in:
toby.zxj 2014-06-11 16:54:25 +08:00
parent a3e1383cac
commit a13a6b14ec
1 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
models/repo.go
View file

@ -831,11 +831,11 @@ func GetCollaborativeRepos(uname string) ([]*Repository, error) {
repos := make([]*Repository, 0, 10) repos := make([]*Repository, 0, 10)
for _, access := range accesses { for _, access := range accesses {
if strings.HasPrefix(access.RepoName, uname) { infos := strings.Split(access.RepoName, "/")
if infos[0] == uname {
continue continue
} }
infos := strings.Split(access.RepoName, "/")
u, err := GetUserByName(infos[0]) u, err := GetUserByName(infos[0])
if err != nil { if err != nil {
return nil, err return nil, err