diff options
| author | Unknwon <u@gogs.io> | 2017-02-24 13:56:02 -0500 |
|---|---|---|
| committer | Unknwon <u@gogs.io> | 2017-02-24 13:56:02 -0500 |
| commit | 10b93efc4a3061d01b75b44b91a617698971e38b (patch) | |
| tree | de3d093876a0de391de72f081a7b15afa6fd27cc /models/repo_collaboration.go | |
| parent | 0696d430c9baa409297dc06ffa0ab84c0ea44a29 (diff) | |
repo: able to add organization member as repository collaborator
Diffstat (limited to 'models/repo_collaboration.go')
| -rw-r--r-- | models/repo_collaboration.go | 28 |
1 files changed, 17 insertions, 11 deletions
diff --git a/models/repo_collaboration.go b/models/repo_collaboration.go index 889ff4ce..2ab52c37 100644 --- a/models/repo_collaboration.go +++ b/models/repo_collaboration.go @@ -62,17 +62,10 @@ func (repo *Repository) AddCollaborator(u *User) error { return err } - if _, err = sess.InsertOne(collaboration); err != nil { + if _, err = sess.Insert(collaboration); err != nil { return err - } - - if repo.Owner.IsOrganization() { - err = repo.recalculateTeamAccesses(sess, 0) - } else { - err = repo.recalculateAccesses(sess) - } - if err != nil { - return fmt.Errorf("recalculateAccesses 'team=%v': %v", repo.Owner.IsOrganization(), err) + } else if err = repo.recalculateAccesses(sess); err != nil { + return fmt.Errorf("recalculateAccesses [repo_id: %v]: %v", repo.ID, err) } return sess.Commit() @@ -148,6 +141,19 @@ func (repo *Repository) ChangeCollaborationAccessMode(userID int64, mode AccessM } collaboration.Mode = mode + // If it's an organizational repository, merge with team access level for highest permission + if repo.Owner.IsOrganization() { + teams, err := GetUserTeams(repo.OwnerID, userID) + if err != nil { + return fmt.Errorf("GetUserTeams: [org_id: %d, user_id: %d]: %v", repo.OwnerID, userID, err) + } + for i := range teams { + if mode < teams[i].Authorize { + mode = teams[i].Authorize + } + } + } + sess := x.NewSession() defer sessionRelease(sess) if err = sess.Begin(); err != nil { @@ -173,7 +179,7 @@ func (repo *Repository) ChangeCollaborationAccessMode(userID int64, mode AccessM _, err = sess.Insert(access) } if err != nil { - return fmt.Errorf("update access table: %v", err) + return fmt.Errorf("update/insert access table: %v", err) } return sess.Commit() |