Remove user's PhonePrefix.

This commit is contained in:
Yang Luo 2021-05-15 13:54:23 +08:00
parent 43db2de8d8
commit 2ea5e7ec78
3 changed files with 9 additions and 9 deletions

View File

@ -37,7 +37,6 @@ type RequestForm struct {
Password string `json:"password"`
Name string `json:"name"`
Email string `json:"email"`
PhonePrefix string `json:"phonePrefix"`
Phone string `json:"phone"`
Affiliation string `json:"affiliation"`
@ -87,7 +86,7 @@ func (c *ApiController) Signup() {
}
userId := fmt.Sprintf("%s/%s", form.Organization, form.Username)
msg := object.CheckUserSignup(form.Organization, form.Username, form.Password, form.Name, form.Email, form.PhonePrefix, form.Phone, form.Affiliation)
msg := object.CheckUserSignup(form.Organization, form.Username, form.Password, form.Name, form.Email, form.Phone, form.Affiliation)
if msg != "" {
resp = Response{Status: "error", Msg: msg, Data: ""}
} else {

View File

@ -27,20 +27,22 @@ func init() {
reWhiteSpace, _ = regexp.Compile("\\s")
}
func CheckUserSignup(organization string, username string, password string, displayName string, email string, phonePrefix string, phone string, affiliation string) string {
func CheckUserSignup(organizationName string, username string, password string, displayName string, email string, phone string, affiliation string) string {
organization := getOrganization("admin", organizationName)
if len(username) == 0 {
return "username cannot be blank"
} else if len(password) == 0 {
return "password cannot be blank"
} else if getOrganization("admin", organization) == nil {
} else if organization == nil {
return "organization does not exist"
} else if reWhiteSpace.MatchString(username) {
return "username cannot contain white spaces"
} else if HasUserByField(organization, "name", username) {
} else if HasUserByField(organizationName, "name", username) {
return "username already exists"
} else if HasUserByField(organization, "email", email) {
} else if HasUserByField(organizationName, "email", email) {
return "email already exists"
} else if HasUserByField(organization, "phone", phone) {
} else if HasUserByField(organizationName, "phone", phone) {
return "phone already exists"
} else if displayName == "" {
return "displayName cannot be blank"
@ -48,7 +50,7 @@ func CheckUserSignup(organization string, username string, password string, disp
return "affiliation cannot be blank"
} else if !util.IsEmailValid(email) {
return "email is invalid"
} else if phonePrefix == "86" && !util.IsPhoneCnValid(phone) {
} else if organization.PhonePrefix == "86" && !util.IsPhoneCnValid(phone) {
return "phone number is invalid"
} else {
return ""

View File

@ -45,7 +45,6 @@ func createUserFromOriginalUser(originalUser *User) *object.User {
DisplayName: originalUser.Name,
Avatar: getFullAvatarUrl(originalUser.Avatar),
Email: "",
PhonePrefix: "86",
Phone: originalUser.Cellphone,
Affiliation: "",
IsAdmin: false,