diff options
author | ElgarL <ElgarL@Palmergames.com> | 2014-04-14 23:40:04 +0100 |
---|---|---|
committer | KHobbits <rob@khobbits.co.uk> | 2014-05-04 12:50:20 +0100 |
commit | aae586f79ac177734eefac9e786dc5fc50dabb43 (patch) | |
tree | 2be8507791fca5e465c43c3126be6f3086fce5c7 /EssentialsGroupManager/src/org/anjocaido/groupmanager/data | |
parent | 32f49350cf7629b741539bdc4c68e7edce78f171 (diff) | |
download | Essentials-aae586f79ac177734eefac9e786dc5fc50dabb43.tar Essentials-aae586f79ac177734eefac9e786dc5fc50dabb43.tar.gz Essentials-aae586f79ac177734eefac9e786dc5fc50dabb43.tar.lz Essentials-aae586f79ac177734eefac9e786dc5fc50dabb43.tar.xz Essentials-aae586f79ac177734eefac9e786dc5fc50dabb43.zip |
Fix overloadedWorldHolder so it correctly calls the new WorldHolder code when the user is not overloaded.
Diffstat (limited to 'EssentialsGroupManager/src/org/anjocaido/groupmanager/data')
-rw-r--r-- | EssentialsGroupManager/src/org/anjocaido/groupmanager/data/DataUnit.java | 6 | ||||
-rw-r--r-- | EssentialsGroupManager/src/org/anjocaido/groupmanager/data/User.java | 18 |
2 files changed, 11 insertions, 13 deletions
diff --git a/EssentialsGroupManager/src/org/anjocaido/groupmanager/data/DataUnit.java b/EssentialsGroupManager/src/org/anjocaido/groupmanager/data/DataUnit.java index 36c26bb1d..90fd28e8d 100644 --- a/EssentialsGroupManager/src/org/anjocaido/groupmanager/data/DataUnit.java +++ b/EssentialsGroupManager/src/org/anjocaido/groupmanager/data/DataUnit.java @@ -100,14 +100,14 @@ public abstract class DataUnit { public String getLastName() { if (uUID.length() < 36) - return uUID; + return this.uUID; - return lastName; + return this.lastName; } public void setLastName(String lastName) { - if (!this.lastName.equals(lastName)) { + if (!lastName.equals(this.lastName)) { this.lastName = lastName; changed = true; diff --git a/EssentialsGroupManager/src/org/anjocaido/groupmanager/data/User.java b/EssentialsGroupManager/src/org/anjocaido/groupmanager/data/User.java index e6d11274f..38663ba7d 100644 --- a/EssentialsGroupManager/src/org/anjocaido/groupmanager/data/User.java +++ b/EssentialsGroupManager/src/org/anjocaido/groupmanager/data/User.java @@ -69,10 +69,10 @@ public class User extends DataUnit implements Cloneable { */ public User clone(WorldDataHolder dataSource) { - if (dataSource.isUserDeclared(this.getLastName())) { + if (dataSource.isUserDeclared(this.getUUID())) { return null; } - User clone = dataSource.createUser(this.getLastName()); + User clone = dataSource.createUser(this.getUUID()); if (dataSource.getGroup(group) == null) { clone.setGroup(dataSource.getDefaultGroup()); } else { @@ -86,17 +86,14 @@ public class User extends DataUnit implements Cloneable { return clone; } - public User clone(String uUID) { + public User clone(String uUID, String CurrentName) { User clone = this.getDataSource().createUser(uUID); - if (this.getDataSource().getGroup(group) == null) { - clone.setGroup(this.getDataSource().getDefaultGroup()); - } else { - clone.setGroup(this.getDataSource().getGroup(this.getGroupName())); - } + clone.setLastName(CurrentName); - clone.setLastName(this.getLastName()); + // Set the group silently. + clone.setGroup(this.getDataSource().getGroup(this.getGroupName()), false); for (String perm : this.getPermissionList()) { clone.addPermission(perm); @@ -181,7 +178,8 @@ public class User extends DataUnit implements Cloneable { if (notify) GroupManager.notify(this.getLastName(), String.format(" moved to the group %s in %s.", group.getName(), this.getDataSource().getName())); - GroupManager.getGMEventHandler().callEvent(this, Action.USER_GROUP_CHANGED); + if (updatePerms) + GroupManager.getGMEventHandler().callEvent(this, Action.USER_GROUP_CHANGED); } } |