diff options
author | Corey Hulen <corey@hulen.com> | 2015-06-29 21:16:07 -0800 |
---|---|---|
committer | Corey Hulen <corey@hulen.com> | 2015-06-29 21:16:07 -0800 |
commit | 4d2ca8881bf1aac6bc393988901b4ef523a5ef14 (patch) | |
tree | ba2e8ac114866477d53060d67913e5ca9dfec64d /api/user.go | |
parent | e6b5bdef82b23d61286ff455ee245a26a40d195f (diff) | |
parent | ba998db0998e7782842e4f86a242edd1e52b5e97 (diff) | |
download | chat-4d2ca8881bf1aac6bc393988901b4ef523a5ef14.tar.gz chat-4d2ca8881bf1aac6bc393988901b4ef523a5ef14.tar.bz2 chat-4d2ca8881bf1aac6bc393988901b4ef523a5ef14.zip |
Merge pull request #98 from mattermost/mm-1199
fixes mm-1199 adds off-topic as a default channel
Diffstat (limited to 'api/user.go')
-rw-r--r-- | api/user.go | 15 |
1 files changed, 5 insertions, 10 deletions
diff --git a/api/user.go b/api/user.go index f8382cf2f..292d2b61b 100644 --- a/api/user.go +++ b/api/user.go @@ -176,21 +176,16 @@ func CreateUser(c *Context, team *model.Team, user *model.User) *model.User { } else { ruser := result.Data.(*model.User) - // Do not error if user cannot be added to the town-square channel - if cresult := <-Srv.Store.Channel().GetByName(team.Id, "town-square"); cresult.Err != nil { - l4g.Error("Failed to get town-square err=%v", cresult.Err) - } else { - cm := &model.ChannelMember{ChannelId: cresult.Data.(*model.Channel).Id, UserId: ruser.Id, NotifyLevel: model.CHANNEL_NOTIFY_ALL, Roles: channelRole} - if cmresult := <-Srv.Store.Channel().SaveMember(cm); cmresult.Err != nil { - l4g.Error("Failed to add member town-square err=%v", cmresult.Err) - } + // Soft error if there is an issue joining the default channels + if err := JoinDefaultChannels(c, ruser, channelRole); err != nil { + l4g.Error("Encountered an issue joining default channels user_id=%s, team_id=%s, err=%v", ruser.Id, ruser.TeamId, err) } //fireAndForgetWelcomeEmail(strings.Split(ruser.FullName, " ")[0], ruser.Email, team.Name, c.TeamUrl+"/channels/town-square") if user.EmailVerified { if cresult := <-Srv.Store.User().VerifyEmail(ruser.Id); cresult.Err != nil { - l4g.Error("Failed to get town-square err=%v", cresult.Err) + l4g.Error("Failed to set email verified err=%v", cresult.Err) } } else { FireAndForgetVerifyEmail(result.Data.(*model.User).Id, strings.Split(ruser.FullName, " ")[0], ruser.Email, team.Name, c.TeamUrl) @@ -198,7 +193,7 @@ func CreateUser(c *Context, team *model.Team, user *model.User) *model.User { ruser.Sanitize(map[string]bool{}) - //This message goes to every channel, so the channelId is irrelevant + // This message goes to every channel, so the channelId is irrelevant message := model.NewMessage(team.Id, "", ruser.Id, model.ACTION_NEW_USER) store.PublishAndForget(message) |