diff --git a/CoreCms.Net.Services/Good/CoreCmsGoodsServices.cs b/CoreCms.Net.Services/Good/CoreCmsGoodsServices.cs
index 31f0e4edbf69e51e9b7ab50619f7bb81e3b0aec5..5e3dac383ad6a09379fc6db1c2a292f545a5d27b 100644
--- a/CoreCms.Net.Services/Good/CoreCmsGoodsServices.cs
+++ b/CoreCms.Net.Services/Good/CoreCmsGoodsServices.cs
@@ -452,10 +452,10 @@ namespace CoreCms.Net.Services
                 if (string.IsNullOrEmpty(promotions.parameters)) return false;
                 var obj = JsonConvert.DeserializeAnonymousType(promotions.parameters, new
                 {
-                    goodId = "",
+                    goodsId = "",
                     num = 0,
                 });
-                if (obj.goodId.ObjectToInt(0) != goodId) return false;
+                if (obj.goodsId.ObjectToInt(0) != goodId) return false;
                 promotionsModel = promotions;
                 return true;
             }
diff --git a/CoreCms.Net.Services/Good/CoreCmsProductsServices.cs b/CoreCms.Net.Services/Good/CoreCmsProductsServices.cs
index 62ed6925b70b7ba69da6f63e4e0207d7eb7285c6..e54880f2d184cb6cdcffbe52fabd13eb5a41061a 100644
--- a/CoreCms.Net.Services/Good/CoreCmsProductsServices.cs
+++ b/CoreCms.Net.Services/Good/CoreCmsProductsServices.cs
@@ -276,7 +276,7 @@ namespace CoreCms.Net.Services
             }
             else if (type == "group" || type == "skill")
             {
-                if (!goodsServices.IsInGroup(productModel.goodsId, out var groupModel))
+                if (!goodsServices.IsInGroup(productModel.goodsId, out var groupModel, groupId))
                 {
                     return null;
                 }