111

Mall.Module.TradePavilion/CarrierModule.cs

@@ -266,31 +266,34 @@ namespace Mall.Module.TradePavilion
 
                 };
                 flag = carrierRepository.Update(fileds, new WhereHelper(nameof(RB_Carrier_Extend.ID), model.ID));
+              
+            }
+            else
+            {
+                Id = carrierRepository.Insert(model);
+                model.ID = Id;
+                flag = Id > 0;
+            }
+
             if (flag)
             {
                 var oldCarrierMetroList = carrierMetroRepository.GetCarrierMetroList(new RB_CarrierMetro_Extend { TenantId = model.TenantId, MallBaseId = model.MallBaseId, CarrierId = model.ID });
+                if (oldCarrierMetroList != null && oldCarrierMetroList.Count > 0)
+                {
                     carrierMetroRepository.DeleteBatch(oldCarrierMetroList);
-                    model.CarrierMetroList.ForEach(x => x.CarrierId = model.ID);
-                    model.CarrierMetroList.ForEach(x => x.Status = 0);
-                    model.CarrierMetroList.ForEach(x => x.TenantId = model.TenantId);
-                    model.CarrierMetroList.ForEach(x => x.MallBaseId = model.MallBaseId);
-                    carrierMetroRepository.InsertBatch(model.CarrierMetroList);
-                }
                 }
-            else
+                if (model.CarrierMetroList != null && model.CarrierMetroList.Count > 0)
                 {
-                Id = carrierRepository.Insert(model);
-                if (Id > 0)
+                    foreach (var item in model.CarrierMetroList)
                     {
-                    model.CarrierMetroList.ForEach(x => x.CarrierId = Id);
-                    model.CarrierMetroList.ForEach(x => x.Status = 0);
-                    model.CarrierMetroList.ForEach(x => x.TenantId = model.TenantId);
-                    model.CarrierMetroList.ForEach(x => x.MallBaseId = model.MallBaseId);
+                        item.CarrierId = model.ID;
+                        item.Status = 0;
+                        item.TenantId = model.TenantId;
+                        item.MallBaseId = model.MallBaseId;
+                    }
                     carrierMetroRepository.InsertBatch(model.CarrierMetroList);
                 }
-                flag = Id > 0;
             }
-
             return flag;
         }
 

Mall.WebApi/Controllers/TradePavilion/TradeController.cs

@@ -1688,11 +1688,11 @@ namespace Mall.WebApi.Controllers.TradePavilion
             {
                 query.FirstShopNum = "";
             }
+            query.MallBaseId = RequestParm.MallBaseId;
+            query.TenantId = RequestParm.TenantId;
             if (query.ID == 0)
             {
                 query.CreateDate = System.DateTime.Now;
-                query.MallBaseId = RequestParm.MallBaseId;
-                query.TenantId = RequestParm.TenantId;
             }
             if (!string.IsNullOrWhiteSpace(query.StartingInfo))
             {
@@ -1709,6 +1709,7 @@ namespace Mall.WebApi.Controllers.TradePavilion
             query.EndAreaRequirement ??= 0;
             query.YeJi ??= 0;
             query.UpdateDate = System.DateTime.Now;
+            
             bool flag = carrierModule.SetCarrier(query);
             return flag ? ApiResult.Success() : ApiResult.Failed();
         }