Merge remote-tracking branch 'origin/mark' into don

business/jxstore/cms/sku.go

@@ -579,18 +579,18 @@ func AddSkuName(ctx *jxcontext.Context, skuNameExt *model.SkuNameExt, userName s
 		// 	return nil, err
 		// }
 		// skuNameExt.ImgEbai = imgHintMap[model.VendorIDEBAI]
-		dataRes, err2 := datares.TryRegisterDataResource(ctx, skuNameExt.Name, skuNameExt.Img, true, model.ImgTypeMain, false)
+		dataRes, err2 := datares.TryRegisterDataResource(ctx, skuNameExt.Name, skuNameExt.Img, model.ImgTypeMain, false)
 		if err = err2; err != nil {
 			return nil, err
 		}
 		// skuNameExt.ImgHashCode = dataRes.HashCode
 		skuNameExt.ImgEbai = dataRes.EbaiURL
 		if skuNameExt.Img2 != "" {
-			datares.TryRegisterDataResource(ctx, skuNameExt.Name, skuNameExt.Img2, true, model.ImgTypeMain, true)
+			datares.TryRegisterDataResource(ctx, skuNameExt.Name, skuNameExt.Img2, model.ImgTypeMain, true)
 		}
 
 		if skuNameExt.DescImg != "" {
-			dataRes, err2 := datares.TryRegisterDataResource(ctx, skuNameExt.Name+"desc", skuNameExt.DescImg, true, model.ImgTypeDesc, false)
+			dataRes, err2 := datares.TryRegisterDataResource(ctx, skuNameExt.Name+"desc", skuNameExt.DescImg, model.ImgTypeDesc, false)
 			if err = err2; err != nil {
 				return nil, err
 			}
@@ -697,7 +697,7 @@ func UpdateSkuName(ctx *jxcontext.Context, nameID int, payload map[string]interf
 				// }
 				// // valid["ImgWeimob"] = imgHintMap[model.VendorIDWSC]
 				// valid["ImgEbai"] = imgHintMap[model.VendorIDEBAI]
-				dataRes, err2 := datares.TryRegisterDataResource(ctx, skuName.Name, valid["img"].(string), true, model.ImgTypeMain, false)
+				dataRes, err2 := datares.TryRegisterDataResource(ctx, skuName.Name, valid["img"].(string), model.ImgTypeMain, false)
 				if err = err2; err != nil {
 					return 0, err
 				}
@@ -705,13 +705,13 @@ func UpdateSkuName(ctx *jxcontext.Context, nameID int, payload map[string]interf
 				valid["ImgEbai"] = dataRes.EbaiURL
 			}
 			if valid["img2"] != nil {
-				datares.TryRegisterDataResource(ctx, skuName.Name, valid["img2"].(string), true, model.ImgTypeMain, true)
+				datares.TryRegisterDataResource(ctx, skuName.Name, valid["img2"].(string), model.ImgTypeMain, true)
 			}
 
 			if valid["descImg"] != nil {
 				descImg := valid["descImg"].(string)
 				if descImg != "" {
-					dataRes, err2 := datares.TryRegisterDataResource(ctx, skuName.Name+"_desc", descImg, true, model.ImgTypeDesc, false)
+					dataRes, err2 := datares.TryRegisterDataResource(ctx, skuName.Name+"_desc", descImg, model.ImgTypeDesc, false)
 					if err = err2; err != nil {
 						return 0, err
 					}

business/jxstore/cms/store.go

@@ -524,6 +524,20 @@ func checkCreateStore(store *model.Store) (err error) {
 	return err
 }
 
+func checkStoreDeliveryRange(deliveryRange string) (err error) {
+	if deliveryRange != "" {
+		points := jxutils.CoordinateStr2Points(deliveryRange)
+		for _, point := range points {
+			for _, coord := range point {
+				if coord == 0 || math.IsNaN(coord) {
+					return fmt.Errorf("坐标点数据非法:%s", deliveryRange)
+				}
+			}
+		}
+	}
+	return err
+}
+
 func UpdateStore(ctx *jxcontext.Context, storeID int, payload map[string]interface{}, userName string) (num int64, err error) {
 	globals.SugarLogger.Debugf("UpdateStore storeID:%d, payload:%s", storeID, utils.Format4Output(payload, true))
 	if err = checkBankBranch(utils.Interface2String(payload["payeeBankBranchName"])); err != nil {
@@ -537,6 +551,9 @@ func UpdateStore(ctx *jxcontext.Context, storeID int, payload map[string]interfa
 	}
 
 	valid := dao.StrictMakeMapByStructObject(payload, store, userName)
+	if err = checkStoreDeliveryRange(utils.Interface2String(valid["deliveryRange"])); err != nil {
+		return 0, err
+	}
 	if globals.EnableWXAuth2 {
 		if err = dao.ValidateRoles(db, utils.Interface2String(valid["marketManRole"]), utils.Interface2String(valid["operatorRole"]), utils.Interface2String(valid["operatorRole2"])); err != nil {
 			return 0, err

business/jxstore/cms/store_sku.go

@@ -175,16 +175,55 @@ var (
 	}
 )
 
-func GetStoreSkus(ctx *jxcontext.Context, storeID int, skuIDs []int, isFocus bool, keyword string, isBySku bool, params map[string]interface{}, offset, pageSize int) (skuNamesInfo *StoreSkuNamesInfo, err error) {
-	return GetStoresSkus(ctx, []int{storeID}, skuIDs, isFocus, keyword, isBySku, params, offset, pageSize)
+func GetStoreSkus(ctx *jxcontext.Context, storeID int, skuIDs []int, isFocus bool, keyword string, isBySku, isAct bool, params map[string]interface{}, offset, pageSize int) (skuNamesInfo *StoreSkuNamesInfo, err error) {
+	return GetStoresSkus(ctx, []int{storeID}, skuIDs, isFocus, keyword, isBySku, isAct, params, offset, pageSize)
 }
 
-func getGetStoresSkusBaseSQL(db *dao.DaoDB, storeIDs, skuIDs []int, isFocus bool, keyword string, isBySku bool, params map[string]interface{}) (sql string, sqlParams []interface{}, err error) {
+func getGetStoresSkusBaseSQL(db *dao.DaoDB, storeIDs, skuIDs []int, isFocus bool, keyword string, isBySku, isAct bool, params map[string]interface{}) (sql string, sqlParams []interface{}, err error) {
 	sql = `
 	FROM sku_name t1
 	JOIN sku t2 FORCE INDEX(PRIMARY) ON t1.id = t2.name_id AND t2.deleted_at = ?/* AND t2.status = ?*/
 	JOIN store t3 ON t3.deleted_at = ?
 	`
+	sqlParams = []interface{}{
+		utils.DefaultTimeValue,
+		// model.SkuStatusNormal,
+		utils.DefaultTimeValue,
+	}
+	if isAct {
+		sql += `
+		JOIN (
+			SELECT t2.store_id, t2.sku_id,
+			MIN(IF(t3.actual_act_price = 0, NULL, t3.actual_act_price)) actual_act_price, MIN(IF(t2.earning_price = 0, NULL, t2.earning_price)) earning_price /*non-zero min value*/
+			FROM act t1
+			JOIN act_store_sku t2 ON t2.act_id = t1.id AND t2.deleted_at = ?
+			JOIN act_store_sku_map t3 ON t3.bind_id = t2.id AND t3.act_id = t1.id AND (t3.sync_status & ? = 0 OR t1.type = ?)
+			JOIN act_map t4 ON t4.act_id = t1.id AND t4.vendor_id = t3.vendor_id AND t4.deleted_at = ? AND (t4.sync_status & ? = 0 OR t1.type = ?)
+			WHERE t1.deleted_at = ? AND t1.status = ? AND NOT (t1.begin_at > ? OR t1.end_at < ?)`
+		sqlParams = append(sqlParams, []interface{}{
+			utils.DefaultTimeValue,
+			model.SyncFlagNewMask,
+			model.ActSkuFake,
+			utils.DefaultTimeValue,
+			model.SyncFlagNewMask,
+			model.ActSkuFake,
+			utils.DefaultTimeValue,
+			model.ActStatusCreated,
+			time.Now(),
+			time.Now(),
+		})
+		if len(storeIDs) > 0 {
+			sql += " AND t2.store_id IN (" + dao.GenQuestionMarks(len(storeIDs)) + ")"
+			sqlParams = append(sqlParams, storeIDs)
+		}
+		if len(skuIDs) > 0 {
+			sql += " AND t2.sku_id IN (" + dao.GenQuestionMarks(len(skuIDs)) + ")"
+			sqlParams = append(sqlParams, skuIDs)
+		}
+		sql += `
+			GROUP BY 1,2
+		) ta ON ta.store_id = t3.id AND ta.sku_id = t2.id`
+	}
 	if !isFocus {
 		sql += " LEFT"
 	}
@@ -193,15 +232,12 @@ func getGetStoresSkusBaseSQL(db *dao.DaoDB, storeIDs, skuIDs []int, isFocus bool
 	LEFT JOIN sku_name_place_bind t5 ON t1.id = t5.name_id AND t3.city_code = t5.place_code
 	WHERE t1.deleted_at = ? AND (t1.is_global = 1 OR t5.id IS NOT NULL OR 1 = ?)/* AND t1.status = ?*/
 	`
-	sqlParams = []interface{}{
-		utils.DefaultTimeValue,
-		// model.SkuStatusNormal,
-		utils.DefaultTimeValue,
+	sqlParams = append(sqlParams, []interface{}{
 		utils.DefaultTimeValue,
 		utils.DefaultTimeValue,
 		utils.Bool2Int(isFocus),
 		// model.SkuStatusNormal,
-	}
+	})
 	if isFocus {
 		sql += " 					  AND ((t2.status = ? AND t1.status = ?) OR t4.status = ?)"
 		sqlParams = append(sqlParams, model.SkuStatusNormal, model.SkuStatusNormal, model.SkuStatusNormal)
@@ -318,11 +354,11 @@ func getGetStoresSkusBaseSQL(db *dao.DaoDB, storeIDs, skuIDs []int, isFocus bool
 	return sql, sqlParams, err
 }
 
-func GetStoresSkus(ctx *jxcontext.Context, storeIDs, skuIDs []int, isFocus bool, keyword string, isBySku bool, params map[string]interface{}, offset, pageSize int) (skuNamesInfo *StoreSkuNamesInfo, err error) {
-	return GetStoresSkusNew(ctx, storeIDs, skuIDs, isFocus, keyword, isBySku, params, offset, pageSize)
+func GetStoresSkus(ctx *jxcontext.Context, storeIDs, skuIDs []int, isFocus bool, keyword string, isBySku, isAct bool, params map[string]interface{}, offset, pageSize int) (skuNamesInfo *StoreSkuNamesInfo, err error) {
+	return GetStoresSkusNew(ctx, storeIDs, skuIDs, isFocus, keyword, isBySku, isAct, params, offset, pageSize)
 }
 
-func GetStoresSkusNew(ctx *jxcontext.Context, storeIDs, skuIDs []int, isFocus bool, keyword string, isBySku bool, params map[string]interface{}, offset, pageSize int) (skuNamesInfo *StoreSkuNamesInfo, err error) {
+func GetStoresSkusNew(ctx *jxcontext.Context, storeIDs, skuIDs []int, isFocus bool, keyword string, isBySku, isAct bool, params map[string]interface{}, offset, pageSize int) (skuNamesInfo *StoreSkuNamesInfo, err error) {
 	if !isFocus && !isBySku && (len(storeIDs) > 1 || len(storeIDs) == 0) {
 		return nil, fmt.Errorf("未关注按SkuName只能查询单店")
 	}
@@ -330,7 +366,7 @@ func GetStoresSkusNew(ctx *jxcontext.Context, storeIDs, skuIDs []int, isFocus bo
 		return nil, fmt.Errorf("GetStoresSkus必须指定storeIDs或skuIDs或分页")
 	}
 	db := dao.GetDB()
-	sql, sqlParams, err := getGetStoresSkusBaseSQL(db, storeIDs, skuIDs, isFocus, keyword, isBySku, params)
+	sql, sqlParams, err := getGetStoresSkusBaseSQL(db, storeIDs, skuIDs, isFocus, keyword, isBySku, isAct, params)
 	if err != nil {
 		return nil, err
 	}

business/jxstore/cms/store_sku_check.go

@@ -330,7 +330,7 @@ func CheckSkuDiffBetweenJxAndVendor(ctx *jxcontext.Context, vendorIDList []int,
 
 							if isGetJxSkuInfoData == false { //only get once jx sku info list every store id
 								isGetJxSkuInfoData = true
-								jxSkuInfoData, _ := GetStoreSkus(ctx, storeID, []int{}, true, "", true, map[string]interface{}{}, 0, -1)
+								jxSkuInfoData, _ := GetStoreSkus(ctx, storeID, []int{}, true, "", true, false, map[string]interface{}{}, 0, -1)
 								filterJxSkuInfoMap = GetFilterJxSkuInfoMap(jxSkuInfoData.SkuNames)
 							}
 

business/jxstore/initdata/initdata.go

@@ -309,7 +309,7 @@ func UploadImg4Vendors(ctx *jxcontext.Context, isAsync, isContinueWhenError bool
 			case 1: // 从SkuName添加缺失的图片至DataResource
 				_, err = dao.ExecuteSQL(db, `
 				INSERT INTO data_resource(created_at, updated_at, last_operator, hash_code,
-					resource_type, name, main_url, ebai_url, qiniu_url, is_vendor)
+					resource_type, name, main_url, ebai_url, qiniu_url, use_type)
 				SELECT t1.created_at, t1.created_at, t1.last_operator, t1.img_hash_code,
 				CASE
 					WHEN INSTR(t1.desc_img, ".jpg") > 0 OR INSTR(t1.desc_img, ".jpeg") > 0 THEN
@@ -324,6 +324,12 @@ func UploadImg4Vendors(ctx *jxcontext.Context, isAsync, isContinueWhenError bool
 				CONCAT(t1.name, '_desc'), desc_img main_url, t1.desc_img_ebai ebai_url,
 				IF(INSTR(t1.desc_img, "image.jxc4.com") > 0, t1.desc_img, '') qiniu_url, 2
 				FROM sku_name t1
+				JOIN (
+					SELECT img_hash_code, MAX(id) id, COUNT(*) ct
+					FROM sku_name
+					WHERE img_hash_code <> '' AND desc_img <> ''
+					GROUP BY 1
+				) t3 ON t3.id = t1.id
 				LEFT JOIN data_resource t2 ON (t2.main_url <> '' AND t2.main_url = t1.desc_img)
 				WHERE t1.desc_img <> '' AND t1.img_hash_code <> '' AND t2.id IS NULL;
 				`)

business/jxstore/misc/store_score.go

@@ -220,7 +220,7 @@ func GetAllStoreSkus(ctx *jxcontext.Context, parentTask tasksch.ITask, storeList
 	taskFunc := func(task *tasksch.ParallelTask, batchItemList []interface{}, params ...interface{}) (retVal interface{}, err error) {
 		storeInfo := batchItemList[0].(*cms.StoreExt)
 		storeID := storeInfo.ID
-		jxSkuInfoData, _ := cms.GetStoreSkus(ctx, storeID, []int{}, true, "", true, map[string]interface{}{}, 0, -1)
+		jxSkuInfoData, _ := cms.GetStoreSkus(ctx, storeID, []int{}, true, "", true, false,map[string]interface{}{}, 0, -1)
 		jxSkuPriceMapData := make(map[int]int)
 		for _, value := range jxSkuInfoData.SkuNames {
 			for _, skuInfo := range value.Skus2 {

business/jxstore/misc/store_sku_sales.go

@@ -75,7 +75,7 @@ func GetStoreSkuSalesInfo(ctx *jxcontext.Context, storeID int) (outStoreSkuSales
 
 	//得到当前门店商品数据
 	storeSkuMapData := make(map[int]*cms.StoreSkuNameExt)
-	storeSkuData, err := cms.GetStoreSkus(ctx, storeID, citySkuIDs, true, "", true, map[string]interface{}{}, 0, -1)
+	storeSkuData, err := cms.GetStoreSkus(ctx, storeID, citySkuIDs, true, "", true, false, map[string]interface{}{}, 0, -1)
 	if err == nil {
 		for _, value := range storeSkuData.SkuNames {
 			for _, skuInfo := range value.Skus2 {
@@ -95,7 +95,7 @@ func GetStoreSkuSalesInfo(ctx *jxcontext.Context, storeID int) (outStoreSkuSales
 	taskFunc := func(task *tasksch.ParallelTask, batchItemList []interface{}, params ...interface{}) (retVal interface{}, err error) {
 		storeInfo := batchItemList[0].(*cms.StoreExt)
 		storeID := storeInfo.ID
-		jxSkuInfoData, err := cms.GetStoreSkus(ctx, storeID, citySkuIDs, true, "", true, map[string]interface{}{}, 0, -1)
+		jxSkuInfoData, err := cms.GetStoreSkus(ctx, storeID, citySkuIDs, true, "", true, false, map[string]interface{}{}, 0, -1)
 		jxSkuPriceMapData := make(map[int]int)
 		for _, value := range jxSkuInfoData.SkuNames {
 			for _, skuInfo := range value.Skus2 {

business/jxstore/tempop/tempop.go

@@ -1,7 +1,12 @@
 package tempop
 
 import (
+	"bytes"
 	"fmt"
+	"image"
+	"image/jpeg"
+	"image/png"
+	"net/http"
 	"regexp"
 	"sync"
 	"time"
@@ -1103,3 +1108,93 @@ func CreateConsumerFromOrders(ctx *jxcontext.Context, vendorIDs []int, fromDate,
 	}
 	return hint, err
 }
+
+func CheckImages(ctx *jxcontext.Context, isAsync, isContinueWhenError bool) (hint string, err error) {
+	db := dao.GetDB()
+	rootTask := tasksch.NewSeqTask("检查图片的有效性", ctx,
+		func(task *tasksch.SeqTask, step int, params ...interface{}) (result interface{}, err error) {
+			switch step {
+			case 0: // 检查图片的格式与大小
+				var imgList []*model.DataResource
+				if err = dao.GetRows(db, &imgList, `
+				SELECT t1.*
+				FROM data_resource t1
+				WHERE t1.use_type = 1
+				`); err == nil && len(imgList) > 0 {
+					calcTask := tasksch.NewParallelTask("检查京西图片的有效性",
+						tasksch.NewParallelConfig().SetIsContinueWhenError(isContinueWhenError).SetParallelCount(5), ctx,
+						func(task *tasksch.ParallelTask, batchItemList []interface{}, params ...interface{}) (retVal interface{}, err error) {
+							dataRes := batchItemList[0].(*model.DataResource)
+							binaryData, hashCode, err2 := jxutils.DownloadFileByURL(dataRes.MainURL)
+							if err = err2; err == nil {
+								errMsg := ""
+								if hashCode != dataRes.HashCode {
+									errMsg = fmt.Sprintf("hashcode:%s", hashCode)
+									dataRes.HashCode = hashCode
+								}
+								resType := http.DetectContentType(binaryData)
+								if resType != dataRes.ResourceType {
+									errMsg += "," + resType
+									dataRes.ResourceType = resType
+								}
+								var img image.Image
+								if dataRes.ResourceType == "image/png" {
+									img, err = png.Decode(bytes.NewReader(binaryData))
+								} else if dataRes.ResourceType == "image/jpeg" {
+									img, err = jpeg.Decode(bytes.NewReader(binaryData))
+								}
+								if err == nil {
+									if img != nil {
+										if img.Bounds().Dx() != 800 || img.Bounds().Dy() != 800 {
+											errMsg += fmt.Sprintf(",size:%dx%d", img.Bounds().Dx(), img.Bounds().Dy())
+										}
+									}
+								} else {
+									errMsg += "," + err.Error()
+								}
+								if errMsg != "" {
+									dataRes.Remark = errMsg
+									dao.UpdateEntity(dao.GetDB(), dataRes, "Remark", "ResourceType", "HashCode")
+								}
+							}
+							return retVal, err
+						}, imgList)
+					tasksch.HandleTask(calcTask, task, false).Run()
+					_, err = calcTask.GetResult(0)
+				}
+			case 1: // 检查京东商品图片
+				skuNameInfo, err2 := cms.GetSkuNames(ctx, "", false, map[string]interface{}{}, 0, -1)
+				if err = err2; err == nil && len(skuNameInfo.SkuNames) > 0 {
+					calcTask := tasksch.NewParallelTask("检查京东图片的有效性",
+						tasksch.NewParallelConfig().SetIsContinueWhenError(isContinueWhenError).SetParallelCount(5), ctx,
+						func(task *tasksch.ParallelTask, batchItemList []interface{}, params ...interface{}) (retVal interface{}, err error) {
+							skuName := batchItemList[0].(*model.SkuNameExt)
+							for _, sku := range skuName.Skus {
+								if sku.JdID != 0 {
+									skuPageInfo, err2 := api.JdAPI.GetSkuPageInfo(sku.JdID)
+									if err = err2; err == nil && skuPageInfo.FixedStatus && len(skuPageInfo.Image) == 0 {
+										err = fmt.Errorf("skuName:%d, sku:%d, jdid:%d, 没有图片", skuName.ID, sku.ID, sku.JdID)
+										skuName.ImgHashCode = "np"
+										dao.UpdateEntity(dao.GetDB(), &skuName.SkuName, "ImgHashCode")
+									}
+									break
+								}
+							}
+							return retVal, err
+						}, skuNameInfo.SkuNames)
+					tasksch.HandleTask(calcTask, task, false).Run()
+					_, err = calcTask.GetResult(0)
+				}
+			}
+			return result, err
+		}, 2)
+	tasksch.HandleTask(rootTask, nil, true).Run()
+	if !isAsync {
+		if _, err = rootTask.GetResult(0); err == nil {
+			hint = "1"
+		}
+	} else {
+		hint = rootTask.ID
+	}
+	return hint, err
+}