Comment 4 for bug 1281750

Revision history for this message
Remington Steed (rjs7) wrote :

Tested and works as expected. I didn't test the heavy load, but since this code prevents retrieving the ids of deleted users, it therefore cannot subsequently fetch other data from those users. Here are my test results:

request open-ils.actor open-ils.actor.usergroup.members.retrieve.authoritative "AUTHTOKEN", 0
-- BEFORE: returns list of user ids
-- AFTER: returns empty list

request open-ils.actor open-ils.actor.usergroup.leaders.retrieve "AUTHTOKEN", 0
-- BEFORE: returns list of user ids
-- AFTER: returns empty list

I couldn't test the other two (usergroup.members.balance_owed and address.members) in the same way, but it seems clear that the code will work in all four cases. So here is my signoff branch.

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/rsteed/lp1281750-ignore-usergroup-0

working/user/rsteed/lp1281750-ignore-usergroup-0