fix(approve): 修复流程配置节点审批人为空时的异常处理

云

2 天以前 dc3336685e80c593a3654a6e53e3e1d1d13b2b50

 src/main/java/com/ruoyi/project/system/controller/SysMenuController.java

@@ -1,30 +1,19 @@
package com.ruoyi.project.system.controller;



import java.util.List;

import org.springframework.beans.factory.annotation.Autowired;

import org.springframework.security.access.prepost.PreAuthorize;

import org.springframework.validation.annotation.Validated;

import org.springframework.web.bind.annotation.DeleteMapping;

import org.springframework.web.bind.annotation.GetMapping;

import org.springframework.web.bind.annotation.PathVariable;

import org.springframework.web.bind.annotation.PostMapping;

import org.springframework.web.bind.annotation.PutMapping;

import org.springframework.web.bind.annotation.RequestBody;

import org.springframework.web.bind.annotation.RequestMapping;

import org.springframework.web.bind.annotation.RestController;

import com.ruoyi.common.constant.Constants;

import com.ruoyi.common.constant.UserConstants;

import com.ruoyi.common.utils.SecurityUtils;

import com.ruoyi.common.utils.ServletUtils;

import com.ruoyi.common.utils.StringUtils;

import com.ruoyi.framework.aspectj.lang.annotation.Log;

import com.ruoyi.framework.aspectj.lang.enums.BusinessType;

import com.ruoyi.framework.security.LoginUser;

import com.ruoyi.framework.security.service.TokenService;

import com.ruoyi.framework.web.controller.BaseController;

import com.ruoyi.framework.web.domain.AjaxResult;

import com.ruoyi.project.system.domain.SysMenu;

import com.ruoyi.project.system.service.ISysMenuService;

import lombok.AllArgsConstructor;

import org.springframework.security.access.prepost.PreAuthorize;

import org.springframework.validation.annotation.Validated;

import org.springframework.web.bind.annotation.*;



import java.util.List;



/**

 * 菜单信息

@@ -33,13 +22,10 @@
 */

@RestController

@RequestMapping("/system/menu")

@AllArgsConstructor

public class SysMenuController extends BaseController

{

    @Autowired

    private ISysMenuService menuService;



    @Autowired

    private TokenService tokenService;



    /**

     * 获取菜单列表

@@ -48,10 +34,8 @@
    @GetMapping("/list")

    public AjaxResult list(SysMenu menu)

    {

        LoginUser loginUser = tokenService.getLoginUser(ServletUtils.getRequest());

        Long userId = loginUser.getUser().getUserId();

        List<SysMenu> menus = menuService.selectMenuList(menu, userId);

        return AjaxResult.success(menus);

        List<SysMenu> menus = menuService.selectMenuList(menu, getUserId());

        return success(menus);

    }



    /**

@@ -61,7 +45,7 @@
    @GetMapping(value = "/{menuId}")

    public AjaxResult getInfo(@PathVariable Long menuId)

    {

        return AjaxResult.success(menuService.selectMenuById(menuId));

        return success(menuService.selectMenuById(menuId));

    }



    /**

@@ -70,10 +54,8 @@
    @GetMapping("/treeselect")

    public AjaxResult treeselect(SysMenu menu)

    {

        LoginUser loginUser = tokenService.getLoginUser(ServletUtils.getRequest());

        Long userId = loginUser.getUser().getUserId();

        List<SysMenu> menus = menuService.selectMenuList(menu, userId);

        return AjaxResult.success(menuService.buildMenuTreeSelect(menus));

        List<SysMenu> menus = menuService.selectMenuList(menu, getUserId());

        return success(menuService.buildMenuTreeSelect(menus));

    }



    /**

@@ -82,8 +64,7 @@
    @GetMapping(value = "/roleMenuTreeselect/{roleId}")

    public AjaxResult roleMenuTreeselect(@PathVariable("roleId") Long roleId)

    {

        LoginUser loginUser = tokenService.getLoginUser(ServletUtils.getRequest());

        List<SysMenu> menus = menuService.selectMenuList(loginUser.getUser().getUserId());

        List<SysMenu> menus = menuService.selectMenuList(getUserId());

        AjaxResult ajax = AjaxResult.success();

        ajax.put("checkedKeys", menuService.selectMenuListByRoleId(roleId));

        ajax.put("menus", menuService.buildMenuTreeSelect(menus));

@@ -98,16 +79,15 @@
    @PostMapping

    public AjaxResult add(@Validated @RequestBody SysMenu menu)

    {

        if (UserConstants.NOT_UNIQUE.equals(menuService.checkMenuNameUnique(menu)))

        if (!menuService.checkMenuNameUnique(menu))

        {

            return AjaxResult.error("新增菜单'" + menu.getMenuName() + "'失败，菜单名称已存在");

            return error("新增菜单'" + menu.getMenuName() + "'失败，菜单名称已存在");

        }

        else if (UserConstants.YES_FRAME.equals(menu.getIsFrame())

                && !StringUtils.startsWithAny(menu.getPath(), Constants.HTTP, Constants.HTTPS))

        else if (UserConstants.YES_FRAME.equals(menu.getIsFrame()) && !StringUtils.ishttp(menu.getPath()))

        {

            return AjaxResult.error("新增菜单'" + menu.getMenuName() + "'失败，地址必须以http(s)://开头");

            return error("新增菜单'" + menu.getMenuName() + "'失败，地址必须以http(s)://开头");

        }

        menu.setCreateBy(SecurityUtils.getUsername());

        menu.setCreateBy(getUsername());

        return toAjax(menuService.insertMenu(menu));

    }



@@ -119,16 +99,19 @@
    @PutMapping

    public AjaxResult edit(@Validated @RequestBody SysMenu menu)

    {

        if (UserConstants.NOT_UNIQUE.equals(menuService.checkMenuNameUnique(menu)))

        if (!menuService.checkMenuNameUnique(menu))

        {

            return AjaxResult.error("修改菜单'" + menu.getMenuName() + "'失败，菜单名称已存在");

            return error("修改菜单'" + menu.getMenuName() + "'失败，菜单名称已存在");

        }

        else if (UserConstants.YES_FRAME.equals(menu.getIsFrame())

                && !StringUtils.startsWithAny(menu.getPath(), Constants.HTTP, Constants.HTTPS))

        else if (UserConstants.YES_FRAME.equals(menu.getIsFrame()) && !StringUtils.ishttp(menu.getPath()))

        {

            return AjaxResult.error("新增菜单'" + menu.getMenuName() + "'失败，地址必须以http(s)://开头");

            return error("修改菜单'" + menu.getMenuName() + "'失败，地址必须以http(s)://开头");

        }

        menu.setUpdateBy(SecurityUtils.getUsername());

        else if (menu.getMenuId().equals(menu.getParentId()))

        {

            return error("修改菜单'" + menu.getMenuName() + "'失败，上级菜单不能选择自己");

        }

        menu.setUpdateBy(getUsername());

        return toAjax(menuService.updateMenu(menu));

    }



@@ -142,11 +125,11 @@
    {

        if (menuService.hasChildByMenuId(menuId))

        {

            return AjaxResult.error("存在子菜单,不允许删除");

            return warn("存在子菜单,不允许删除");

        }

        if (menuService.checkMenuExistRole(menuId))

        {

            return AjaxResult.error("菜单已分配,不允许删除");

            return warn("菜单已分配,不允许删除");

        }

        return toAjax(menuService.deleteMenuById(menuId));

    }