Use array list instead of a TreeSt for the service configurators

Using a TreeSet leads to service configurators with the same priority to be ignored
filiphr · Oct 7, 2024 · 24433c9 · 24433c9
1 parent b21ab92
commit 24433c9
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/...ne-common/src/main/java/org/flowable/common/engine/impl/AbstractServiceConfiguration.java b/...ne-common/src/main/java/org/flowable/common/engine/impl/AbstractServiceConfiguration.java
@@ -12,11 +12,11 @@
  */
 package org.flowable.common.engine.impl;
 
+import java.util.ArrayList;
 import java.util.Collection;
 import java.util.Comparator;
 import java.util.List;
 import java.util.Map;
-import java.util.TreeSet;
 
 import org.flowable.common.engine.api.delegate.event.FlowableEventDispatcher;
 import org.flowable.common.engine.api.delegate.event.FlowableEventListener;
@@ -75,7 +75,7 @@ public void setConfigurators(Collection<ServiceConfigurator<S>> configurators) {
         initConfigurators();
         this.configurators.clear();
         if (configurators != null) {
-            this.configurators.addAll(configurators);
+            this.configurators.addAll(configurators.stream().sorted(Comparator.comparingInt(ServiceConfigurator::getPriority)).toList());
         }
     }
 
@@ -87,7 +87,7 @@ public AbstractServiceConfiguration<S> addConfigurator(ServiceConfigurator<S> co
 
     protected void initConfigurators() {
         if (this.configurators == null) {
-            this.configurators = new TreeSet<>(Comparator.comparingInt(ServiceConfigurator::getPriority));
+            this.configurators = new ArrayList<>();
         }
     }