minecraft/Essentials
1
0
Fork 0
mirror of https://github.com/essentials/Essentials.git synced 2025-08-03 21:28:00 +02:00
Code Issues Releases Wiki Activity

Throw a better error than 'null' when someone removes all groups from a

Browse Source 
yml.
This commit is contained in:
ElgarL
2012-07-06 08:33:57 +01:00
parent 017136ff1c
commit fa7688b4b5
2 changed files with 6 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
EssentialsGroupManager/src/Changelog.txt
View File

@@ -187,3 +187,4 @@ v 2.0:
- Only output a Data update message if something has changed. - Only output a Data update message if something has changed.
- Fix loading users with only numerals in their names to be seen as strings. - Fix loading users with only numerals in their names to be seen as strings.
- Ignore any sub folders in the Worlds folder which start with a period (fix for storing data in svn respoitories). - Ignore any sub folders in the Worlds folder which start with a period (fix for storing data in svn respoitories).
- Throw a better error than 'null' when someone removes all groups from a yml.

4
EssentialsGroupManager/src/org/anjocaido/groupmanager/dataholder/WorldDataHolder.java
View File

@@ -509,6 +509,10 @@ public class WorldDataHolder {
throw new IllegalArgumentException("Your " + groupsFile.getPath() + " file is invalid. See console for details.", ex); throw new IllegalArgumentException("Your " + groupsFile.getPath() + " file is invalid. See console for details.", ex);
} }
if (allGroupsNode == null) {
throw new IllegalArgumentException("You have no groups in " + groupsFile.getPath() + ".");
}
Iterator<String> groupItr = allGroupsNode.keySet().iterator(); Iterator<String> groupItr = allGroupsNode.keySet().iterator();
String groupKey; String groupKey;
Integer groupCount = 0; Integer groupCount = 0;