Added a unique error message for bad username and simplified error outputed to user
Этот коммит содержится в:
@@ -161,7 +161,11 @@ func (us SqlUserStore) Update(user *model.User, allowActiveUpdate bool) StoreCha
|
|||||||
}
|
}
|
||||||
|
|
||||||
if count, err := us.GetMaster().Update(user); err != nil {
|
if count, err := us.GetMaster().Update(user); err != nil {
|
||||||
result.Err = model.NewAppError("SqlUserStore.Update", "We encounted an error updating the account", "user_id="+user.Id+", "+err.Error())
|
if IsUniqueConstraintError(err.Error(), "Username", "users_username_teamid_key") {
|
||||||
|
result.Err = model.NewAppError("SqlUserStore.Update", "This username is already taken. Please choose another.", "user_id="+user.Id+", "+err.Error())
|
||||||
|
} else {
|
||||||
|
result.Err = model.NewAppError("SqlUserStore.Update", "We encounted an error updating the account", "user_id="+user.Id+", "+err.Error())
|
||||||
|
}
|
||||||
} else if count != 1 {
|
} else if count != 1 {
|
||||||
result.Err = model.NewAppError("SqlUserStore.Update", "We couldn't update the account", fmt.Sprintf("user_id=%v, count=%v", user.Id, count))
|
result.Err = model.NewAppError("SqlUserStore.Update", "We couldn't update the account", fmt.Sprintf("user_id=%v, count=%v", user.Id, count))
|
||||||
} else {
|
} else {
|
||||||
|
|||||||
@@ -709,7 +709,7 @@ var GeneralTab = React.createClass({
|
|||||||
}.bind(this),
|
}.bind(this),
|
||||||
function(err) {
|
function(err) {
|
||||||
state = this.getInitialState();
|
state = this.getInitialState();
|
||||||
state.server_error = err;
|
state.server_error = err.message;
|
||||||
this.setState(state);
|
this.setState(state);
|
||||||
}.bind(this)
|
}.bind(this)
|
||||||
);
|
);
|
||||||
|
|||||||
Ссылка в новой задаче
Block a user