AfflictionNetworks/ooo-user
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

We have a lower case email index, we should be using it for lookup.

Browse Source
This commit is contained in:
Mark Johnson
2012-07-12 17:48:40 -07:00
parent c3e151c30b
commit 0968284e69
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/main/java/com/threerings/user/depot/DepotUserRepository.java
+3 -3
View File
@@ -145,8 +145,8 @@ public class DepotUserRepository extends DepotRepository
public Iterable<OOOUser> lookupUsersByEmail (String email) public Iterable<OOOUser> lookupUsersByEmail (String email)
{ {
List<OOOUser> users = Lists.newArrayList(); List<OOOUser> users = Lists.newArrayList();
for (OOOUserRecord record : findAll(OOOUserRecord.class, Where where = new Where(StringFuncs.lower(OOOUserRecord.EMAIL).eq(email.toLowerCase()));
new Where(OOOUserRecord.EMAIL, email))) { for (OOOUserRecord record : findAll(OOOUserRecord.class, where)) {
users.add(toUser(record)); users.add(toUser(record));
} }
return users; return users;
@@ -289,7 +289,7 @@ public class DepotUserRepository extends DepotRepository
public String[] getUsernames (String email) public String[] getUsernames (String email)
{ {
List<String> usernames = Lists.newArrayList(); List<String> usernames = Lists.newArrayList();
Where where = new Where(OOOUserRecord.EMAIL, email); Where where = new Where(StringFuncs.lower(OOOUserRecord.EMAIL).eq(email.toLowerCase()));
for (OOOUserRecord record : findAll(OOOUserRecord.class, where)) { for (OOOUserRecord record : findAll(OOOUserRecord.class, where)) {
usernames.add(record.username); usernames.add(record.username);
} }