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]