Browse Source

Fix listing team members (#3048)

for-closed-social
Kim Carlbäcker 8 years ago
committed by Unknwon
parent
commit
3c0c7a9f83
1 changed files with 3 additions and 1 deletions
  1. +3
    -1
      models/org.go

+ 3
- 1
models/org.go View File

@ -443,7 +443,7 @@ func RemoveOrgRepo(orgID, repoID int64) error {
// that the user with the given userID has access to. // that the user with the given userID has access to.
func (org *User) GetUserRepositories(userID int64) (err error) { func (org *User) GetUserRepositories(userID int64) (err error) {
teams := make([]*Team, 0, org.NumTeams) 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 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 { WHERE team_user.org_id = ? AND team_user.uid = ?`, org.Id, userID).Find(&teams); err != nil {
return fmt.Errorf("get teams: %v", err) 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) return fmt.Errorf("get teams: %v", err)
} }
org.Teams = teams
// FIXME: should I change this value inside method, // FIXME: should I change this value inside method,
// or only in location of caller where it's really needed? // or only in location of caller where it's really needed?
org.NumTeams = len(org.Teams) org.NumTeams = len(org.Teams)

Loading…
Cancel
Save