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
_______________________________________________
jboss-user mailing list
[email protected]
https://lists.jboss.org/mailman/listinfo/jboss-user

Reply via email to