Skip to content

Commit

Permalink
fix: user check (#702)
Browse files Browse the repository at this point in the history
  • Loading branch information
masaimu authored Oct 19, 2023
1 parent 9fae622 commit ea10d5b
Showing 1 changed file with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -160,10 +160,10 @@ public JsonResult<Boolean> updateC(@RequestBody AlarmGroupDTO alarmGroup) {
return jsonResult;
}

return update(alarmGroup);
return update(alarmGroup, true);
}

public JsonResult<Boolean> update(AlarmGroupDTO alarmGroup) {
public JsonResult<Boolean> update(AlarmGroupDTO alarmGroup, boolean needCheckUser) {
final JsonResult<Boolean> result = new JsonResult<>();
facadeTemplate.manage(result, new ManageCallback() {
@Override
Expand All @@ -178,7 +178,7 @@ public void checkParameter() {
}
List<String> persons = alarmGroup.getUserList();
MonitorUser mu = RequestContext.getContext().mu;
if (!CollectionUtils.isEmpty(persons)) {
if (!CollectionUtils.isEmpty(persons) && needCheckUser) {
for (String person : persons) {
ParaCheckUtil.checkParaBoolean(
parameterSecurityService.checkUserTenantAndWorkspace(person, mu),
Expand Down

0 comments on commit ea10d5b

Please sign in to comment.