CandyShop / gerrit

Automatically exported from code.google.com/p/gerrit
Apache License 2.0
1 stars 0 forks source link

wrong reviewer suggestion with suggest.fullTextSearch #3435

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
Affected Version:
2.11.1

What steps will reproduce the problem?
1. Have users register an email address with Upper-case chars
2. start typing their name into the reviewers field of a change
3. they will be suggested, but with a lower-cased address
4. hitting return / clicking on "Add" just displays the the following error 
message:
   "<user> does not identify a registered user or group"
5. it works after manually editing the suggested email address to the correct 
case

What is the expected output? What do you see instead?
Reviewers should always be suggested with the correct address.
As Gerrit is case-sensitive w.r.t. email addresses, this should be honored by 
the suggestion, too.

Original issue reported on code.google.com by Martin.W...@gmail.com on 15 Jun 2015 at 2:02

GoogleCodeExporter commented 9 years ago
We are facing the same issue.

Quick workaround was to let the user register the email once more with 
lowercase letters.

Original comment by s...@imsand.li on 19 Jun 2015 at 7:16

GoogleCodeExporter commented 9 years ago

Original comment by david.pu...@sonymobile.com on 25 Jun 2015 at 2:47

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
sasa.zivkov@sap.com rewrote the full text search parts for 2.11.1 [1]

/Sven

[1] https://gerrit-review.googlesource.com/#/c/67570/

Original comment by sven.sel...@sonymobile.com on 25 Jun 2015 at 9:00