Fixed using the wrong type of Map for storing perm attachment values

This commit is contained in:
Nathan Adams 2011-10-27 22:03:21 +01:00
parent e33fce94b5
commit cf34c6c305

View File

@ -1,8 +1,8 @@
package org.bukkit.permissions; package org.bukkit.permissions;
import java.util.LinkedHashMap;
import java.util.Map; import java.util.Map;
import java.util.TreeMap;
import org.bukkit.plugin.Plugin; import org.bukkit.plugin.Plugin;
/** /**
@ -10,7 +10,7 @@ import org.bukkit.plugin.Plugin;
*/ */
public class PermissionAttachment { public class PermissionAttachment {
private PermissionRemovedExecutor removed; private PermissionRemovedExecutor removed;
private final TreeMap<String, Boolean> permissions = new TreeMap<String, Boolean>(); private final Map<String, Boolean> permissions = new LinkedHashMap<String, Boolean>();
private final Permissible permissible; private final Permissible permissible;
private final Plugin plugin; private final Plugin plugin;
@ -69,7 +69,7 @@ public class PermissionAttachment {
* @return Copy of all permissions and values expressed by this attachment * @return Copy of all permissions and values expressed by this attachment
*/ */
public Map<String, Boolean> getPermissions() { public Map<String, Boolean> getPermissions() {
return (Map<String, Boolean>)permissions.clone(); return new LinkedHashMap<String, Boolean>(permissions);
} }
/** /**