Chore: add missing Override annotation

Author: dbmalkovsky

modules/flowable-engine-common-api/src/main/java/org/flowable/common/engine/api/delegate/FlowableMultiFunctionDelegate.java

@@ -20,10 +20,12 @@
  */
 public interface FlowableMultiFunctionDelegate extends FlowableFunctionDelegate {
 
+    @Override
     default String localName() {
         throw new UnsupportedOperationException("Function has more than one local name");
     }
 
+    @Override
     Collection<String> localNames();
 
     @Override

modules/flowable-engine-common/src/main/java/org/flowable/common/engine/impl/persistence/entity/data/impl/MybatisPropertyDataManager.java

@@ -42,6 +42,7 @@ public PropertyEntity create() {
         return new PropertyEntityImpl();
     }
 
+    @Override
     public void directInsertProperty(String name, String value) {
         HashMap<String, Object> params = new HashMap<>();
         params.put("name", name);

modules/flowable-engine/src/main/java/org/flowable/engine/delegate/event/impl/FlowableConditionalEventImpl.java

@@ -36,6 +36,7 @@ public void setConditionExpression(String conditionExpression) {
         this.conditionExpression = conditionExpression;
     }
 
+    @Override
     public String getConditionLanguage() {
         return conditionLanguage;
     }

modules/flowable-engine/src/main/java/org/flowable/engine/impl/ProcessInstanceQueryImpl.java

@@ -697,6 +697,7 @@ public ProcessInstanceQuery processInstanceNameLikeIgnoreCase(String nameLikeIgn
         return this;
     }
 
+    @Override
     public ProcessInstanceQuery processInstanceRootScopeId(String rootId) {
         if (inOrStatement) {
             this.currentOrQueryObject.rootScopeId = rootId;

modules/flowable-engine/src/main/java/org/flowable/engine/impl/bpmn/listener/ScriptExecutionListener.java

@@ -23,6 +23,7 @@ public class ScriptExecutionListener extends AbstractScriptEvaluator implements
 
     private static final long serialVersionUID = 1L;
 
+    @Override
     protected ScriptingEngines getScriptingEngines() {
         return CommandContextUtil.getProcessEngineConfiguration().getScriptingEngines();
     }

modules/flowable-engine/src/main/java/org/flowable/engine/impl/persistence/entity/ExecutionEntityImpl.java

@@ -1296,6 +1296,7 @@ public String getActivityName() {
         return activityName;
     }
 
+    @Override
     public String getCurrentActivityName() {
         return activityName;
     }

modules/flowable-engine/src/test/java/org/flowable/engine/test/bpmn/usertask/TestListener.java

@@ -23,6 +23,7 @@
 
 public class TestListener implements JavaDelegate, TaskListener {
 
+    @Override
     public void execute(DelegateExecution execution) {
         // noop
     }
@@ -34,4 +35,4 @@ public void notify(DelegateTask delegateTask) {
             delegateTask.setVariable("identityLinks_" + i, identityLinks.get(i).getId());
         }
     }
-}
+}

modules/flowable-event-registry-api/src/main/java/org/flowable/eventregistry/api/InboundEventInfoAwarePayloadExtractor.java

@@ -23,10 +23,12 @@
 @FunctionalInterface
 public interface InboundEventInfoAwarePayloadExtractor<T> extends InboundEventPayloadExtractor<T> {
 
+    @Override
     default Collection<EventPayloadInstance> extractPayload(EventModel eventModel, T payload) {
         throw new UnsupportedOperationException("Payload extraction should never call this ");
     }
 
+    @Override
     Collection<EventPayloadInstance> extractPayload(EventModel eventModel, FlowableEventInfo<T> event);
 
 }

modules/flowable-job-service/src/main/java/org/flowable/job/service/impl/nontx/NonTransactionalJobHandler.java

@@ -27,6 +27,7 @@
  */
 public interface NonTransactionalJobHandler<Output> extends JobHandler {
 
+    @Override
     default void execute(JobEntity job, String configuration, VariableScope variableScope, CommandContext commandContext) {
         throw new UnsupportedOperationException();
     }

modules/flowable-variable-service-api/src/main/java/org/flowable/variable/api/delegate/EmptyVariableScope.java

@@ -27,138 +27,172 @@
 class EmptyVariableScope implements VariableScope {
     static final EmptyVariableScope INSTANCE = new EmptyVariableScope();
 
+    @Override
     public Map<String, Object> getVariables() {
         return Collections.emptyMap();
     }
 
+    @Override
     public Map<String, Object> getVariablesLocal() {
         return Collections.emptyMap();
     }
 
+    @Override
     public Map<String, Object> getVariables(Collection<String> variableNames) {
         return Collections.emptyMap();
     }
 
+    @Override
     public Map<String, Object> getVariables(Collection<String> variableNames, boolean fetchAllVariables) {
         return Collections.emptyMap();
     }
 
+    @Override
     public Map<String, Object> getVariablesLocal(Collection<String> variableNames) {
         return Collections.emptyMap();
     }
 
+    @Override
     public Map<String, Object> getVariablesLocal(Collection<String> variableNames, boolean fetchAllVariables) {
         return Collections.emptyMap();
     }
 
+    @Override
     public Object getVariable(String variableName) {
         return null;
     }
 
+    @Override
     public Object getVariable(String variableName, boolean fetchAllVariables) {
         return null;
     }
 
+    @Override
     public Object getVariableLocal(String variableName) {
         return null;
     }
 
+    @Override
     public Object getVariableLocal(String variableName, boolean fetchAllVariables) {
         return null;
     }
 
+    @Override
     public <T> T getVariable(String variableName, Class<T> variableClass) {
         return null;
     }
 
+    @Override
     public <T> T getVariableLocal(String variableName, Class<T> variableClass) {
         return null;
     }
 
+    @Override
     public Set<String> getVariableNames() {
         return Collections.emptySet();
     }
 
+    @Override
     public Set<String> getVariableNamesLocal() {
         return null;
     }
 
+    @Override
     public Map<String, VariableInstance> getVariableInstances() {
         return null;
     }
 
+    @Override
     public Map<String, VariableInstance> getVariableInstances(Collection<String> variableNames) {
         return null;
     }
 
+    @Override
     public Map<String, VariableInstance> getVariableInstances(Collection<String> variableNames, boolean fetchAllVariables) {
         return null;
     }
 
+    @Override
     public Map<String, VariableInstance> getVariableInstancesLocal() {
         return null;
     }
 
+    @Override
     public Map<String, VariableInstance> getVariableInstancesLocal(Collection<String> variableNames) {
         return null;
     }
 
+    @Override
     public Map<String, VariableInstance> getVariableInstancesLocal(Collection<String> variableNames, boolean fetchAllVariables) {
         return null;
     }
 
+    @Override
     public VariableInstance getVariableInstance(String variableName) {
         return null;
     }
 
+    @Override
     public VariableInstance getVariableInstance(String variableName, boolean fetchAllVariables) {
         return null;
     }
 
+    @Override
     public VariableInstance getVariableInstanceLocal(String variableName) {
         return null;
     }
 
+    @Override
     public VariableInstance getVariableInstanceLocal(String variableName, boolean fetchAllVariables) {
         return null;
     }
 
+    @Override
     public void setVariable(String variableName, Object value) {
         throw new UnsupportedOperationException("Empty object, no variables can be set");
     }
 
+    @Override
     public void setVariable(String variableName, Object value, boolean fetchAllVariables) {
         throw new UnsupportedOperationException("Empty object, no variables can be set");
     }
 
+    @Override
     public Object setVariableLocal(String variableName, Object value) {
         throw new UnsupportedOperationException("Empty object, no variables can be set");
     }
 
+    @Override
     public Object setVariableLocal(String variableName, Object value, boolean fetchAllVariables) {
         throw new UnsupportedOperationException("Empty object, no variables can be set");
     }
 
+    @Override
     public void setVariables(Map<String, ? extends Object> variables) {
         throw new UnsupportedOperationException("Empty object, no variables can be set");
     }
 
+    @Override
     public void setVariablesLocal(Map<String, ? extends Object> variables) {
         throw new UnsupportedOperationException("Empty object, no variables can be set");
     }
 
+    @Override
     public boolean hasVariables() {
         return false;
     }
 
+    @Override
     public boolean hasVariablesLocal() {
         return false;
     }
 
+    @Override
     public boolean hasVariable(String variableName) {
         return false;
     }
 
+    @Override
     public boolean hasVariableLocal(String variableName) {
         return false;
     }
@@ -171,78 +205,97 @@ public void createVariablesLocal(Map<String, ? extends Object> variables) {
         throw new UnsupportedOperationException("Empty object, no variables can be created");
     }
 
+    @Override
     public void removeVariable(String variableName) {
         throw new UnsupportedOperationException("Empty object, no variables can be removed");
     }
 
+    @Override
     public void removeVariableLocal(String variableName) {
         throw new UnsupportedOperationException("Empty object, no variables can be removed");
     }
 
+    @Override
     public void removeVariables() {
         throw new UnsupportedOperationException("Empty object, no variables can be removed");
     }
 
+    @Override
     public void removeVariablesLocal() {
         throw new UnsupportedOperationException("Empty object, no variables can be removed");
     }
 
+    @Override
     public void removeVariables(Collection<String> variableNames) {
         throw new UnsupportedOperationException("Empty object, no variables can be removed");
     }
 
+    @Override
     public void removeVariablesLocal(Collection<String> variableNames) {
         throw new UnsupportedOperationException("Empty object, no variables can be removed");
     }
 
+    @Override
     public void setTransientVariablesLocal(Map<String, Object> transientVariables) {
         throw new UnsupportedOperationException("Empty object, no variables can be set");
     }
 
+    @Override
     public void setTransientVariableLocal(String variableName, Object variableValue) {
         throw new UnsupportedOperationException("Empty object, no variables can be set");
     }
 
+    @Override
     public void setTransientVariables(Map<String, Object> transientVariables) {
         throw new UnsupportedOperationException("Empty object, no variables can be set");
     }
 
+    @Override
     public void setTransientVariable(String variableName, Object variableValue) {
         throw new UnsupportedOperationException("Empty object, no variables can be set");
     }
 
+    @Override
     public Object getTransientVariableLocal(String variableName) {
         return null;
     }
 
+    @Override
     public Map<String, Object> getTransientVariablesLocal() {
         return null;
     }
 
+    @Override
     public Object getTransientVariable(String variableName) {
         return null;
     }
 
+    @Override
     public Map<String, Object> getTransientVariables() {
         return null;
     }
 
+    @Override
     public void removeTransientVariableLocal(String variableName) {
         throw new UnsupportedOperationException("Empty object, no variables can be removed");
     }
 
+    @Override
     public void removeTransientVariablesLocal() {
         throw new UnsupportedOperationException("Empty object, no variables can be removed");
     }
 
+    @Override
     public void removeTransientVariable(String variableName) {
         throw new UnsupportedOperationException("Empty object, no variables can be removed");
     }
 
+    @Override
     public void removeTransientVariables() {
         throw new UnsupportedOperationException("Empty object, no variables can be removed");
     }
 
+    @Override
     public String getTenantId() {
         return null;
     }