major
nice-to-have
defect
major
enhancement
minor
#23221
Utilities for group membership
The code ` group.getMembers().contains(person)`, which is supposedly used to determine whether someone belongs to a group
- Code bloat, because a set is created unnecessarily just to perform a single test and then hand the whole thing over to the garbage collector.
- Incorrect, because the person for whom this group might be the “representative group” is not included in the group’s “members.”
Instead: group.containsPerson(person) or person.isInGroup(group).
Test
- test.com.top_logic.tool.boundsec.wrap.TestGroup.testIsInRepresentativeGroup()