diff --git a/business/jxstore/tempop/tempop.go b/business/jxstore/tempop/tempop.go
index 339602576..802d591b6 100644
--- a/business/jxstore/tempop/tempop.go
+++ b/business/jxstore/tempop/tempop.go
@@ -1873,7 +1873,7 @@ func UploadJdsImage(ctx *jxcontext.Context) (err error) {
 		err = dao.GetRow(db, &sku, sql, sqlParams)
 		skus, _ := dao.GetStoreSkus(db, model.VendorIDJDShop, model.JdShopMainStoreID, []int{sku.ID})
 		storeSku := skus[0]
-		if storeSku.Comment != "" {
+		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))
@@ -1882,7 +1882,7 @@ func UploadJdsImage(ctx *jxcontext.Context) (err error) {
 					flag = true
 				}
 			}
-			if !flag {
+			if flag {
 				err = api.JdShopAPI.UpdateWareSaleAttrvalueAlias(&jdshopapi.UpdateWareSaleAttrvalueAliasParam{
 					WareID: vendorSku.WareID,
 					Props: []*jdshopapi.CreateSkuParamAttrs2{
@@ -1894,7 +1894,7 @@ func UploadJdsImage(ctx *jxcontext.Context) (err error) {
 					},
 				})
 			} else {
-				api.JdShopAPI.SaveVenderAttrValue(specQuality, utils.Str2Int(vendorSku.SaleAttrs[0].AttrID), int(storeSku.VendorVendorCatID), maxNo+1)
+				api.JdShopAPI.SaveVenderAttrValue(sku.Comment, utils.Str2Int(vendorSku.SaleAttrs[0].AttrID), int(storeSku.VendorVendorCatID), maxNo+1)
 			}
 		}
 		// attrValue := vendorSku.SaleAttrs[0].AttrValueAlias[0]