diff options
author | ElgarL <ElgarL@palmergames.com> | 2012-04-10 17:02:50 +0100 |
---|---|---|
committer | ElgarL <ElgarL@palmergames.com> | 2012-04-10 17:02:50 +0100 |
commit | 74534d7781850f4cbb854c084911503c12f13cdf (patch) | |
tree | 9e4139085fbd0519ed0f3b8c7f1f22084afc8ef1 /EssentialsGroupManager/src/org | |
parent | cfda56697b7b077b6c7512936ccda8f50f638bcc (diff) | |
download | Essentials-74534d7781850f4cbb854c084911503c12f13cdf.tar Essentials-74534d7781850f4cbb854c084911503c12f13cdf.tar.gz Essentials-74534d7781850f4cbb854c084911503c12f13cdf.tar.lz Essentials-74534d7781850f4cbb854c084911503c12f13cdf.tar.xz Essentials-74534d7781850f4cbb854c084911503c12f13cdf.zip |
Properly fix concurrent modification when removing all attachments.
Diffstat (limited to 'EssentialsGroupManager/src/org')
-rw-r--r-- | EssentialsGroupManager/src/org/anjocaido/groupmanager/permissions/BukkitPermissions.java | 13 |
1 files changed, 11 insertions, 2 deletions
diff --git a/EssentialsGroupManager/src/org/anjocaido/groupmanager/permissions/BukkitPermissions.java b/EssentialsGroupManager/src/org/anjocaido/groupmanager/permissions/BukkitPermissions.java index 51f019efc..516679544 100644 --- a/EssentialsGroupManager/src/org/anjocaido/groupmanager/permissions/BukkitPermissions.java +++ b/EssentialsGroupManager/src/org/anjocaido/groupmanager/permissions/BukkitPermissions.java @@ -357,7 +357,7 @@ public class BukkitPermissions { } catch (IllegalArgumentException e) {
/*
* Failed to remove attachment
- * This usually means Bukkit no longer know of it.
+ * This usually means Bukkit no longer knows of it.
*/
}
attachments.remove(player);
@@ -372,8 +372,17 @@ public class BukkitPermissions { Iterator<Player> itr = attachments.keySet().iterator();
while (itr.hasNext()){
- removeAttachment(itr.next());
+ Player player = itr.next();
+ try {
+ player.removeAttachment(attachments.get(player));
+ } catch (IllegalArgumentException e) {
+ /*
+ * Failed to remove attachment
+ * This usually means Bukkit no longer knows of it.
+ */
+ }
}
+ attachments.clear();
}
/**
|