Incorrect behavior when adding participants "matches multiple users in record", Part II

Description

Test Plan...
1) Create 2 users in sakai with the same email address and a user with that email address as their id
2) Using Site Editor, Add participants add user with email
3) It should give you a message like below:
'tester1@testme.com' matches multiple users in record. Please select from the following usernames: tester1@testme.com, tester1
and it should display all user ids in the Official Email Address or Username box.
4) You should be able to pick the user id (delete the others) and add the participant

Attachments

2

Activity

Show:

Zhen Qian August 11, 2010 at 6:45 AM

Applied the cle-18874.patch in r81060. Thanks!

Sam Ottenhoff August 10, 2010 at 2:48 PM

Patch that stops looking up by email if there is a user with an eid that is the email address.

Sam Ottenhoff August 10, 2010 at 2:46 PM

Thanks for the patch Brooke. Zhen, do you agree with the patch? I also have a patch to skip checking emails if the email is an eid.

Brooke Biltimier August 10, 2010 at 2:38 PM

We have a patch for this issue which I will attach to the ticket. Basically all we did was change the severity of the multiple users on record message to SEVERITY_INFO. This allowed the changed input of the TextArea to be updated.

Sam Ottenhoff August 10, 2010 at 2:18 PM

My two cents on how this should function:

1) If a user is found when looking up by an eid that is an email address, go with that user. Stop doing lookups.

2) If you need to rely on doing an email lookup, there is clearly something broken in 2.7.x/2.7.1. The user is not able to proceed past the alert message.

Fixed

Details

Priority

Affects versions

Fix versions

Components

Assignee

Reporter

Created July 30, 2010 at 9:04 AM
Updated March 27, 2012 at 7:01 AM
Resolved August 11, 2010 at 6:45 AM