From 3c0c7a9f83effc7415a500309c6a27648bfb7005 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Kim=20Carlb=C3=A4cker?= Date: Sat, 7 May 2016 02:02:36 +0200 Subject: [PATCH] Fix listing team members (#3048) --- models/org.go | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/models/org.go b/models/org.go index 540498d1c0..bac3ad25ca 100644 --- a/models/org.go +++ b/models/org.go @@ -443,7 +443,7 @@ func RemoveOrgRepo(orgID, repoID int64) error { // that the user with the given userID has access to. func (org *User) GetUserRepositories(userID int64) (err error) { teams := make([]*Team, 0, org.NumTeams) - if err = x.Sql(`SELECT team.id FROM team + if err = x.Sql(`SELECT team.id FROM team INNER JOIN team_user ON team_user.team_id = team.id WHERE team_user.org_id = ? AND team_user.uid = ?`, org.Id, userID).Find(&teams); err != nil { return fmt.Errorf("get teams: %v", err) @@ -484,6 +484,8 @@ WHERE team_user.org_id = ? AND team_user.uid = ?`, return fmt.Errorf("get teams: %v", err) } + org.Teams = teams + // FIXME: should I change this value inside method, // or only in location of caller where it's really needed? org.NumTeams = len(org.Teams)