Return sms error message.

controllers/verification.go

@@ -45,14 +45,14 @@ func (c *ApiController) SendVerificationCode() {
 		return
 	}
 
-	ret := "Invalid dest type."
+	msg := "Invalid dest type."
 	switch destType {
 	case "email":
 		if !util.IsEmailValid(dest) {
 			c.ResponseError("Invalid Email address")
 			return
 		}
-		ret = object.SendVerificationCodeToEmail(remoteAddr, dest)
+		msg = object.SendVerificationCodeToEmail(remoteAddr, dest)
 	case "phone":
 		if !util.IsPhoneCnValid(dest) {
 			c.ResponseError("Invalid phone number")
@@ -64,17 +64,15 @@ func (c *ApiController) SendVerificationCode() {
 			phonePrefix = org.PhonePrefix
 		}
 		dest = fmt.Sprintf("+%s%s", phonePrefix, dest)
-		ret = object.SendVerificationCodeToPhone(remoteAddr, dest)
+		msg = object.SendVerificationCodeToPhone(remoteAddr, dest)
 	}
 
-	var status string
+	status := "ok"
-	if len(ret) == 0 {
+	if msg != "" {
-		status = "ok"
-	} else {
 		status = "error"
 	}
 
-	c.Data["json"] = Response{Status: status, Msg: ret}
+	c.Data["json"] = Response{Status: status, Msg: msg}
 	c.ServeJSON()
 }
 

object/sms.go

@@ -15,8 +15,6 @@
 package object
 
 import (
-	"fmt"
-
 	"github.com/astaxie/beego"
 	"github.com/casdoor/go-sms-sender"
 )
@@ -35,14 +33,14 @@ func InitSmsClient() {
 	client = go_sms_sender.NewSmsClient(provider, accessId, accessKey, sign, region, templateId, appId)
 }
 
-func SendCodeToPhone(phone, code string) {
+func SendCodeToPhone(phone, code string) string {
 	if client == nil {
 		InitSmsClient()
 		if client == nil {
-			fmt.Println("Sms Config Error")
+			return "SMS config error"
-			return
 		}
 	}
+
 	param := make(map[string]string)
 	if provider == "tencent" {
 		param["0"] = code
@@ -50,4 +48,5 @@ func SendCodeToPhone(phone, code string) {
 		param["code"] = code
 	}
 	client.SendMessage(param, phone)
+	return ""
 }

object/verification.go

@@ -51,8 +51,7 @@ func SendVerificationCodeToPhone(remoteAddr, dest string) string {
 		return result
 	}
 
-	SendCodeToPhone(dest, code)
+	return SendCodeToPhone(dest, code)
-	return ""
 }
 
 func AddToVerificationRecord(remoteAddr, dest, code string) string {

web/src/locales/zh.json

@@ -130,9 +130,9 @@
     "Code Sent": "验证码已发送",
     "Input your email": "请输入邮箱",
     "Input your phone number": "输入手机号",
-    "New phone": "新的手机号",
+    "New phone": "新手机号",
-    "New email": "新的邮箱",
+    "New email": "新邮箱",
-    "Code You Received": "你收到的验证码",
+    "Code You Received": "验证码",
     "Enter your code": "输入你的验证码",
     "You can only send one code in 60s.": "每分钟你只能发送一次验证码",
     "Code has not been sent yet!": "你还没有发送验证码",