fix(测试计划): 计划模块树权限控制有误
--bug=1041244 --user=宋昌昌 【测试计划】当前用户有测试计划删除和编辑权限-删除和重命名模块-提示无权限 https://www.tapd.cn/55049933/s/1521464
This commit is contained in:
parent
d99a79ac0c
commit
dc22373191
|
@ -52,7 +52,7 @@ public class TestPlanModuleController {
|
||||||
@PostMapping("/update")
|
@PostMapping("/update")
|
||||||
@Operation(summary = "测试计划管理-模块树-修改模块")
|
@Operation(summary = "测试计划管理-模块树-修改模块")
|
||||||
@RequiresPermissions(PermissionConstants.TEST_PLAN_READ_UPDATE)
|
@RequiresPermissions(PermissionConstants.TEST_PLAN_READ_UPDATE)
|
||||||
@CheckOwner(resourceId = "#request.getId()", resourceType = "file_module")
|
@CheckOwner(resourceId = "#request.getId()", resourceType = "test_plan_module")
|
||||||
public boolean list(@RequestBody @Validated TestPlanModuleUpdateRequest request) {
|
public boolean list(@RequestBody @Validated TestPlanModuleUpdateRequest request) {
|
||||||
testPlanManagementService.checkModuleIsOpen(request.getId(), TestPlanResourceConfig.CHECK_TYPE_TEST_PLAN_MODULE, Collections.singletonList(TestPlanResourceConfig.CHECK_TYPE_TEST_PLAN));
|
testPlanManagementService.checkModuleIsOpen(request.getId(), TestPlanResourceConfig.CHECK_TYPE_TEST_PLAN_MODULE, Collections.singletonList(TestPlanResourceConfig.CHECK_TYPE_TEST_PLAN));
|
||||||
testPlanModuleService.update(request, SessionUtils.getUserId(),"/test-plan/module/update", HttpMethodConstants.POST.name());
|
testPlanModuleService.update(request, SessionUtils.getUserId(),"/test-plan/module/update", HttpMethodConstants.POST.name());
|
||||||
|
@ -62,7 +62,7 @@ public class TestPlanModuleController {
|
||||||
@GetMapping("/delete/{deleteId}")
|
@GetMapping("/delete/{deleteId}")
|
||||||
@Operation(summary = "测试计划管理-模块树-删除模块")
|
@Operation(summary = "测试计划管理-模块树-删除模块")
|
||||||
@RequiresPermissions(PermissionConstants.TEST_PLAN_READ_DELETE)
|
@RequiresPermissions(PermissionConstants.TEST_PLAN_READ_DELETE)
|
||||||
@CheckOwner(resourceId = "#deleteId", resourceType = "file_module")
|
@CheckOwner(resourceId = "#deleteId", resourceType = "test_plan_module")
|
||||||
public void deleteNode(@PathVariable String deleteId) {
|
public void deleteNode(@PathVariable String deleteId) {
|
||||||
testPlanManagementService.checkModuleIsOpen(deleteId, TestPlanResourceConfig.CHECK_TYPE_TEST_PLAN_MODULE, Collections.singletonList(TestPlanResourceConfig.CHECK_TYPE_TEST_PLAN));
|
testPlanManagementService.checkModuleIsOpen(deleteId, TestPlanResourceConfig.CHECK_TYPE_TEST_PLAN_MODULE, Collections.singletonList(TestPlanResourceConfig.CHECK_TYPE_TEST_PLAN));
|
||||||
testPlanModuleService.deleteModule(deleteId, SessionUtils.getUserId(),"/test-plan/module/delete",HttpMethodConstants.GET.name());
|
testPlanModuleService.deleteModule(deleteId, SessionUtils.getUserId(),"/test-plan/module/delete",HttpMethodConstants.GET.name());
|
||||||
|
|
Loading…
Reference in New Issue