MM-2056 fixes problem with creating team

This commit is contained in:
=Corey Hulen
2015-08-31 13:33:22 -07:00
parent 82a54f9363
commit 7de7de6ad0
10 changed files with 12 additions and 18 deletions

View File

@@ -36,7 +36,7 @@ func InitTeam(r *mux.Router) {
}
func signupTeam(c *Context, w http.ResponseWriter, r *http.Request) {
if !utils.Cfg.ServiceSettings.AllowEmailSignUp {
if utils.Cfg.ServiceSettings.DisableEmailSignUp {
c.Err = model.NewAppError("signupTeam", "Team sign-up with email is disabled.", "")
c.Err.StatusCode = http.StatusNotImplemented
return
@@ -139,7 +139,7 @@ func createTeamFromSSO(c *Context, w http.ResponseWriter, r *http.Request) {
}
func createTeamFromSignup(c *Context, w http.ResponseWriter, r *http.Request) {
if !utils.Cfg.ServiceSettings.AllowEmailSignUp {
if utils.Cfg.ServiceSettings.DisableEmailSignUp {
c.Err = model.NewAppError("createTeamFromSignup", "Team sign-up with email is disabled.", "")
c.Err.StatusCode = http.StatusNotImplemented
return
@@ -239,7 +239,7 @@ func createTeamFromSignup(c *Context, w http.ResponseWriter, r *http.Request) {
}
func createTeam(c *Context, w http.ResponseWriter, r *http.Request) {
if !utils.Cfg.ServiceSettings.AllowEmailSignUp {
if utils.Cfg.ServiceSettings.DisableEmailSignUp {
c.Err = model.NewAppError("createTeam", "Team sign-up with email is disabled.", "")
c.Err.StatusCode = http.StatusNotImplemented
return

View File

@@ -58,7 +58,7 @@ func InitUser(r *mux.Router) {
}
func createUser(c *Context, w http.ResponseWriter, r *http.Request) {
if !utils.Cfg.ServiceSettings.AllowEmailSignUp {
if utils.Cfg.ServiceSettings.DisableEmailSignUp {
c.Err = model.NewAppError("signupTeam", "User sign-up with email is disabled.", "")
c.Err.StatusCode = http.StatusNotImplemented
return

View File

@@ -68,12 +68,6 @@ func TestCreateUser(t *testing.T) {
}
}
user2 := model.User{TeamId: rteam.Data.(*model.Team).Id, Email: strings.ToLower(model.NewId()) + "corey@test.com", Nickname: "Corey Hulen", Password: "hello", Username: model.BOT_USERNAME}
if _, err := Client.CreateUser(&user2, ""); err == nil {
t.Fatal("Should have failed using reserved bot name")
}
if _, err := Client.DoPost("/users/create", "garbage"); err == nil {
t.Fatal("should have been an error")
}

View File

@@ -23,7 +23,7 @@
"UseLocalStorage": true,
"StorageDirectory": "./data/",
"AllowedLoginAttempts": 10,
"AllowEmailSignUp": true
"DisableEmailSignUp": false
},
"SSOSettings": {
"gitlab": {

View File

@@ -23,7 +23,7 @@
"UseLocalStorage": true,
"StorageDirectory": "/mattermost/data/",
"AllowedLoginAttempts": 10,
"AllowEmailSignUp": true
"DisableEmailSignUp": false
},
"SSOSettings": {
"gitlab": {

View File

@@ -23,7 +23,7 @@
"UseLocalStorage": true,
"StorageDirectory": "/mattermost/data/",
"AllowedLoginAttempts": 10,
"AllowEmailSignUp": true
"DisableEmailSignUp": false
},
"SSOSettings": {
"gitlab": {

View File

@@ -23,7 +23,7 @@
"UseLocalStorage": true,
"StorageDirectory": "/mattermost/data/",
"AllowedLoginAttempts": 10,
"AllowEmailSignUp": true
"DisableEmailSignUp": false
},
"SSOSettings": {
"gitlab": {

View File

@@ -333,7 +333,6 @@ func IsUsernameValid(username string) bool {
var validUsernameChars = regexp.MustCompile(`^[a-z0-9\.\-_]+$`)
var restrictedUsernames = []string{
BOT_USERNAME,
"all",
"channel",
}

View File

@@ -31,7 +31,7 @@ type ServiceSettings struct {
UseLocalStorage bool
StorageDirectory string
AllowedLoginAttempts int
AllowEmailSignUp bool
DisableEmailSignUp bool
}
type SSOSetting struct {
@@ -278,7 +278,7 @@ func GetAllowedAuthServices() []string {
}
}
if Cfg.ServiceSettings.AllowEmailSignUp {
if !Cfg.ServiceSettings.DisableEmailSignUp {
authServices = append(authServices, "email")
}

View File

@@ -57,13 +57,14 @@ module.exports = React.createClass({
window.location.href = '/verify_email?email=' + encodeURIComponent(teamSignup.team.email) + '&teamname=' + encodeURIComponent(teamSignup.team.name);
} else {
this.setState({serverError: err.message});
$('#finish-button').button('reset');
}
}.bind(this)
);
}.bind(this),
function error(err) {
this.setState({serverError: err.message});
$('#sign-up-button').button('reset');
$('#finish-button').button('reset');
}.bind(this)
);
},