This content has been marked as final.
Show 1 reply
-
1. Re: ExpressionAssignmentHandlerTest.testNextTermMember error
bnsbr Dec 13, 2006 6:32 AM (in response to 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?