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

[Luk Zdobylak (JIRA)]

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

Luk Zdobylak updated JBPM-788:
------------------------------

    Description: 
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);
      }

  was:
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.


> [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