mirror of
https://git.krews.org/morningstar/Arcturus-Community.git
synced 2024-11-26 16:30:52 +01:00
Fix NullPointerException when rank doesn't exist. Renamed some methods to be more descriptive.
This commit is contained in:
parent
ea5811800d
commit
8254e7de61
@ -41,7 +41,7 @@ public class GiveRankCommand extends Command
|
||||
}
|
||||
else
|
||||
{
|
||||
rank = Emulator.getGameEnvironment().getPermissionsManager().getRank(params[2]);
|
||||
rank = Emulator.getGameEnvironment().getPermissionsManager().getRankByName(params[2]);
|
||||
}
|
||||
|
||||
if (rank != null)
|
||||
|
@ -108,7 +108,7 @@ public class PermissionsManager
|
||||
}
|
||||
|
||||
|
||||
public Rank getRank(String rankName)
|
||||
public Rank getRankByName(String rankName)
|
||||
{
|
||||
for (Rank rank : this.ranks.valueCollection())
|
||||
{
|
||||
@ -161,8 +161,12 @@ public class PermissionsManager
|
||||
return this.badges.keySet();
|
||||
}
|
||||
|
||||
public List<Rank> getRanks(String code)
|
||||
public List<Rank> getRanksByBadgeCode(String code)
|
||||
{
|
||||
return this.badges.get(code);
|
||||
}
|
||||
|
||||
public List<Rank> getAllRanks() {
|
||||
return new ArrayList<>(this.ranks.valueCollection());
|
||||
}
|
||||
}
|
||||
|
@ -80,6 +80,8 @@ public class HabboInfo implements Runnable
|
||||
if (this.rank == null)
|
||||
{
|
||||
Emulator.getLogging().logErrorLine("No existing rank found with id " + set.getInt("rank") + ". Make sure an entry in the permissions table exists.");
|
||||
Emulator.getLogging().logUserLine(this.username + " has an invalid rank with id " + set.getInt("rank") + ". Make sure an entry in the permissions table exists.");
|
||||
this.rank = Emulator.getGameEnvironment().getPermissionsManager().getRank(1);
|
||||
}
|
||||
|
||||
this.accountCreated = set.getInt("account_created");
|
||||
@ -514,7 +516,7 @@ public class HabboInfo implements Runnable
|
||||
statement.setInt(6, Emulator.getIntUnixTimestamp());
|
||||
statement.setInt(8, this.homeRoom);
|
||||
statement.setString(9, this.ipLogin);
|
||||
statement.setInt(10, this.rank.getId());
|
||||
statement.setInt(10, this.rank != null ? this.rank.getId() : 1);
|
||||
statement.setString(11, this.machineID);
|
||||
statement.setString(12, this.username);
|
||||
statement.setInt(13, this.id);
|
||||
|
@ -41,7 +41,7 @@ public class BadgesComponent
|
||||
{
|
||||
boolean delete = true;
|
||||
|
||||
for (Rank rank : Emulator.getGameEnvironment().getPermissionsManager().getRanks(badge.getCode()))
|
||||
for (Rank rank : Emulator.getGameEnvironment().getPermissionsManager().getRanksByBadgeCode(badge.getCode()))
|
||||
{
|
||||
if (rank.getId() == habbo.getHabboInfo().getRank().getId())
|
||||
{
|
||||
|
Loading…
Reference in New Issue
Block a user