In this example, all of the /api/Objects/Get/Clearance/<id> are failing because the responses are arrays and not single objects. The memberships should all be in sync; it should not be deleting all 7 memberships.

If the custom API class sets the targetGroup object to null instead of throwing an exception, this somehow works and doesn't delete anything. I guess it gets the previous objects from the cache? Why doesn't the exception version do that too? Failing to get the group shouldn't be treated as the group being deleted. It should set all the membership syncs for that group as an error or something.

 

 

Fix: Added a check in the compareTargetMemberships method to not delete memberships when group or user wrapper has an error code.