[jboss-jira] [JBoss JIRA] Closed: (JBPM-788) [Identity component] ExpressionAssigmentHandler assigning group to pooled actors assigns group name not actors

[Ronald van Kuijk (JIRA)]

     [ http://jira.jboss.com/jira/browse/JBPM-788?page=all ]

Ronald van Kuijk closed JBPM-788.
---------------------------------

    Resolution: Rejected

No, the group should be in there and the membership is decided at the time the task is retrieved, not when it is assigned. The word 'pooled actors' is somewhat misleading since an actor (for jBPM core) is just a string of anything (groupname, individual user, etc...)

> [Identity component] ExpressionAssigmentHandler assigning group to pooled actors assigns group name not actors
> --------------------------------------------------------------------------------------------------------------
>
>                 Key: JBPM-788
>                 URL: http://jira.jboss.com/jira/browse/JBPM-788
>             Project: JBoss jBPM
>          Issue Type: Bug
>          Components: Core Engine
>            Reporter: Luk Zdobylak
>         Assigned To: Tom Baeyens
>
> FIle ExpressionAssigmentHandler.java
> from package org.jbpm.identity.assignment 
> lines from 91-94 
> } else if (entity instanceof Group) {
>         // put the group in the pool
>         assignable.setPooledActors(new String []{entity.getName()});
> }
> It seems that in line :         assignable.setPooledActors(new String []{entity.getName()});
> should be put not entity name but there should be code that retreives all actors assigned to group and prepares String[] array of their ids.
> sollution code to insert instead of code above:
> else if (entity instanceof Group) {
>         // put the group in the pool
>           Group g=(Group)entity;
>           Iterator i=g.getUsers().iterator();
>           String[] actorids=new String[g.getUsers().size()];
>           int j=0;
>         while (i.hasNext()) {
>             User user = (User) i.next();
>             actorids[j++]=user.getName();
>         }  
>         assignable.setPooledActors(actorids);
>       }

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira