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.
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