Unverified Commit c94ad601 authored by lgcareer's avatar lgcareer Committed by GitHub
Browse files

Merge pull request #287 from lgcareer/branch-1.0.2 

add determine if the login user is the owner of the process definition or schedule
parents cca2877b be230c61

escheduler-api/src/main/java/cn/escheduler/api/service/ProcessDefinitionService.java

+7 −1
Original line number Diff line number Diff line
@@ -357,13 +357,19 @@ public class ProcessDefinitionService extends BaseDAGService { 
            return checkResult;

        }





        ProcessDefinition processDefinition = processDefineMapper.queryByDefineId(processDefinitionId);



        if (processDefinition == null) {

            putMsg(result, Status.PROCESS_DEFINE_NOT_EXIST, processDefinitionId);

            return result;

        }



        // Determine if the login user is the owner of the process definition

        if (loginUser.getId() != processDefinition.getUserId()) {

            putMsg(result, Status.USER_NO_OPERATION_PERM);

            return result;

        }



        // check process definition is already online

        if (processDefinition.getReleaseState() == ReleaseState.ONLINE) {

            putMsg(result, Status.PROCESS_DEFINE_STATE_ONLINE,processDefinitionId);