update

physical-module-system/physical-system-biz/src/main/java/org/jeecg/modules/database/controller/DocumentFavoritesController.java

@@ -25,6 +25,7 @@ import org.springframework.web.bind.annotation.*;
 import org.springframework.web.servlet.ModelAndView;
 
 import java.util.Arrays;
+import java.util.Objects;
 
 /**
  * @Description: 我的收藏文档
@@ -84,14 +85,21 @@ public class DocumentFavoritesController extends JeecgController<DocumentFavorit
     public Result<String> add(@RequestBody DocumentFavorites documentFavorites, HttpServletRequest request) {
         String documentId = documentFavorites.getDocumentId();
         DocumentLibrary documentLibrary = documentLibraryService.getById(documentId);
-        documentFavorites.setDocumentTitle(documentLibrary.getTitle());
 
         String username = JwtUtil.getUserNameByToken(request);
         SysUser userByName = userService.getUserByName(username);
-        documentFavorites.setUserId(userByName.getId());
 
-        documentFavoritesService.save(documentFavorites);
+        DocumentFavorites old = documentFavoritesService.getByDocIdAndUserId(documentId, userByName.getId());
-        return Result.OK("添加成功！");
+        if (Objects.isNull(old)) {
+            documentFavorites.setId(null);
+        }else{
+            documentFavorites.setId(old.getId());
+        }
+        documentFavorites.setUserId(userByName.getId());
+        documentFavorites.setDocumentTitle(documentLibrary.getTitle());
+
+        documentFavoritesService.saveOrUpdate(documentFavorites);
+        return Result.OK("收藏成功！");
     }
 
     /**

physical-module-system/physical-system-biz/src/main/java/org/jeecg/modules/database/service/IDocumentFavoritesService.java

@@ -13,4 +13,5 @@ import java.util.List;
  */
 public interface IDocumentFavoritesService extends IService<DocumentFavorites> {
     List<DocumentFavorites> getTopFavoritesByUserId(String userId);
+    DocumentFavorites getByDocIdAndUserId(String userId,String documentId);
 }

physical-module-system/physical-system-biz/src/main/java/org/jeecg/modules/database/service/impl/DocumentFavoritesServiceImpl.java

@@ -27,4 +27,12 @@ public class DocumentFavoritesServiceImpl extends ServiceImpl<DocumentFavoritesM
         Page<DocumentFavorites> page = new Page<>(1, 5);
         return list(page, queryWrapper);
     }
+
+    @Override
+    public DocumentFavorites getByDocIdAndUserId(String userId, String documentId) {
+        LambdaQueryWrapper<DocumentFavorites> queryWrapper = new LambdaQueryWrapper<>();
+        queryWrapper.eq(DocumentFavorites::getUserId, userId);
+        queryWrapper.eq(DocumentFavorites::getDocumentId, userId);
+        return getOne(queryWrapper,false);
+    }
 }

physical-module-system/physical-system-biz/src/main/java/org/jeecg/modules/database/service/impl/DocumentVisitHistoryServiceImpl.java

@@ -30,7 +30,7 @@ public class DocumentVisitHistoryServiceImpl extends ServiceImpl<DocumentVisitHi
     @Override
     public DocumentVisitHistory findByUserAndDocId(String userId, String documentId) {
         DocumentVisitHistory one = getOne(new LambdaQueryWrapper<DocumentVisitHistory>().eq(DocumentVisitHistory::getUserId, userId)
-                .eq(DocumentVisitHistory::getDocumentId, documentId));
+                .eq(DocumentVisitHistory::getDocumentId, documentId), false);
         if (Objects.isNull(one)) {
             DocumentVisitHistory documentVisitHistory = new DocumentVisitHistory();
             documentVisitHistory.setUserId(userId);

physical-module-system/physical-system-biz/src/main/java/org/jeecg/modules/database/service/impl/ExperimentAnnealProcessServiceImpl.java

@@ -19,6 +19,6 @@ public class ExperimentAnnealProcessServiceImpl extends ServiceImpl<ExperimentAn
 
     @Override
     public ExperimentAnnealProcess getByExperimentId(String experimentId) {
-        return getOne(new LambdaQueryWrapper<ExperimentAnnealProcess>().eq(ExperimentAnnealProcess::getExperimentId, experimentId));
+        return getOne(new LambdaQueryWrapper<ExperimentAnnealProcess>().eq(ExperimentAnnealProcess::getExperimentId, experimentId),false);
     }
 }

physical-module-system/physical-system-biz/src/main/java/org/jeecg/modules/database/service/impl/ExperimentRadiationProcessServiceImpl.java

@@ -19,6 +19,6 @@ public class ExperimentRadiationProcessServiceImpl extends ServiceImpl<Experimen
 
     @Override
     public ExperimentRadiationProcess getByExperimentId(String experimentId) {
-        return getOne(new LambdaQueryWrapper<ExperimentRadiationProcess>().eq(ExperimentRadiationProcess::getExperimentId, experimentId));
+        return getOne(new LambdaQueryWrapper<ExperimentRadiationProcess>().eq(ExperimentRadiationProcess::getExperimentId, experimentId),false);
     }
 }

physical-module-system/physical-system-biz/src/main/java/org/jeecg/modules/database/service/impl/ExperimentReportServiceImpl.java

@@ -19,6 +19,6 @@ public class ExperimentReportServiceImpl extends ServiceImpl<ExperimentReportMap
 
     @Override
     public ExperimentReport getByExperimentId(String experimentId) {
-        return getOne(new LambdaQueryWrapper<ExperimentReport>().eq(ExperimentReport::getExperimentId, experimentId));
+        return getOne(new LambdaQueryWrapper<ExperimentReport>().eq(ExperimentReport::getExperimentId, experimentId),false);
     }
 }

physical-module-system/physical-system-biz/src/main/java/org/jeecg/modules/database/service/impl/ExperimentReviewServiceImpl.java

@@ -19,6 +19,6 @@ public class ExperimentReviewServiceImpl extends ServiceImpl<ExperimentReviewMap
 
     @Override
     public ExperimentReview getByExperimentId(String experimentId) {
-        return getOne(new LambdaQueryWrapper<ExperimentReview>().eq(ExperimentReview::getExperimentId, experimentId));
+        return getOne(new LambdaQueryWrapper<ExperimentReview>().eq(ExperimentReview::getExperimentId, experimentId),false);
     }
 }

physical-module-system/physical-system-biz/src/main/java/org/jeecg/modules/database/service/impl/ExperimentTestProcessServiceImpl.java

@@ -19,6 +19,6 @@ public class ExperimentTestProcessServiceImpl extends ServiceImpl<ExperimentTest
 
     @Override
     public ExperimentTestProcess getByExperimentId(String experimentId) {
-        return getOne(new LambdaQueryWrapper<ExperimentTestProcess>().eq(ExperimentTestProcess::getExperimentId, experimentId));
+        return getOne(new LambdaQueryWrapper<ExperimentTestProcess>().eq(ExperimentTestProcess::getExperimentId, experimentId),false);
     }
 }