diff options
author | George Goldberg <george@gberg.me> | 2017-04-03 19:32:57 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-04-03 19:32:57 +0100 |
commit | 1bcb5cdb9d6de7690edf271da5eedc6fb74789d9 (patch) | |
tree | 7103c1ebf91cc1c04b82c75432faf7f55f49f8ec /api4/team.go | |
parent | 4ebc85e05c3f45d2978c6fb8f9874b93658c3607 (diff) | |
download | chat-1bcb5cdb9d6de7690edf271da5eedc6fb74789d9.tar.gz chat-1bcb5cdb9d6de7690edf271da5eedc6fb74789d9.tar.bz2 chat-1bcb5cdb9d6de7690edf271da5eedc6fb74789d9.zip |
PLT-6169: Batch add users to teams. (#5953)
Diffstat (limited to 'api4/team.go')
-rw-r--r-- | api4/team.go | 50 |
1 files changed, 50 insertions, 0 deletions
diff --git a/api4/team.go b/api4/team.go index 6ea989a9b..a420e77f4 100644 --- a/api4/team.go +++ b/api4/team.go @@ -15,6 +15,10 @@ import ( "github.com/mattermost/platform/utils" ) +const ( + MAX_ADD_MEMBERS_BATCH = 20 +) + func InitTeam() { l4g.Debug(utils.T("api.team.init.debug")) @@ -33,6 +37,7 @@ func InitTeam() { BaseRoutes.TeamMembers.Handle("/ids", ApiSessionRequired(getTeamMembersByIds)).Methods("POST") BaseRoutes.TeamMembersForUser.Handle("", ApiSessionRequired(getTeamMembersForUser)).Methods("GET") BaseRoutes.TeamMembers.Handle("", ApiSessionRequired(addTeamMember)).Methods("POST") + BaseRoutes.TeamMembers.Handle("/batch", ApiSessionRequired(addTeamMembers)).Methods("POST") BaseRoutes.TeamMember.Handle("", ApiSessionRequired(removeTeamMember)).Methods("DELETE") BaseRoutes.TeamForUser.Handle("/unread", ApiSessionRequired(getTeamUnread)).Methods("GET") @@ -367,6 +372,51 @@ func addTeamMember(c *Context, w http.ResponseWriter, r *http.Request) { w.Write([]byte(member.ToJson())) } +func addTeamMembers(c *Context, w http.ResponseWriter, r *http.Request) { + c.RequireTeamId() + if c.Err != nil { + return + } + + var err *model.AppError + members := model.TeamMembersFromJson(r.Body) + + if len(members) > MAX_ADD_MEMBERS_BATCH || len(members) == 0 { + c.SetInvalidParam("too many members in batch") + return + } + + var userIds []string + for _, member := range members { + if member.TeamId != c.Params.TeamId { + c.SetInvalidParam("team_id for member with user_id=" + member.UserId) + return + } + + if len(member.UserId) != 26 { + c.SetInvalidParam("user_id") + return + } + + userIds = append(userIds, member.UserId) + } + + if !app.SessionHasPermissionToTeam(c.Session, c.Params.TeamId, model.PERMISSION_ADD_USER_TO_TEAM) { + c.SetPermissionError(model.PERMISSION_ADD_USER_TO_TEAM) + return + } + + members, err = app.AddTeamMembers(c.Params.TeamId, userIds, c.GetSiteURL()) + + if err != nil { + c.Err = err + return + } + + w.WriteHeader(http.StatusCreated) + w.Write([]byte(model.TeamMembersToJson(members))) +} + func removeTeamMember(c *Context, w http.ResponseWriter, r *http.Request) { c.RequireTeamId().RequireUserId() if c.Err != nil { |