diff --git a/litemall-admin-api/src/main/java/org/linlinjava/litemall/admin/service/AdminGoodsService.java b/litemall-admin-api/src/main/java/org/linlinjava/litemall/admin/service/AdminGoodsService.java
index 1ebee041..4007d735 100644
--- a/litemall-admin-api/src/main/java/org/linlinjava/litemall/admin/service/AdminGoodsService.java
+++ b/litemall-admin-api/src/main/java/org/linlinjava/litemall/admin/service/AdminGoodsService.java
@@ -187,7 +187,7 @@ public class AdminGoodsService {
                 attributeService.add(attribute);
             }
             else if(attribute.getDeleted()){
-                attributeService.deleteByGid(attribute.getId());
+                attributeService.deleteById(attribute.getId());
             }
             else if(attribute.getUpdateTime() == null){
                 attributeService.updateById(attribute);
diff --git a/litemall-db/src/main/java/org/linlinjava/litemall/db/service/LitemallGoodsAttributeService.java b/litemall-db/src/main/java/org/linlinjava/litemall/db/service/LitemallGoodsAttributeService.java
index d36f00ea..61e85310 100644
--- a/litemall-db/src/main/java/org/linlinjava/litemall/db/service/LitemallGoodsAttributeService.java
+++ b/litemall-db/src/main/java/org/linlinjava/litemall/db/service/LitemallGoodsAttributeService.java
@@ -36,6 +36,10 @@ public class LitemallGoodsAttributeService {
         goodsAttributeMapper.logicalDeleteByExample(example);
     }
 
+    public void deleteById(Integer id) {
+        goodsAttributeMapper.logicalDeleteByPrimaryKey(id);
+    }
+
     public void updateById(LitemallGoodsAttribute attribute) {
         attribute.setUpdateTime(LocalDateTime.now());
         goodsAttributeMapper.updateByPrimaryKeySelective(attribute);