From 83427db136cb3f030ab05e8f3b840175a2974087 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E8=8B=8F=E5=B0=B9=E5=B2=9A?= <770236076@qq.com>
Date: Thu, 11 Jun 2020 15:23:09 +0800
Subject: [PATCH] =?UTF-8?q?=E4=BA=AC=E4=B8=9C=E5=95=86=E5=93=81=E5=90=8E?=
 =?UTF-8?q?=E7=BC=80=E4=BF=AE=E6=94=B9?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 business/jxstore/tempop/tempop.go | 15 ++++++---------
 1 file changed, 6 insertions(+), 9 deletions(-)

diff --git a/business/jxstore/tempop/tempop.go b/business/jxstore/tempop/tempop.go
index d8eedbfef..8a05f9ab6 100644
--- a/business/jxstore/tempop/tempop.go
+++ b/business/jxstore/tempop/tempop.go
@@ -1875,14 +1875,8 @@ func UploadJdsImage(ctx *jxcontext.Context) (err error) {
 		storeSku := skus[0]
 		if sku.Comment != "" {
 			var specQuality = utils.Float64ToStr(float64(storeSku.SpecQuality)) + storeSku.SpecUnit
-			flag := false
-			valuesSku, maxNo, _ := api.JdShopAPI.FindValuesByAttrId(utils.Str2Int(vendorSku.SaleAttrs[0].AttrID))
-			for _, vv := range valuesSku {
-				if vv.Name == specQuality {
-					flag = true
-				}
-			}
-			if flag {
+			vendorSkuAttr := vendorSku.SaleAttrs[0].AttrValueAlias[0]
+			if vendorSkuAttr == specQuality {
 				err = api.JdShopAPI.UpdateWareSaleAttrvalueAlias(&jdshopapi.UpdateWareSaleAttrvalueAliasParam{
 					WareID: vendorSku.WareID,
 					Props: []*jdshopapi.CreateSkuParamAttrs2{
@@ -1896,7 +1890,10 @@ func UploadJdsImage(ctx *jxcontext.Context) (err error) {
 					},
 				})
 			} else {
-				api.JdShopAPI.SaveVenderAttrValue(sku.Comment, utils.Str2Int(vendorSku.SaleAttrs[0].AttrID), int(storeSku.VendorVendorCatID), maxNo+1)
+				if vendorSkuAttr != sku.Comment {
+					_, maxNo, _ := api.JdShopAPI.FindValuesByAttrId(utils.Str2Int(vendorSku.SaleAttrs[0].AttrID))
+					api.JdShopAPI.SaveVenderAttrValue(sku.Comment, utils.Str2Int(vendorSku.SaleAttrs[0].AttrID), int(storeSku.VendorVendorCatID), maxNo+1)
+				}
 			}
 		}
 		// attrValue := vendorSku.SaleAttrs[0].AttrValueAlias[0]