From 566f5ff327499debe19ab7b2c70f1eb665993d01 Mon Sep 17 00:00:00 2001
From: ElgarL <ElgarL@palmergames.com>
Date: Thu, 5 Apr 2012 15:02:08 +0100
Subject: Fix an error I caused trying to modify an unmodifiable list when
 parsing '*' permissions.

---
 EssentialsGroupManager/src/Changelog.txt                              | 3 ++-
 .../anjocaido/groupmanager/permissions/AnjoPermissionsHandler.java    | 4 +++-
 2 files changed, 5 insertions(+), 2 deletions(-)

(limited to 'EssentialsGroupManager')

diff --git a/EssentialsGroupManager/src/Changelog.txt b/EssentialsGroupManager/src/Changelog.txt
index a636ac6fe..c5fe5564d 100644
--- a/EssentialsGroupManager/src/Changelog.txt
+++ b/EssentialsGroupManager/src/Changelog.txt
@@ -162,4 +162,5 @@ v 1.9:
 	  using GroupManager commands (they will still require each commands permission node to use them).
 	- Prevent Null entries in group inheritance from throwing errors.
 v 2.0:
-	- Fix GM reporting of permission inheritance to retain the correct order. Lower inheritance groups can no longer negate a higher groups permissions.
\ No newline at end of file
+	- Fix GM reporting of permission inheritance to retain the correct order. Lower inheritance groups can no longer negate a higher groups permissions.
+	- Fix an error I caused trying to modify an unmodifiable list when parsing '*' permissions.
\ No newline at end of file
diff --git a/EssentialsGroupManager/src/org/anjocaido/groupmanager/permissions/AnjoPermissionsHandler.java b/EssentialsGroupManager/src/org/anjocaido/groupmanager/permissions/AnjoPermissionsHandler.java
index 08240441d..b40694bb0 100644
--- a/EssentialsGroupManager/src/org/anjocaido/groupmanager/permissions/AnjoPermissionsHandler.java
+++ b/EssentialsGroupManager/src/org/anjocaido/groupmanager/permissions/AnjoPermissionsHandler.java
@@ -155,8 +155,10 @@ public class AnjoPermissionsHandler extends PermissionsReaderInterface {
 		return playerPermArray;
 	}
 	
-	private Set<String> populatePerms (List<String>  perms, boolean includeChildren) {
+	private Set<String> populatePerms (List<String>  permsList, boolean includeChildren) {
 		
+		// Create a new array so it's modifiable.
+		List<String> perms = new ArrayList<String>(permsList);
 		Set<String> permArray = new HashSet<String>();
 		Boolean allPerms = false;
 		
-- 
cgit v1.2.3