getter / setters seems fine ... also there is no null case.
However, an interesting observation; this gives me an error:
@PreAuthorize("canIEditGroupProfile(#membership.group.id)") public int updateGroupMembership(GroupMembership membership) throws GroupsServiceException;
This works great:
@PreAuthorize("canIEditGroupProfile(#groupmembership.group.id)") public int updateGroupMembership(GroupMembership groupmembership) throws GroupsServiceException;
Further, I noticed that the parameter name was a mismatch in the first case (for example, Service and ServiceImpl had different parameter names).
Now, while maintaining uniformity, the problem seems to be fixed.
source share