findUsersByEmail includes duplicate records when user has internal and provided record
GENERAL
TESTING
GENERAL
TESTING
Description
relates to
Activity
Show:
Alexandre Ballesté Crevillén February 14, 2017 at 5:55 AM
Tested on Sakai - 11.3-RC01 2017/02/10 https://qa1-us.nightly.sakaiproject.org/ I've done the same neal did in trunk. I've did a search by email in User Membership and get no error. So I tested that there is no regression after applying the merge.
Neal Caidin December 15, 2016 at 9:29 AM
Logged in as admin I am able to search for an account I created by email address on
https://trunk-oracle.nightly.sakaiproject.org/portal/
with Mac/Chrome.
Sam Ottenhoff December 8, 2016 at 9:57 AM
I recommend a simple regression test. Can you still for a user by email address in a tool like User Membership?
Neal Caidin December 8, 2016 at 9:51 AM
Since we don't have a QA LDAP server, if we know this is used in production somewhere can we set this to Verified so it can be merged into the 11 branch?
To replicate:
1) Setup Sakai to use LDAP
2) Add internal record with identical email in SAKAI_USER table
3) Go to course site -> Site Info -> Add PArticipants
4) Try to add the user by email
Error: ''{0}'' matches multiple users in record. Please select from the following usernames: {1}
Problem is that findUserByEmail is adding both to a Collection instead of just using a unique Set