KNL-523 introduced separate events for joining and unjoining a site. However there is a bug in this code. The 'realm.join' or 'realm.unjoin' event is only added if the event is null, which it never is:
BaseAuthzGroupService. joinGroup method has this: ((BaseAuthzGroup) azGroup).setEvent(SECURE_UPDATE_OWN_AUTHZ_GROUP);
which then calls addMemberToAuthzGroup which has this:
So the event string will always be set as the first one, whidh is realm.upd.own, and not realm.join.
This is a problem in both joining and unjoining and the fix is the same, remove the first setEvent call and have each individual method set the correct event.
bulk close using query: "1.3.x status" in (Resolved, Closed) and status in (Resolved, Verified) and project = "KNL" and fixVersion = "1.3.0" and resolution = Fixed
KNL-523 introduced separate events for joining and unjoining a site. However there is a bug in this code. The 'realm.join' or 'realm.unjoin' event is only added if the event is null, which it never is:
BaseAuthzGroupService. joinGroup method has this:
((BaseAuthzGroup) azGroup).setEvent(SECURE_UPDATE_OWN_AUTHZ_GROUP);
which then calls addMemberToAuthzGroup which has this:
String event = ((BaseAuthzGroup) azGroup).getEvent();
if (event == null) event = SECURE_JOIN_AUTHZ_GROUP;
So the event string will always be set as the first one, whidh is realm.upd.own, and not realm.join.
This is a problem in both joining and unjoining and the fix is the same, remove the first setEvent call and have each individual method set the correct event.