diff --git a/CoreCms.Net.Services/Good/CoreCmsGoodsServices.cs b/CoreCms.Net.Services/Good/CoreCmsGoodsServices.cs
index 31f0e4e..5e3dac3 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 62ed692..e54880f 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;
                 }