Parcourir la source

delAlertgroupById add entity exist check (#1602)

* misspell  words

* modify common queue TaskQueueZKImplTest.java unit test

* extends  BaseTaskQueueTest get zkServer

* modify zk config

* add MonitorServiceTest

* mobile phone need 11 number

* delete file

* delAlertgroupById add entity exist check
samz406 il y a 5 ans
Parent
commit
597d685955

+ 6 - 0
dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/service/AlertGroupService.java

@@ -193,6 +193,12 @@ public class AlertGroupService extends BaseService{
         if (checkAdmin(loginUser, result)){
             return result;
         }
+        //check exist
+        AlertGroup alertGroup = alertGroupMapper.selectById(id);
+        if (alertGroup == null) {
+            putMsg(result, Status.ALERT_GROUP_NOT_EXIST);
+            return result;
+        }
 
         userAlertGroupMapper.deleteByAlertgroupId(id);
         alertGroupMapper.deleteById(id);