[syncope] branch master updated: [SYNCOPE-1454] Ensuring no duplicated tasks

[ilgrosso]

This is an automated email from the ASF dual-hosted git repository.

ilgrosso pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/syncope.git


The following commit(s) were added to refs/heads/master by this push:
     new 289aeb3  [SYNCOPE-1454] Ensuring no duplicated tasks
289aeb3 is described below

commit 289aeb35eed92e2cd5e9ce1b20af52d8a6620609
Author: Francesco Chicchiriccò <ilgro...@apache.org>
AuthorDate: Tue Apr 2 12:17:58 2019 +0200

    [SYNCOPE-1454] Ensuring no duplicated tasks
---
 .../core/provisioning/java/propagation/PropagationManagerImpl.java  | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git 
a/core/provisioning-java/src/main/java/org/apache/syncope/core/provisioning/java/propagation/PropagationManagerImpl.java
 
b/core/provisioning-java/src/main/java/org/apache/syncope/core/provisioning/java/propagation/PropagationManagerImpl.java
index 4ca33c7..94a82bc 100644
--- 
a/core/provisioning-java/src/main/java/org/apache/syncope/core/provisioning/java/propagation/PropagationManagerImpl.java
+++ 
b/core/provisioning-java/src/main/java/org/apache/syncope/core/provisioning/java/propagation/PropagationManagerImpl.java
@@ -211,11 +211,13 @@ public class PropagationManagerImpl implements 
PropagationManager {
         UserUR userUR = wfResult.getResult().getLeft();
 
         // Propagate password update only to requested resources
-        List<PropagationTaskInfo> tasks = new ArrayList<>();
+        List<PropagationTaskInfo> tasks;
         if (userUR.getPassword() == null) {
             // a. no specific password propagation request: generate 
propagation tasks for any resource associated
             tasks = getUserUpdateTasks(wfResult, false, null);
         } else {
+            tasks = new ArrayList<>();
+
             // b. generate the propagation task list in two phases: first the 
ones containing password,
             // the the rest (with no password)
             WorkflowResult<Pair<UserUR, Boolean>> pwdWFResult = new 
WorkflowResult<>(
@@ -244,6 +246,8 @@ public class PropagationManagerImpl implements 
PropagationManager {
             if (!noPwdWFResult.getPropByRes().isEmpty()) {
                 tasks.addAll(getUserUpdateTasks(noPwdWFResult, false, 
pwdResourceNames));
             }
+
+            tasks = tasks.stream().distinct().collect(Collectors.toList());
         }
 
         return tasks;