[jboss-user] [JBoss jBPM] - Re: ExpressionAssignmentHandlerTest.testNextTermMember error

[bnsbr]

I think the error is in the factory of Membership.

The 'name' field shouldn't be the 'role' of the user?
It's actually implemented as:

  |   public static Membership create(User user, String name, Group group) {
  |     Membership membership = create(user, group);
  |     membership.name = name;
  |     return membership;
  |   }
  | 

I think that it should be:


  |   public static Membership create(User user, String role, Group group) {
  |     Membership membership = create(user, group);
  |     membership.role = role;
  |     return membership;
  |   }
  | 

Or should we have 3 factories? Is there any reason to be using the 'name' field instead of the 'role' field?
Can anyone comment on this?

View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=3993348#3993348

Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=3993348