TotalFreedomMC/TF-EssentialsX
1
0
Fork 0
mirror of https://github.com/TotalFreedomMC/TF-EssentialsX.git synced 2024-09-12 05:08:48 +00:00
Code Issues Projects Releases Wiki Activity

Properly fix concurrent modification when removing all attachments.

Browse source
This commit is contained in:
ElgarL 2012-04-10 17:02:50 +01:00
parent ec4f5cc34d
commit f02691863b
1 changed files with 11 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
EssentialsGroupManager/src/org/anjocaido/groupmanager/permissions/BukkitPermissions.java
View file

@ -357,7 +357,7 @@ public class BukkitPermissions {
} catch (IllegalArgumentException e) { } catch (IllegalArgumentException e) {
/* /*
* Failed to remove attachment * 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); attachments.remove(player);
@ -372,9 +372,18 @@ public class BukkitPermissions {
Iterator<Player> itr = attachments.keySet().iterator(); Iterator<Player> itr = attachments.keySet().iterator();
while (itr.hasNext()){ 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();
}
/** /**
* Player events tracked to cause Superperms updates * Player events tracked to cause Superperms updates