Prevent Group.equals tests throwing a NullPointerException for

GlobalGroups.
This commit is contained in:
ElgarL 2011-11-19 20:46:34 +00:00
parent 7b7d1f557f
commit 317b0833ef
3 changed files with 7 additions and 3 deletions

View file

@ -72,4 +72,6 @@ v 1.5:
- Fix for an error in checkFullUserPermission caused by players disconnecting mid perms update.
- Notification of being moved to the default group only happens if it's a demotion/promotion (not on join).
- Fixed GM holding files open and causing the time stamp to be incorrect. This caused GM to require a '/mansave force' when it shouldn't be needed.
- Fixed a crash on reload due to bukkit not unloading plugins before reloading.
- Fixed a crash on reload due to bukkit not unloading plugins before reloading.
v 1.6:
- Prevent Group.equals tests throwing a NullPointerException for GlobalGroups.

View file

@ -39,7 +39,9 @@ public abstract class DataUnit {
public boolean equals(Object o) {
if (o instanceof DataUnit) {
DataUnit go = (DataUnit) o;
if (this.getName().equalsIgnoreCase(go.getName()) && this.dataSource.getName().equalsIgnoreCase(go.getDataSource().getName())) {
if (this.getName().equalsIgnoreCase(go.getName())
&& ((this.dataSource == null && go.getDataSource() == null)
|| (this.dataSource.getName().equalsIgnoreCase(go.getDataSource().getName())))) {
return true;
}
}

View file

@ -1,5 +1,5 @@
name: GroupManager
version: "1.5 (Phoenix)"
version: "1.6 (Phoenix)"
main: org.anjocaido.groupmanager.GroupManager
website: http://www.anjocaido.info/
description: Provides on-the-fly system for permissions system created by Nijikokun. But all in memory, and with flat-file saving schedule.