diff --git a/modules/activiti-engine/src/main/java/org/activiti/engine/impl/bpmn/parser/BpmnParse.java b/modules/activiti-engine/src/main/java/org/activiti/engine/impl/bpmn/parser/BpmnParse.java index 3617eff4fd49087ba9f2173fbe4d10fef13ce459..2e936e2f29488929bf4696a803c651877c7b72cf 100644 --- a/modules/activiti-engine/src/main/java/org/activiti/engine/impl/bpmn/parser/BpmnParse.java +++ b/modules/activiti-engine/src/main/java/org/activiti/engine/impl/bpmn/parser/BpmnParse.java @@ -2202,18 +2202,20 @@ public class BpmnParse extends Parse { // TimeDate TimerDeclarationType type = TimerDeclarationType.DATE; Expression expression = parseExpression(timerEventDefinition, "timeDate"); - // TimeCycle if (expression == null) { type = TimerDeclarationType.CYCLE; expression = parseExpression(timerEventDefinition, "timeCycle"); } - // TimeDuration if (expression == null) { type = TimerDeclarationType.DURATION; expression = parseExpression(timerEventDefinition, "timeDuration"); - } + } + // neither date, cycle or duration configured! + if (expression==null) { + addError("Timer needs configuration (either timeDate, timeCycle or timeDuration is needed).", timerEventDefinition); + } // Parse the timer declaration // TODO move the timer declaration into the bpmn activity or next to the