From b559abbaeb69d8b163c01b4a05e8655c44b4c8e4 Mon Sep 17 00:00:00 2001
From: DengMin <16679070171@qq.com>
Date: Fri, 5 Aug 2022 14:22:07 +0800
Subject: [PATCH] 内容视频关系接口修改

---
 src/main/java/com/subsidy/controller/ContentVodMappingController.java    |  6 +++---
 src/main/java/com/subsidy/service/ContentVodMappingService.java          |  2 +-
 src/main/java/com/subsidy/service/impl/ContentVodMappingServiceImpl.java | 41 +++++++++++++++++++++--------------------
 3 files changed, 25 insertions(+), 24 deletions(-)

diff --git a/src/main/java/com/subsidy/controller/ContentVodMappingController.java b/src/main/java/com/subsidy/controller/ContentVodMappingController.java
index ce06b3f..db806d5 100644
--- a/src/main/java/com/subsidy/controller/ContentVodMappingController.java
+++ b/src/main/java/com/subsidy/controller/ContentVodMappingController.java
@@ -63,10 +63,10 @@ public class ContentVodMappingController {
     }
 
     @PostMapping("updateContentVod")
-    @ApiOperation("编辑课程内容下的视频： contentId/内容ID、vodIds [vodId, vodAliasName]")
+    @ApiOperation("编辑课程内容下的视频： contentId/内容ID、vodId/视频ID, vodAliasName/视频别名")
     @LoginRequired
-    public ResponseVO updateContentVod(@RequestBody AddContentVodDTO addContentVod) {
-        contentVodMappingService.updateContentVod(addContentVod);
+    public ResponseVO updateContentVod(@RequestBody ContentVodDTO contentVodDTO) {
+        contentVodMappingService.updateContentVod(contentVodDTO);
         return ResponseData.generateCreatedResponse(0);
     }
 
diff --git a/src/main/java/com/subsidy/service/ContentVodMappingService.java b/src/main/java/com/subsidy/service/ContentVodMappingService.java
index fd1a6f4..634e99c 100644
--- a/src/main/java/com/subsidy/service/ContentVodMappingService.java
+++ b/src/main/java/com/subsidy/service/ContentVodMappingService.java
@@ -28,5 +28,5 @@ public interface ContentVodMappingService extends IService<ContentVodMappingDO> 
 
     void addContentVod(AddContentVodDTO addContentVod);
 
-    void updateContentVod(AddContentVodDTO addContentVod);
+    void updateContentVod(ContentVodDTO contentVodDTO);
 }
diff --git a/src/main/java/com/subsidy/service/impl/ContentVodMappingServiceImpl.java b/src/main/java/com/subsidy/service/impl/ContentVodMappingServiceImpl.java
index 3c0c438..ea14c0f 100644
--- a/src/main/java/com/subsidy/service/impl/ContentVodMappingServiceImpl.java
+++ b/src/main/java/com/subsidy/service/impl/ContentVodMappingServiceImpl.java
@@ -77,31 +77,32 @@ public class ContentVodMappingServiceImpl extends ServiceImpl<ContentVodMappingM
             }
 
             for (int i = 0; i < addContentVod.getVodIds().size(); i++) {
-                ContentVodMappingDO contentVodMappingDO = new ContentVodMappingDO();
-                contentVodMappingDO.setContentId(addContentVod.getContentId());
-                contentVodMappingDO.setVodId(addContentVod.getVodIds().get(i).getVodId());
-                contentVodMappingDO.setVodAliasName(addContentVod.getVodIds().get(i).getVodAliasName());
-                contentVodMappingDO.setOrderNo(i+1);
-                this.baseMapper.insert(contentVodMappingDO);
+                ContentVodMappingDO cvm =  this.baseMapper.selectOne(new QueryWrapper<ContentVodMappingDO>().lambda()
+                        .eq(ContentVodMappingDO::getVodId, addContentVod.getVodIds().get(i).getVodId())
+                        .eq(ContentVodMappingDO::getContentId, addContentVod.getContentId()));
+                if(cvm == null) {
+                    ContentVodMappingDO contentVodMappingDO = new ContentVodMappingDO();
+                    contentVodMappingDO.setContentId(addContentVod.getContentId());
+                    contentVodMappingDO.setVodId(addContentVod.getVodIds().get(i).getVodId());
+                    contentVodMappingDO.setVodAliasName(addContentVod.getVodIds().get(i).getVodAliasName());
+                    contentVodMappingDO.setOrderNo(i+1);
+                    this.baseMapper.insert(contentVodMappingDO);
+                }
             }
         }
     }
 
     @Override
-    public void updateContentVod(AddContentVodDTO addContentVodDTO) {
-        if(CollectionUtils.isNotEmpty(addContentVodDTO.getVodIds())) {
-            for (ContentVodDTO contentVodDTO : addContentVodDTO.getVodIds()) {
-                ContentVodMappingDO contentVodMappingDO = this.baseMapper.selectOne(new QueryWrapper<ContentVodMappingDO>()
-                        .lambda()
-                        .eq(ContentVodMappingDO::getVodId, contentVodDTO.getVodId())
-                        .eq(ContentVodMappingDO::getContentId, addContentVodDTO.getContentId()));
-                if(contentVodMappingDO != null) {
-                    ContentVodMappingDO cvm = new ContentVodMappingDO();
-                    cvm.setId(contentVodMappingDO.getId());
-                    cvm.setVodAliasName(contentVodDTO.getVodAliasName());
-                    this.baseMapper.updateById(cvm);
-                }
-            }
+    public void updateContentVod(ContentVodDTO contentVodDTO) {
+        ContentVodMappingDO contentVodMappingDO = this.baseMapper.selectOne(new QueryWrapper<ContentVodMappingDO>()
+                .lambda()
+                .eq(ContentVodMappingDO::getVodId, contentVodDTO.getVodId())
+                .eq(ContentVodMappingDO::getContentId, contentVodDTO.getContentId()));
+        if(contentVodMappingDO != null) {
+            ContentVodMappingDO cvm = new ContentVodMappingDO();
+            cvm.setId(contentVodMappingDO.getId());
+            cvm.setVodAliasName(contentVodDTO.getVodAliasName());
+            this.baseMapper.updateById(cvm);
         }
     }
 }
--
libgit2 0.25.0