diff options
author | Saturnino Abril <saturnino.abril@gmail.com> | 2018-02-19 23:47:34 +0800 |
---|---|---|
committer | Saturnino Abril <saturnino.abril@gmail.com> | 2018-02-19 23:47:34 +0800 |
commit | 45d7c51d9f27a60c15e833706cab07dd7e014e2d (patch) | |
tree | 167b3315158be4483495009779649bdbf867972e /app/team.go | |
parent | 30197584d5a215a3b25bffa79a034ed9e360cf52 (diff) | |
parent | a9d4c7d4c1f6398bebba732d07108fe30ddd6319 (diff) | |
download | chat-45d7c51d9f27a60c15e833706cab07dd7e014e2d.tar.gz chat-45d7c51d9f27a60c15e833706cab07dd7e014e2d.tar.bz2 chat-45d7c51d9f27a60c15e833706cab07dd7e014e2d.zip |
merge 4.7.1 to master
Diffstat (limited to 'app/team.go')
-rw-r--r-- | app/team.go | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/app/team.go b/app/team.go index 8e8c29e2a..a15c64c3f 100644 --- a/app/team.go +++ b/app/team.go @@ -302,10 +302,16 @@ func (a *App) joinUserToTeam(team *model.Team, user *model.User) (*model.TeamMem return rtm, true, nil } - if tmr := <-a.Srv.Store.Team().UpdateMember(tm); tmr.Err != nil { - return nil, false, tmr.Err + if membersCount := <-a.Srv.Store.Team().GetActiveMemberCount(tm.TeamId); membersCount.Err != nil { + return nil, false, membersCount.Err + } else if membersCount.Data.(int64) >= int64(*a.Config().TeamSettings.MaxUsersPerTeam) { + return nil, false, model.NewAppError("joinUserToTeam", "app.team.join_user_to_team.max_accounts.app_error", nil, "teamId="+tm.TeamId, http.StatusBadRequest) } else { - return tmr.Data.(*model.TeamMember), false, nil + if tmr := <-a.Srv.Store.Team().UpdateMember(tm); tmr.Err != nil { + return nil, false, tmr.Err + } else { + return tmr.Data.(*model.TeamMember), false, nil + } } } else { // Membership appears to be missing. Lets try to add. |