- more append err msg when ebai api failed.

platformapi/ebaiapi/ebaiapi.go

@@ -129,17 +129,19 @@ func (a *API) AccessAPI(cmd string, body map[string]interface{}) (retVal *Respon
 		})
 	if err != nil {
 		if codeErr, ok := err.(*utils.ErrorWithCode); ok {
-			if body[KeyBaiduShopID] != nil || body[KeyShopID] != nil {
+			appendErrList := []string{}
-				shopInfoList := []string{}
 			for _, key := range []string{
 				KeyBaiduShopID,
 				KeyShopID,
+				KeySkuID,
+				KeyCustomSkuID,
 			} {
 				if body[key] != nil {
-						shopInfoList = append(shopInfoList, fmt.Sprintf("[%s:%v]", key, body[key]))
+					appendErrList = append(appendErrList, fmt.Sprintf("[%s:%v]", key, body[key]))
 				}
 			}
-				err = utils.NewErrorCode(strings.Join(shopInfoList, ",")+", "+codeErr.ErrMsg(), codeErr.Code())
+			if len(appendErrList) > 0 {
+				err = utils.NewErrorCode(strings.Join(appendErrList, ",")+", "+codeErr.ErrMsg(), codeErr.Code())
 			}
 		}
 	}

platformapi/ebaiapi/shop.go

@@ -32,6 +32,9 @@ const (
 	KeyShopID      = "shop_id"
 	KeyBaiduShopID = "baidu_shop_id"
 
+	KeyCustomSkuID = "custom_sku_id"
+	KeySkuID       = "sku_id"
+
 	KeyName  = "name"
 	KeyPhone = "phone"
 )

platformapi/ebaiapi/shop_sku.go

@@ -112,7 +112,7 @@ func (a *API) SkuList(shopID string, params map[string]interface{}) (skuInfo *Pa
 func (a *API) SkuCreate(shopID string, customSkuID int, params map[string]interface{}) (skuID int64, err error) {
 	defParams := map[string]interface{}{
 		KeyShopID:      shopID,
-		"custom_sku_id": customSkuID,
+		KeyCustomSkuID: customSkuID,
 	}
 	if params["upc"] == nil {
 		defParams["upc_type"] = UPCTypePrivate
@@ -128,7 +128,7 @@ func (a *API) SkuCreate(shopID string, customSkuID int, params map[string]interf
 	// baseapi.SugarLogger.Debugf(utils.Format4Output(params, false))
 	result, err := a.AccessAPI("sku.create", params)
 	if err == nil {
-		return utils.Str2Int64(utils.Interface2String(result.Data.(map[string]interface{})["sku_id"])), nil
+		return utils.Str2Int64(utils.Interface2String(result.Data.(map[string]interface{})[KeySkuID])), nil
 	}
 	return 0, err
 }
@@ -136,11 +136,11 @@ func (a *API) SkuCreate(shopID string, customSkuID int, params map[string]interf
 func (a *API) SkuUpdate(shopID string, ebaiSkuID int64, params map[string]interface{}) (skuID int64, err error) {
 	defParams := map[string]interface{}{
 		KeyShopID: shopID,
-		"sku_id":  ebaiSkuID,
+		KeySkuID:  ebaiSkuID,
 	}
 	result, err := a.AccessAPI("sku.update", utils.MergeMaps(params, defParams))
 	if err == nil {
-		return utils.Str2Int64(utils.Interface2String(result.Data.(map[string]interface{})["sku_id"])), nil
+		return utils.Str2Int64(utils.Interface2String(result.Data.(map[string]interface{})[KeySkuID])), nil
 	}
 	return 0, err
 }
@@ -148,7 +148,7 @@ func (a *API) SkuUpdate(shopID string, ebaiSkuID int64, params map[string]interf
 func (a *API) SkuDelete(shopID, skuIDsStr string) (err error) {
 	params := map[string]interface{}{
 		KeyShopID: shopID,
-		"sku_id":  skuIDsStr,
+		KeySkuID:  skuIDsStr,
 	}
 	_, err = a.AccessAPI("sku.delete", params)
 	if err == nil {
@@ -160,7 +160,7 @@ func (a *API) SkuDelete(shopID, skuIDsStr string) (err error) {
 func (a *API) SkuDeleteByCustomIDs(shopID, customSkuIDsStr string) (err error) {
 	params := map[string]interface{}{
 		KeyShopID:      shopID,
-		"custom_sku_id": customSkuIDsStr,
+		KeyCustomSkuID: customSkuIDsStr,
 	}
 	_, err = a.AccessAPI("sku.delete", params)
 	if err == nil {
@@ -172,7 +172,7 @@ func (a *API) SkuDeleteByCustomIDs(shopID, customSkuIDsStr string) (err error) {
 func (a *API) SkuOnline(shopID, skuIDsStr string) (err error) {
 	params := map[string]interface{}{
 		KeyShopID: shopID,
-		"sku_id":  skuIDsStr,
+		KeySkuID:  skuIDsStr,
 	}
 	_, err = a.AccessAPI("sku.online", params)
 	if err == nil {
@@ -184,7 +184,7 @@ func (a *API) SkuOnline(shopID, skuIDsStr string) (err error) {
 func (a *API) SkuOffline(shopID, skuIDsStr string) (err error) {
 	params := map[string]interface{}{
 		KeyShopID: shopID,
-		"sku_id":  skuIDsStr,
+		KeySkuID:  skuIDsStr,
 	}
 	_, err = a.AccessAPI("sku.offline", params)
 	if err == nil {
@@ -196,7 +196,7 @@ func (a *API) SkuOffline(shopID, skuIDsStr string) (err error) {
 func (a *API) SkuShopCategoryMap(shopID string, skuID int64, categoryIDsStr string) (err error) {
 	params := map[string]interface{}{
 		KeyShopID:     shopID,
-		"sku_id":      skuID,
+		KeySkuID:      skuID,
 		"category_id": categoryIDsStr,
 	}
 	_, err = a.AccessAPI("sku.shop.category.map", params)

platformapi/ebaiapi/shop_sku_test.go

@@ -49,7 +49,7 @@ func TestSkuGetItemsByCategoryId(t *testing.T) {
 
 func TestSkuList(t *testing.T) {
 	result, err := api.SkuList(testShopID, map[string]interface{}{
-		// "sku_id": 153879464137191,
+		// KeySkuID: 153879464137191,
 		// "delete": 1,
 	})
 	if err != nil {