From 92c26cea4de00349b346f71da3cf530b26c31eb0 Mon Sep 17 00:00:00 2001
From: Administrator <15274802129@163.com>
Date: Tue, 10 Jun 2025 09:40:59 +0800
Subject: [PATCH] fix(mall): 修复用户重复加入标签的问题

---
 src/main/java/cc/mrbird/febs/mall/service/impl/MallMemberCollectionServiceImpl.java |   30 ++++++++++++++++++++++--------
 1 files changed, 22 insertions(+), 8 deletions(-)

diff --git a/src/main/java/cc/mrbird/febs/mall/service/impl/MallMemberCollectionServiceImpl.java b/src/main/java/cc/mrbird/febs/mall/service/impl/MallMemberCollectionServiceImpl.java
index bba034f..748f162 100644
--- a/src/main/java/cc/mrbird/febs/mall/service/impl/MallMemberCollectionServiceImpl.java
+++ b/src/main/java/cc/mrbird/febs/mall/service/impl/MallMemberCollectionServiceImpl.java
@@ -1,8 +1,10 @@
 package cc.mrbird.febs.mall.service.impl;
 
+import cc.mrbird.febs.common.entity.FebsResponse;
 import cc.mrbird.febs.common.exception.FebsException;
 import cc.mrbird.febs.common.utils.LoginUserUtil;
 import cc.mrbird.febs.mall.dto.AddCollectionDto;
+import cc.mrbird.febs.mall.dto.ApiColletDelDto;
 import cc.mrbird.febs.mall.dto.MemberCollectionListDto;
 import cc.mrbird.febs.mall.entity.MallGoods;
 import cc.mrbird.febs.mall.entity.MallMember;
@@ -47,15 +49,12 @@
         queryWrapper.eq(MallMemberCollection::getMemberId, member.getId());
         queryWrapper.eq(MallMemberCollection::getGoodsId, addCollectionDto.getGoodsId());
         List<MallMemberCollection> mallMemberCollections = this.baseMapper.selectList(queryWrapper);
-        if (CollUtil.isNotEmpty(mallMemberCollections)) {
-            throw new FebsException("商品已收藏");
+        if (CollUtil.isEmpty(mallMemberCollections)) {
+            MallMemberCollection collection = new MallMemberCollection();
+            collection.setMemberId(member.getId());
+            collection.setGoodsId(addCollectionDto.getGoodsId());
+            this.baseMapper.insert(collection);
         }
-
-        MallMemberCollection collection = new MallMemberCollection();
-        collection.setMemberId(member.getId());
-        collection.setGoodsId(addCollectionDto.getGoodsId());
-
-        this.baseMapper.insert(collection);
     }
 
     @Override
@@ -95,4 +94,19 @@
         delQuery.eq(MallMemberCollection::getMemberId, member.getId());
         this.baseMapper.delete(delQuery);
     }
+
+    @Override
+    public FebsResponse delCollection(ApiColletDelDto dto) {
+        MallMember member = LoginUserUtil.getLoginUser();
+        List<Long> ids = dto.getIds();
+        if (CollUtil.isEmpty(ids)) {
+            return new FebsResponse().success().message("操作成功");
+        }
+
+        LambdaQueryWrapper<MallMemberCollection> delQuery = new LambdaQueryWrapper<>();
+        delQuery.in(MallMemberCollection::getId, ids);
+        delQuery.eq(MallMemberCollection::getMemberId, member.getId());
+        this.baseMapper.delete(delQuery);
+        return new FebsResponse().success().message("操作成功");
+    }
 }

--
Gitblit v1.9.1