review--->edit

product/product-biz/src/main/java/cn/iocoder/mall/product/biz/dao/product/ProductBrandMapper.java

@@ -13,8 +13,7 @@ import org.springframework.stereotype.Repository;
 @Repository
 public interface ProductBrandMapper extends BaseMapper<ProductBrandDO> {
 
-    // TODO FROM 芋艿 to q2118cs：selectPageByParams
-    default IPage<ProductBrandDO> selectListByParams(ProductBrandPageDTO productBrandPageDTO) {
+    default IPage<ProductBrandDO> selectPageByParams(ProductBrandPageDTO productBrandPageDTO) {
         Page<ProductBrandDO> page = new Page<>(productBrandPageDTO.getPageNo(), productBrandPageDTO.getPageSize());
         LambdaQueryWrapper<ProductBrandDO> queryWrapper = Wrappers.<ProductBrandDO>query().lambda()
                 .like(StringUtils.isNotBlank(productBrandPageDTO.getName()), ProductBrandDO::getName, productBrandPageDTO.getName())

product/product-biz/src/main/java/cn/iocoder/mall/product/biz/service/product/impl/ProductBrandServiceImpl.java

@@ -32,7 +32,7 @@ public class ProductBrandServiceImpl implements ProductBrandService {
 
     @Override
     public PageResult<ProductBrandBO> getProductBrandPage(ProductBrandPageDTO productBrandPageDTO) {
-        IPage<ProductBrandDO> brandPage = productBrandMapper.selectListByParams(productBrandPageDTO);
+        IPage<ProductBrandDO> brandPage = productBrandMapper.selectPageByParams(productBrandPageDTO);
         return ProductBrandConvert.INSTANCE.convertPage(brandPage);
     }
 

product/product-rest/src/main/java/cn/iocoder/mall/product/rest/controller/admins/AdminsProductBrandController.java

@@ -17,6 +17,7 @@ import io.swagger.annotations.Api;
 import io.swagger.annotations.ApiImplicitParam;
 import io.swagger.annotations.ApiImplicitParams;
 import io.swagger.annotations.ApiOperation;
+import lombok.AllArgsConstructor;
 import org.springframework.validation.annotation.Validated;
 import org.springframework.web.bind.annotation.*;
 
@@ -24,11 +25,11 @@ import static cn.iocoder.common.framework.vo.CommonResult.success;
 
 @RestController
 @RequestMapping("admins/brand")
-@Api("商品品牌") // TODO FROM 芋艿 to q2118cs：管理员 - 商品品牌 API。。哈哈哈，原来没规范
+@Api("管理员 - 商品品牌 API")
+@AllArgsConstructor
 public class AdminsProductBrandController {
 
-    // TODO FROM 芋艿 to q2118cs：貌似没注入哈
-    private ProductBrandService productBrandService;
+    private final ProductBrandService productBrandService;
 
     @PostMapping("/add")
     @ApiOperation("创建品牌")