Skip to content

Commit c6dfa74

Browse files
committed
Case sensitive works, so make all Users names case insensitive as no duplicate names will be allowed.
1 parent e36f26e commit c6dfa74

File tree

1 file changed

+28
-28
lines changed

1 file changed

+28
-28
lines changed

EssentialsGroupManager/src/org/anjocaido/groupmanager/dataholder/WorldDataHolder.java

Lines changed: 28 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -118,26 +118,26 @@ public void updateDataSource() {
118118
*/
119119
public User getUser(String userId) {
120120

121-
if (getUsers().containsKey(userId)) {
122-
return getUsers().get(userId);
121+
if (getUsers().containsKey(userId.toLowerCase())) {
122+
return getUsers().get(userId.toLowerCase());
123123
}
124124

125125
// Legacy name matching
126-
if ((userId.length() < 36) && nameToUUIDLookup.containsKey(userId)) {
126+
if ((userId.length() < 36) && nameToUUIDLookup.containsKey(userId.toLowerCase())) {
127127

128128
// Search for a name to UUID match
129-
for (String uid : getUUIDLookup(userId)) {
129+
for (String uid : getUUIDLookup(userId.toLowerCase())) {
130130

131-
User user = getUsers().get(uid);
131+
User user = getUsers().get(uid.toLowerCase());
132132

133-
if ((user != null) && user.getLastName().equals(userId)) {
133+
if ((user != null) && user.getLastName().equalsIgnoreCase(userId)) {
134134
return user;
135135
}
136136
}
137137

138138
}
139139

140-
if (!nameToUUIDLookup.containsKey(userId)) {
140+
if (!nameToUUIDLookup.containsKey(userId.toLowerCase())) {
141141
GroupManager.logger.fine("ERROR: No lookup for: " + userId);
142142
}
143143

@@ -157,7 +157,7 @@ public User getUser(String userId) {
157157
public User getUser(String uUID, String currentName) {
158158

159159
// Check for a UUID account
160-
User user = getUsers().get(uUID);
160+
User user = getUsers().get(uUID.toLowerCase());
161161

162162
if (user != null) {
163163

@@ -168,28 +168,28 @@ public User getUser(String uUID, String currentName) {
168168
}
169169

170170
// Search for a LastName match
171-
user = getUsers().get(currentName);
171+
user = getUsers().get(currentName.toLowerCase());
172172

173-
if ((user != null) && user.getLastName().equals(currentName) && user.getUUID().equals(user.getLastName())) {
173+
if ((user != null) && user.getLastName().equalsIgnoreCase(currentName) && user.getUUID().equalsIgnoreCase(user.getLastName())) {
174174

175175
// Clone this user so we can set it's uUID
176176
User usr = user.clone(uUID, currentName);
177177

178178
// Delete it and replace with the new clone.
179-
this.removeUser(user.getUUID());
179+
this.removeUser(user.getUUID().toLowerCase());
180180
this.addUser(usr);
181181

182182
GroupManager.logger.fine("Updating User record for UUID: " + uUID + ":" + currentName);
183183

184-
return getUsers().get(uUID);
184+
return getUsers().get(uUID.toLowerCase());
185185
}
186186

187187
if (user != null) {
188188
GroupManager.logger.fine("User record found but UUID mismatch for: " + currentName);
189189
}
190190

191191
// No user account found so create a new one.
192-
User newUser = createUser(uUID);
192+
User newUser = createUser(uUID.toLowerCase());
193193
newUser.setLastName(currentName);
194194

195195
GroupManager.logger.fine("New User record created: " + uUID + ":" + currentName);
@@ -213,8 +213,8 @@ public void addUser(User theUser) {
213213
if ((theUser.getGroup() == null)) {
214214
theUser.setGroup(groups.getDefaultGroup());
215215
}
216-
removeUser(theUser.getUUID());
217-
getUsers().put(theUser.getUUID(), theUser);
216+
removeUser(theUser.getUUID().toLowerCase());
217+
getUsers().put(theUser.getUUID().toLowerCase(), theUser);
218218

219219
// Store for name to UUID lookups.
220220
//putUUIDLookup(theUser.getLastName(), theUser.getUUID().toLowerCase());
@@ -232,14 +232,14 @@ public void addUser(User theUser) {
232232
*/
233233
public boolean removeUser(String userId) {
234234

235-
if (getUsers().containsKey(userId)) {
235+
if (getUsers().containsKey(userId.toLowerCase())) {
236236

237-
User user = getUser(userId);
237+
User user = getUser(userId.toLowerCase());
238238

239239
// Remove the name to UUID lookup for this user object.
240-
removeUUIDLookup(user.getLastName(), user.getUUID());
240+
removeUUIDLookup(user.getLastName().toLowerCase(), user.getUUID());
241241

242-
getUsers().remove(userId);
242+
getUsers().remove(userId.toLowerCase());
243243

244244
setUsersChanged(true);
245245

@@ -258,7 +258,7 @@ public boolean removeUser(String userId) {
258258
*/
259259
public boolean isUserDeclared(String userId) {
260260

261-
return getUsers().containsKey(userId);
261+
return getUsers().containsKey(userId.toLowerCase());
262262
}
263263

264264
/**
@@ -374,7 +374,7 @@ public boolean removeGroup(String groupName) {
374374
*/
375375
public User createUser(String userId) {
376376

377-
if (getUsers().containsKey(userId)) {
377+
if (getUsers().containsKey(userId.toLowerCase())) {
378378
return null;
379379
}
380380
User newUser = new User(this, userId);
@@ -1489,14 +1489,14 @@ public void setTimeStamps() {
14891489
*/
14901490
public void putUUIDLookup(String name, String UUID) {
14911491

1492-
Set<String> lookup = getUUIDLookup(name);
1492+
Set<String> lookup = getUUIDLookup(name.toLowerCase());
14931493

14941494
if (lookup == null)
14951495
lookup = new TreeSet<String>();
14961496

14971497
lookup.add(UUID);
14981498

1499-
nameToUUIDLookup.put(name, lookup);
1499+
nameToUUIDLookup.put(name.toLowerCase(), lookup);
15001500
}
15011501

15021502
/**
@@ -1508,18 +1508,18 @@ public void putUUIDLookup(String name, String UUID) {
15081508
*/
15091509
public void removeUUIDLookup(String name, String UUID) {
15101510

1511-
if (nameToUUIDLookup.containsKey(name)) {
1511+
if (nameToUUIDLookup.containsKey(name.toLowerCase())) {
15121512

1513-
Set<String> lookup = getUUIDLookup(name);
1513+
Set<String> lookup = getUUIDLookup(name.toLowerCase());
15141514

15151515
lookup.remove(UUID);
15161516

15171517
if (lookup.isEmpty()) {
1518-
nameToUUIDLookup.remove(name);
1518+
nameToUUIDLookup.remove(name.toLowerCase());
15191519
return;
15201520
}
15211521

1522-
nameToUUIDLookup.put(name, lookup);
1522+
nameToUUIDLookup.put(name.toLowerCase(), lookup);
15231523

15241524
}
15251525

@@ -1532,7 +1532,7 @@ public void removeUUIDLookup(String name, String UUID) {
15321532
*/
15331533
public Set<String> getUUIDLookup(String name) {
15341534

1535-
return nameToUUIDLookup.get(name);
1535+
return nameToUUIDLookup.get(name.toLowerCase());
15361536
}
15371537

15381538
/**

0 commit comments

Comments
 (0)