- refactor globals.

controllers/jd_order.go

@@ -2,11 +2,13 @@ package controllers
 
 import (
 	"encoding/json"
+	"net/url"
 
 	"github.com/astaxie/beego/logs"
 
 	"git.rosy.net.cn/baseapi/platform/jdapi"
 	"git.rosy.net.cn/jx-callback/business/jd/controller"
+	"git.rosy.net.cn/jx-callback/globals"
 	"github.com/astaxie/beego"
 )
 
@@ -21,8 +23,15 @@ func (c *JDOrderController) URLMapping() {
 	c.Mapping("NewOrder", c.NewOrder)
 }
 
-func (c *JDOrderController) handleJDCallback(obj interface{}, hanlder func() interface{}) {
+func (c *JDOrderController) handleJDCallback(obj interface{}, needUnescape bool, hanlder func() interface{}) {
 	jdParamJSON := c.GetString(JD_PARAM_JSON)
+	if needUnescape {
+		jdParamJSON2, err := url.QueryUnescape(jdParamJSON)
+		if err != nil {
+			globals.SugarLogger.Warnf("can not escape data:%v, error:%v", jdParamJSON, err)
+		}
+		jdParamJSON = jdParamJSON2
+	}
 	err := json.Unmarshal([]byte(jdParamJSON), obj)
 	if err != nil {
 		logs.Error(err)
@@ -35,7 +44,7 @@ func (c *JDOrderController) handleJDCallback(obj interface{}, hanlder func() int
 
 func (c *JDOrderController) orderStatus() {
 	var ob jdapi.JDOrderMsg
-	c.handleJDCallback(&ob, func() interface{} {
+	c.handleJDCallback(&ob, false, func() interface{} {
 		cc := controller.OrderControler{}
 		return cc.OrderStatus(&ob)
 	})
@@ -139,7 +148,7 @@ func (c *JDOrderController) ApplyCancelOrder() {
 // @router /pushDeliveryStatus [post]
 func (c *JDOrderController) PushDeliveryStatus() {
 	var ob jdapi.JDDeliveryStatusMsg
-	c.handleJDCallback(&ob, func() interface{} {
+	c.handleJDCallback(&ob, true, func() interface{} {
 		cc := controller.OrderControler{}
 		return cc.OrderDeliveryStatus(&ob)
 	})