fix: some bugs about SMS API (#310)

Signed-off-by: “seriouszyx” <674965440@qq.com>

controllers/service.go

@@ -96,17 +96,21 @@ func (c *ApiController) SendSms() {
 	var smsForm struct {
 		Content   string   `json:"content"`
 		Receivers []string `json:"receivers"`
+		OrgId     string   `json:"organizationId"` // e.g. "admin/built-in"
 	}
 	err := json.Unmarshal(c.Ctx.Input.RequestBody, &smsForm)
 	if err != nil {
 		c.ResponseError(err.Error())
 		return
 	}
 
+	org := object.GetOrganization(smsForm.OrgId)
 	var invalidReceivers []string
-	for _, receiver := range smsForm.Receivers {
+	for idx, receiver := range smsForm.Receivers {
 		if !util.IsPhoneCnValid(receiver) {
 			invalidReceivers = append(invalidReceivers, receiver)
+		} else {
+			smsForm.Receivers[idx] = fmt.Sprintf("+%s%s", org.PhonePrefix, receiver)
 		}
 	}
 

controllers/verification.go

@@ -63,7 +63,7 @@ func (c *ApiController) SendVerificationCode() {
 	user := c.getCurrentUser()
 	organization := object.GetOrganization(orgId)
 	application := object.GetApplicationByOrganizationName(organization.Name)
-	
+
 	sendResp := errors.New("Invalid dest type.")
 	switch destType {
 	case "email":
@@ -90,12 +90,12 @@ func (c *ApiController) SendVerificationCode() {
 		sendResp = object.SendVerificationCodeToPhone(organization, user, provider, remoteAddr, dest)
 	}
 
-	status := "ok"
 	if sendResp != nil {
-		status = "error"
+		c.Data["json"] = Response{Status: "error", Msg: sendResp.Error()}
+	} else {
+		c.Data["json"] = Response{Status: "ok"}
 	}
 
-	c.Data["json"] = Response{Status: status, Msg: sendResp.Error()}
 	c.ServeJSON()
 }
 

object/verification.go

@@ -68,7 +68,7 @@ func SendVerificationCodeToPhone(organization *Organization, user *User, provide
 		return err
 	}
 
-	return SendSms(provider, dest, code)
+	return SendSms(provider, code, dest)
 }
 
 func AddToVerificationRecord(user *User, provider *Provider, remoteAddr, recordType, dest, code string) error {