You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@syncope.apache.org by il...@apache.org on 2014/11/05 11:08:26 UTC

[1/8] git commit: Fix typo

Repository: syncope
Updated Branches:
  refs/heads/1_1_X 05d2985a4 -> 296ffa369
  refs/heads/1_2_X bcc3e5be5 -> 76b35a787
  refs/heads/master a3ea02de5 -> 681cf9f5a


Fix typo


Project: http://git-wip-us.apache.org/repos/asf/syncope/repo
Commit: http://git-wip-us.apache.org/repos/asf/syncope/commit/296ffa36
Tree: http://git-wip-us.apache.org/repos/asf/syncope/tree/296ffa36
Diff: http://git-wip-us.apache.org/repos/asf/syncope/diff/296ffa36

Branch: refs/heads/1_1_X
Commit: 296ffa369f50f31ccef85078a60daeb8c16ff912
Parents: 05d2985
Author: Francesco Chicchiriccò <il...@apache.org>
Authored: Wed Nov 5 10:50:01 2014 +0100
Committer: Francesco Chicchiriccò <il...@apache.org>
Committed: Wed Nov 5 10:50:01 2014 +0100

----------------------------------------------------------------------
 .../core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/syncope/blob/296ffa36/core/src/main/java/org/apache/syncope/core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/syncope/core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java b/core/src/main/java/org/apache/syncope/core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java
index 7d40183..646bbca 100644
--- a/core/src/main/java/org/apache/syncope/core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java
+++ b/core/src/main/java/org/apache/syncope/core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java
@@ -595,7 +595,7 @@ public class ActivitiUserWorkflowAdapter extends AbstractUserWorkflowAdapter {
 
         if (!checked.getKey().getOwner().equals(username)) {
             throw new WorkflowException(new IllegalArgumentException("Task " + form.getTaskId() + " assigned to "
-                    + checked.getKey().getOwner() + " but submited by " + username));
+                    + checked.getKey().getOwner() + " but submitted by " + username));
         }
 
         SyncopeUser user = userDAO.findByWorkflowId(checked.getKey().getProcessInstanceId());


[7/8] git commit: [SYNCOPE-600] Adding miss pre-condition for form availability after form submit

Posted by il...@apache.org.
[SYNCOPE-600] Adding miss pre-condition for form availability after form submit


Project: http://git-wip-us.apache.org/repos/asf/syncope/repo
Commit: http://git-wip-us.apache.org/repos/asf/syncope/commit/76b35a78
Tree: http://git-wip-us.apache.org/repos/asf/syncope/tree/76b35a78
Diff: http://git-wip-us.apache.org/repos/asf/syncope/diff/76b35a78

Branch: refs/heads/1_2_X
Commit: 76b35a7870eee368d8b78e71d39bd9ecc2de5934
Parents: 377963e
Author: Francesco Chicchiriccò <il...@apache.org>
Authored: Wed Nov 5 11:07:41 2014 +0100
Committer: Francesco Chicchiriccò <il...@apache.org>
Committed: Wed Nov 5 11:07:41 2014 +0100

----------------------------------------------------------------------
 .../activiti/ActivitiUserWorkflowAdapter.java   | 32 +++++++++++---------
 1 file changed, 18 insertions(+), 14 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/syncope/blob/76b35a78/core/src/main/java/org/apache/syncope/core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/syncope/core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java b/core/src/main/java/org/apache/syncope/core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java
index fdf35de..4f0d876 100644
--- a/core/src/main/java/org/apache/syncope/core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java
+++ b/core/src/main/java/org/apache/syncope/core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java
@@ -208,8 +208,9 @@ public class ActivitiUserWorkflowAdapter extends AbstractUserWorkflowAdapter {
     private Set<String> getPerformedTasks(final SyncopeUser user) {
         final Set<String> result = new HashSet<String>();
 
-        for (HistoricActivityInstance task : historyService.createHistoricActivityInstanceQuery().executionId(user.
-                getWorkflowId()).list()) {
+        for (HistoricActivityInstance task
+                : historyService.createHistoricActivityInstanceQuery().executionId(user.getWorkflowId()).list()) {
+
             result.add(task.getActivityId());
         }
 
@@ -261,11 +262,11 @@ public class ActivitiUserWorkflowAdapter extends AbstractUserWorkflowAdapter {
             throwException(e, "While starting " + WF_PROCESS_ID + " instance");
         }
 
-        SyncopeUser user = (SyncopeUser) runtimeService.getVariable(
-                processInstance.getProcessInstanceId(), SYNCOPE_USER);
+        SyncopeUser user =
+                runtimeService.getVariable(processInstance.getProcessInstanceId(), SYNCOPE_USER, SyncopeUser.class);
 
-        Boolean updatedEnabled = (Boolean) runtimeService.getVariable(
-                processInstance.getProcessInstanceId(), ENABLED);
+        Boolean updatedEnabled =
+                runtimeService.getVariable(processInstance.getProcessInstanceId(), ENABLED, Boolean.class);
         if (updatedEnabled != null) {
             user.setSuspended(!updatedEnabled);
         }
@@ -278,8 +279,8 @@ public class ActivitiUserWorkflowAdapter extends AbstractUserWorkflowAdapter {
         updateStatus(user);
         user = userDAO.save(user);
 
-        Boolean propagateEnable = (Boolean) runtimeService.getVariable(
-                processInstance.getProcessInstanceId(), PROPAGATE_ENABLE);
+        Boolean propagateEnable =
+                runtimeService.getVariable(processInstance.getProcessInstanceId(), PROPAGATE_ENABLE, Boolean.class);
         if (propagateEnable == null) {
             propagateEnable = enabled;
         }
@@ -353,12 +354,12 @@ public class ActivitiUserWorkflowAdapter extends AbstractUserWorkflowAdapter {
         updateStatus(user);
         SyncopeUser updated = userDAO.save(user);
 
-        PropagationByResource propByRes = (PropagationByResource) runtimeService.getVariable(
-                user.getWorkflowId(), PROP_BY_RESOURCE);
+        PropagationByResource propByRes =
+                runtimeService.getVariable(user.getWorkflowId(), PROP_BY_RESOURCE, PropagationByResource.class);
 
         saveForFormSubmit(updated, userMod.getPassword(), propByRes);
 
-        Boolean propagateEnable = (Boolean) runtimeService.getVariable(user.getWorkflowId(), PROPAGATE_ENABLE);
+        Boolean propagateEnable = runtimeService.getVariable(user.getWorkflowId(), PROPAGATE_ENABLE, Boolean.class);
 
         return new WorkflowResult<Map.Entry<UserMod, Boolean>>(
                 new SimpleEntry<UserMod, Boolean>(userMod, propagateEnable), propByRes, tasks);
@@ -826,16 +827,19 @@ public class ActivitiUserWorkflowAdapter extends AbstractUserWorkflowAdapter {
 
         // see if there is any propagation to be done
         PropagationByResource propByRes =
-                (PropagationByResource) runtimeService.getVariable(user.getWorkflowId(), PROP_BY_RESOURCE);
+                runtimeService.getVariable(user.getWorkflowId(), PROP_BY_RESOURCE, PropagationByResource.class);
 
         // fetch - if available - the encrypted password
         String clearPassword = null;
-        String encryptedPwd = (String) runtimeService.getVariable(user.getWorkflowId(), ENCRYPTED_PWD);
+        String encryptedPwd = runtimeService.getVariable(user.getWorkflowId(), ENCRYPTED_PWD, String.class);
         if (StringUtils.isNotBlank(encryptedPwd)) {
             clearPassword = decrypt(encryptedPwd);
         }
 
-        UserMod userMod = (UserMod) runtimeService.getVariable(user.getWorkflowId(), USER_MOD);
+        // supports approval chains
+        saveForFormSubmit(user, clearPassword, propByRes);
+
+        UserMod userMod = runtimeService.getVariable(user.getWorkflowId(), USER_MOD, UserMod.class);
         if (userMod == null) {
             userMod = new UserMod();
             userMod.setId(updated.getId());


[3/8] git commit: Fix typo

Posted by il...@apache.org.
Fix typo


Project: http://git-wip-us.apache.org/repos/asf/syncope/repo
Commit: http://git-wip-us.apache.org/repos/asf/syncope/commit/296ffa36
Tree: http://git-wip-us.apache.org/repos/asf/syncope/tree/296ffa36
Diff: http://git-wip-us.apache.org/repos/asf/syncope/diff/296ffa36

Branch: refs/heads/master
Commit: 296ffa369f50f31ccef85078a60daeb8c16ff912
Parents: 05d2985
Author: Francesco Chicchiriccò <il...@apache.org>
Authored: Wed Nov 5 10:50:01 2014 +0100
Committer: Francesco Chicchiriccò <il...@apache.org>
Committed: Wed Nov 5 10:50:01 2014 +0100

----------------------------------------------------------------------
 .../core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/syncope/blob/296ffa36/core/src/main/java/org/apache/syncope/core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/syncope/core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java b/core/src/main/java/org/apache/syncope/core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java
index 7d40183..646bbca 100644
--- a/core/src/main/java/org/apache/syncope/core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java
+++ b/core/src/main/java/org/apache/syncope/core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java
@@ -595,7 +595,7 @@ public class ActivitiUserWorkflowAdapter extends AbstractUserWorkflowAdapter {
 
         if (!checked.getKey().getOwner().equals(username)) {
             throw new WorkflowException(new IllegalArgumentException("Task " + form.getTaskId() + " assigned to "
-                    + checked.getKey().getOwner() + " but submited by " + username));
+                    + checked.getKey().getOwner() + " but submitted by " + username));
         }
 
         SyncopeUser user = userDAO.findByWorkflowId(checked.getKey().getProcessInstanceId());


[2/8] git commit: Fix typo

Posted by il...@apache.org.
Fix typo


Project: http://git-wip-us.apache.org/repos/asf/syncope/repo
Commit: http://git-wip-us.apache.org/repos/asf/syncope/commit/296ffa36
Tree: http://git-wip-us.apache.org/repos/asf/syncope/tree/296ffa36
Diff: http://git-wip-us.apache.org/repos/asf/syncope/diff/296ffa36

Branch: refs/heads/1_2_X
Commit: 296ffa369f50f31ccef85078a60daeb8c16ff912
Parents: 05d2985
Author: Francesco Chicchiriccò <il...@apache.org>
Authored: Wed Nov 5 10:50:01 2014 +0100
Committer: Francesco Chicchiriccò <il...@apache.org>
Committed: Wed Nov 5 10:50:01 2014 +0100

----------------------------------------------------------------------
 .../core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/syncope/blob/296ffa36/core/src/main/java/org/apache/syncope/core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/syncope/core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java b/core/src/main/java/org/apache/syncope/core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java
index 7d40183..646bbca 100644
--- a/core/src/main/java/org/apache/syncope/core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java
+++ b/core/src/main/java/org/apache/syncope/core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java
@@ -595,7 +595,7 @@ public class ActivitiUserWorkflowAdapter extends AbstractUserWorkflowAdapter {
 
         if (!checked.getKey().getOwner().equals(username)) {
             throw new WorkflowException(new IllegalArgumentException("Task " + form.getTaskId() + " assigned to "
-                    + checked.getKey().getOwner() + " but submited by " + username));
+                    + checked.getKey().getOwner() + " but submitted by " + username));
         }
 
         SyncopeUser user = userDAO.findByWorkflowId(checked.getKey().getProcessInstanceId());


[4/8] git commit: Merge branch '1_1_X' into 1_2_X

Posted by il...@apache.org.
Merge branch '1_1_X' into 1_2_X


Project: http://git-wip-us.apache.org/repos/asf/syncope/repo
Commit: http://git-wip-us.apache.org/repos/asf/syncope/commit/377963e8
Tree: http://git-wip-us.apache.org/repos/asf/syncope/tree/377963e8
Diff: http://git-wip-us.apache.org/repos/asf/syncope/diff/377963e8

Branch: refs/heads/1_2_X
Commit: 377963e869b67809ec5b22c5cb65a00328a1b281
Parents: bcc3e5b 296ffa3
Author: Francesco Chicchiriccò <il...@apache.org>
Authored: Wed Nov 5 10:50:08 2014 +0100
Committer: Francesco Chicchiriccò <il...@apache.org>
Committed: Wed Nov 5 10:50:08 2014 +0100

----------------------------------------------------------------------
 .../core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/syncope/blob/377963e8/core/src/main/java/org/apache/syncope/core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java
----------------------------------------------------------------------


[8/8] git commit: Merge branch '1_2_X'

Posted by il...@apache.org.
Merge branch '1_2_X'


Project: http://git-wip-us.apache.org/repos/asf/syncope/repo
Commit: http://git-wip-us.apache.org/repos/asf/syncope/commit/681cf9f5
Tree: http://git-wip-us.apache.org/repos/asf/syncope/tree/681cf9f5
Diff: http://git-wip-us.apache.org/repos/asf/syncope/diff/681cf9f5

Branch: refs/heads/master
Commit: 681cf9f5aafae60863c877a13d924ac2412ad8dc
Parents: a3ea02d 76b35a7
Author: Francesco Chicchiriccò <il...@apache.org>
Authored: Wed Nov 5 11:08:06 2014 +0100
Committer: Francesco Chicchiriccò <il...@apache.org>
Committed: Wed Nov 5 11:08:06 2014 +0100

----------------------------------------------------------------------
 .../activiti/ActivitiUserWorkflowAdapter.java   | 34 +++++++++++---------
 1 file changed, 19 insertions(+), 15 deletions(-)
----------------------------------------------------------------------



[6/8] git commit: [SYNCOPE-600] Adding miss pre-condition for form availability after form submit

Posted by il...@apache.org.
[SYNCOPE-600] Adding miss pre-condition for form availability after form submit


Project: http://git-wip-us.apache.org/repos/asf/syncope/repo
Commit: http://git-wip-us.apache.org/repos/asf/syncope/commit/76b35a78
Tree: http://git-wip-us.apache.org/repos/asf/syncope/tree/76b35a78
Diff: http://git-wip-us.apache.org/repos/asf/syncope/diff/76b35a78

Branch: refs/heads/master
Commit: 76b35a7870eee368d8b78e71d39bd9ecc2de5934
Parents: 377963e
Author: Francesco Chicchiriccò <il...@apache.org>
Authored: Wed Nov 5 11:07:41 2014 +0100
Committer: Francesco Chicchiriccò <il...@apache.org>
Committed: Wed Nov 5 11:07:41 2014 +0100

----------------------------------------------------------------------
 .../activiti/ActivitiUserWorkflowAdapter.java   | 32 +++++++++++---------
 1 file changed, 18 insertions(+), 14 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/syncope/blob/76b35a78/core/src/main/java/org/apache/syncope/core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/syncope/core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java b/core/src/main/java/org/apache/syncope/core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java
index fdf35de..4f0d876 100644
--- a/core/src/main/java/org/apache/syncope/core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java
+++ b/core/src/main/java/org/apache/syncope/core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java
@@ -208,8 +208,9 @@ public class ActivitiUserWorkflowAdapter extends AbstractUserWorkflowAdapter {
     private Set<String> getPerformedTasks(final SyncopeUser user) {
         final Set<String> result = new HashSet<String>();
 
-        for (HistoricActivityInstance task : historyService.createHistoricActivityInstanceQuery().executionId(user.
-                getWorkflowId()).list()) {
+        for (HistoricActivityInstance task
+                : historyService.createHistoricActivityInstanceQuery().executionId(user.getWorkflowId()).list()) {
+
             result.add(task.getActivityId());
         }
 
@@ -261,11 +262,11 @@ public class ActivitiUserWorkflowAdapter extends AbstractUserWorkflowAdapter {
             throwException(e, "While starting " + WF_PROCESS_ID + " instance");
         }
 
-        SyncopeUser user = (SyncopeUser) runtimeService.getVariable(
-                processInstance.getProcessInstanceId(), SYNCOPE_USER);
+        SyncopeUser user =
+                runtimeService.getVariable(processInstance.getProcessInstanceId(), SYNCOPE_USER, SyncopeUser.class);
 
-        Boolean updatedEnabled = (Boolean) runtimeService.getVariable(
-                processInstance.getProcessInstanceId(), ENABLED);
+        Boolean updatedEnabled =
+                runtimeService.getVariable(processInstance.getProcessInstanceId(), ENABLED, Boolean.class);
         if (updatedEnabled != null) {
             user.setSuspended(!updatedEnabled);
         }
@@ -278,8 +279,8 @@ public class ActivitiUserWorkflowAdapter extends AbstractUserWorkflowAdapter {
         updateStatus(user);
         user = userDAO.save(user);
 
-        Boolean propagateEnable = (Boolean) runtimeService.getVariable(
-                processInstance.getProcessInstanceId(), PROPAGATE_ENABLE);
+        Boolean propagateEnable =
+                runtimeService.getVariable(processInstance.getProcessInstanceId(), PROPAGATE_ENABLE, Boolean.class);
         if (propagateEnable == null) {
             propagateEnable = enabled;
         }
@@ -353,12 +354,12 @@ public class ActivitiUserWorkflowAdapter extends AbstractUserWorkflowAdapter {
         updateStatus(user);
         SyncopeUser updated = userDAO.save(user);
 
-        PropagationByResource propByRes = (PropagationByResource) runtimeService.getVariable(
-                user.getWorkflowId(), PROP_BY_RESOURCE);
+        PropagationByResource propByRes =
+                runtimeService.getVariable(user.getWorkflowId(), PROP_BY_RESOURCE, PropagationByResource.class);
 
         saveForFormSubmit(updated, userMod.getPassword(), propByRes);
 
-        Boolean propagateEnable = (Boolean) runtimeService.getVariable(user.getWorkflowId(), PROPAGATE_ENABLE);
+        Boolean propagateEnable = runtimeService.getVariable(user.getWorkflowId(), PROPAGATE_ENABLE, Boolean.class);
 
         return new WorkflowResult<Map.Entry<UserMod, Boolean>>(
                 new SimpleEntry<UserMod, Boolean>(userMod, propagateEnable), propByRes, tasks);
@@ -826,16 +827,19 @@ public class ActivitiUserWorkflowAdapter extends AbstractUserWorkflowAdapter {
 
         // see if there is any propagation to be done
         PropagationByResource propByRes =
-                (PropagationByResource) runtimeService.getVariable(user.getWorkflowId(), PROP_BY_RESOURCE);
+                runtimeService.getVariable(user.getWorkflowId(), PROP_BY_RESOURCE, PropagationByResource.class);
 
         // fetch - if available - the encrypted password
         String clearPassword = null;
-        String encryptedPwd = (String) runtimeService.getVariable(user.getWorkflowId(), ENCRYPTED_PWD);
+        String encryptedPwd = runtimeService.getVariable(user.getWorkflowId(), ENCRYPTED_PWD, String.class);
         if (StringUtils.isNotBlank(encryptedPwd)) {
             clearPassword = decrypt(encryptedPwd);
         }
 
-        UserMod userMod = (UserMod) runtimeService.getVariable(user.getWorkflowId(), USER_MOD);
+        // supports approval chains
+        saveForFormSubmit(user, clearPassword, propByRes);
+
+        UserMod userMod = runtimeService.getVariable(user.getWorkflowId(), USER_MOD, UserMod.class);
         if (userMod == null) {
             userMod = new UserMod();
             userMod.setId(updated.getId());


[5/8] git commit: Merge branch '1_1_X' into 1_2_X

Posted by il...@apache.org.
Merge branch '1_1_X' into 1_2_X


Project: http://git-wip-us.apache.org/repos/asf/syncope/repo
Commit: http://git-wip-us.apache.org/repos/asf/syncope/commit/377963e8
Tree: http://git-wip-us.apache.org/repos/asf/syncope/tree/377963e8
Diff: http://git-wip-us.apache.org/repos/asf/syncope/diff/377963e8

Branch: refs/heads/master
Commit: 377963e869b67809ec5b22c5cb65a00328a1b281
Parents: bcc3e5b 296ffa3
Author: Francesco Chicchiriccò <il...@apache.org>
Authored: Wed Nov 5 10:50:08 2014 +0100
Committer: Francesco Chicchiriccò <il...@apache.org>
Committed: Wed Nov 5 10:50:08 2014 +0100

----------------------------------------------------------------------
 .../core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/syncope/blob/377963e8/core/src/main/java/org/apache/syncope/core/workflow/user/activiti/ActivitiUserWorkflowAdapter.java
----------------------------------------------------------------------