From 210828d1a6947596fe00ab30fcd8725c6714dde4 Mon Sep 17 00:00:00 2001
From: puzhibing <393733352@qq.com>
Date: 星期四, 02 一月 2025 21:04:16 +0800
Subject: [PATCH] 修改bug

---
 ruoyi-service/ruoyi-other/src/main/java/com/ruoyi/other/controller/GoodsEvaluateController.java |   46 +++++++++++++++++++++++++---------------------
 1 files changed, 25 insertions(+), 21 deletions(-)

diff --git a/ruoyi-service/ruoyi-other/src/main/java/com/ruoyi/other/controller/GoodsEvaluateController.java b/ruoyi-service/ruoyi-other/src/main/java/com/ruoyi/other/controller/GoodsEvaluateController.java
index b1153e2..8ff1004 100644
--- a/ruoyi-service/ruoyi-other/src/main/java/com/ruoyi/other/controller/GoodsEvaluateController.java
+++ b/ruoyi-service/ruoyi-other/src/main/java/com/ruoyi/other/controller/GoodsEvaluateController.java
@@ -3,6 +3,7 @@
 
 import com.baomidou.mybatisplus.core.conditions.query.LambdaQueryWrapper;
 import com.baomidou.mybatisplus.core.conditions.update.LambdaUpdateWrapper;
+import com.baomidou.mybatisplus.extension.plugins.pagination.Page;
 import com.ruoyi.account.api.feignClient.AppUserClient;
 import com.ruoyi.account.api.model.AppUser;
 import com.ruoyi.common.core.domain.R;
@@ -49,9 +50,11 @@
         List<GoodsEvaluate> list = goodsEvaluateService.lambdaQuery().eq(GoodsEvaluate::getGoodsId, goodsId).list();
         for (GoodsEvaluate goodsEvaluate : list) {
             AppUser appUserById = appUserClient.getAppUserById(goodsEvaluate.getAppUserId());
-            goodsEvaluate.setUserName(appUserById.getName());
-            goodsEvaluate.setAvatar(appUserById.getAvatar());
-            goodsEvaluate.setIdStr(String.valueOf(goodsEvaluate.getId()));
+            if(null != appUserById){
+                goodsEvaluate.setUserName(appUserById.getName());
+                goodsEvaluate.setAvatar(appUserById.getAvatar());
+                goodsEvaluate.setIdStr(String.valueOf(goodsEvaluate.getId()));
+            }
         }
         return R.ok(list);
     }
@@ -90,7 +93,7 @@
      * 删除评论
      */
     @DeleteMapping("/delete/{id}")
-    @ApiOperation(value = "商品管理-评价管理-删除评论-删除评论", tags = {"管理后台"})
+    @ApiOperation(value = "删除评论", tags = {"管理后台-商品管理-评价管理"})
     public R<Void> delete(@PathVariable("id") Long id){
         goodsEvaluateService.removeById(id);
         return R.ok();
@@ -100,37 +103,38 @@
      * 评论列表
      */
     @GetMapping("/list")
-    @ApiOperation(value = "商品管理-评价管理-评论列表-评论列表", tags = {"管理后台"})
-    public R<List<GoodsEvaluate>> list(GoodsEvaluate goodsEvaluate){
-        List<GoodsEvaluate> list = goodsEvaluateService.lambdaQuery()
-                .like(StringUtils.isNotEmpty(goodsEvaluate.getGoodsName()),GoodsEvaluate::getGoodsName, goodsEvaluate.getComment())
-                .like(StringUtils.isNotEmpty(goodsEvaluate.getUserName()),GoodsEvaluate::getUserName, goodsEvaluate.getUserName())
-                .like(StringUtils.isNotEmpty(goodsEvaluate.getPhone()),GoodsEvaluate::getPhone, goodsEvaluate.getPhone())
-                .eq(goodsEvaluate.getStatus()!=null,GoodsEvaluate::getStatus, goodsEvaluate.getStatus())
-                .orderByDesc(GoodsEvaluate::getCreateTime)
-                .list();
+    @ApiOperation(value = "评论列表", tags = {"管理后台-商品管理-评价管理"})
+    public R<Page<GoodsEvaluate>> list(@ApiParam("页码") @RequestParam Integer pageNum,
+                                       @ApiParam("每一页数据大小") Integer pageSize,
+                                       GoodsEvaluate goodsEvaluate){
+        Page<GoodsEvaluate> page = goodsEvaluateService.page(Page.of(pageNum, pageSize), new LambdaQueryWrapper<GoodsEvaluate>()
+                .like(StringUtils.isNotEmpty(goodsEvaluate.getGoodsName()), GoodsEvaluate::getGoodsName, goodsEvaluate.getComment())
+                .like(StringUtils.isNotEmpty(goodsEvaluate.getUserName()), GoodsEvaluate::getUserName, goodsEvaluate.getUserName())
+                .like(StringUtils.isNotEmpty(goodsEvaluate.getPhone()), GoodsEvaluate::getPhone, goodsEvaluate.getPhone())
+                .eq(goodsEvaluate.getStatus() != null, GoodsEvaluate::getStatus, goodsEvaluate.getStatus())
+                .orderByDesc(GoodsEvaluate::getCreateTime));
 
-        list.forEach(this::buildDetail);
 
-        return R.ok(list);
+        page.getRecords().forEach(this::buildDetail);
+
+        return R.ok(page);
     }
 
     private void buildDetail(GoodsEvaluate item) {
         AppUser appUser = appUserClient.getAppUserById(item.getAppUserId());
-        if (appUser == null){
-            throw new RuntimeException("获取用户信息失败");
+        if (appUser != null){
+            item.setPhone(appUser.getPhone());
+            item.setUserName(appUser.getName());
         }
         Goods goods = goodsService.getById(item.getGoodsId());
         item.setGoodsName(goods.getName());
-        item.setPhone(appUser.getPhone());
-        item.setUserName(appUser.getName());
     }
 
     /**
      * 评论详情
      */
     @GetMapping("/{id}")
-    @ApiOperation(value = "商品管理-评价管理-评论详情", tags = {"管理后台"})
+    @ApiOperation(value = "评论详情", tags = {"管理后台-商品管理-评价管理"})
     public R<GoodsEvaluate> getDetail(@PathVariable("id") Long id){
         GoodsEvaluate goodsEvaluate = goodsEvaluateService.getById(id);
         buildDetail(goodsEvaluate);
@@ -141,7 +145,7 @@
      * 评论上/下架
      */
     @PutMapping("/changeStatus")
-    @ApiOperation(value = "商品管理-评价管理-评论上/下架", tags = {"管理后台"})
+    @ApiOperation(value = "上/下架", tags = {"管理后台-商品管理-评价管理"})
     public R<Void> changeStatus(@RequestBody GoodsEvaluate goodsEvaluate){
         goodsEvaluateService.update(new LambdaUpdateWrapper<GoodsEvaluate>()
                 .eq(GoodsEvaluate::getId, goodsEvaluate.getId())

--
Gitblit v1.7.1