From 8c9f3126af1a7a6f7cc2cff51449da6f77f56e01 Mon Sep 17 00:00:00 2001
From: zhangqian <zhangqian@123.com>
Date: 星期五, 18 八月 2023 17:36:11 +0800
Subject: [PATCH] fix

---
 service/salesDetails.go |   39 ++++++++++++++++++++++++---------------
 1 files changed, 24 insertions(+), 15 deletions(-)

diff --git a/service/salesDetails.go b/service/salesDetails.go
index a2e5ad8..73ff94b 100644
--- a/service/salesDetails.go
+++ b/service/salesDetails.go
@@ -53,7 +53,7 @@
 
 func (SalesDetailsService) UpdateSalesDetails(salesDetails *model.SalesDetails) int {
 	// check salesDetails exist
-	old, err := model.NewSalesDetailsSearch().SetId(salesDetails.Id).First()
+	old, err := model.NewSalesDetailsSearch().SetId(salesDetails.Id).SetPreload(true).First()
 	if err != nil {
 		return ecode.SalesDetailsNotExist
 	}
@@ -77,22 +77,31 @@
 		if err != nil {
 			return err
 		}
-		err = model.NewProductSearch(db).SetIds(removedProductIds).Delete()
-		if err != nil {
-			return err
+		if len(removedProductIds) > 0 {
+			err = model.NewProductSearch(db).SetIds(removedProductIds).Delete()
+			if err != nil {
+				return err
+			}
 		}
-		err = model.NewProductSearch(db).CreateBatch(newProducts)
-		if err != nil {
-			return err
+		if len(newProducts) > 0 {
+			err = model.NewProductSearch(db).CreateBatch(newProducts)
+			if err != nil {
+				return err
+			}
+			var rel []*model.SalesDetailsProduct
+			for _, p := range newProducts {
+				rel = append(rel, &model.SalesDetailsProduct{
+					SalesDetailsId: salesDetails.Id,
+					ProductId:      p.Id,
+				})
+			}
+			err = model.NewSalesDetailsProductSearch().CreateBatch(rel)
+			if err != nil {
+				return err
+			}
 		}
-		var rel []*model.SalesDetailsProduct
-		for _, p := range newProducts {
-			rel = append(rel, &model.SalesDetailsProduct{
-				SalesDetailsId: salesDetails.Id,
-				ProductId:      p.Id,
-			})
-		}
-		return model.NewSalesDetailsProductSearch().CreateBatch(rel)
+		return nil
+
 	})
 
 	if err != nil {

--
Gitblit v1.8.0