Upstream/mmocore
1
0
Fork 1
mirror of https://gitlab.com/phoenix-dvpmt/mmocore.git synced 2024-10-23 20:19:57 +02:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fixed two random NPE

Browse Source
This commit is contained in:
ASangarin 2020-12-17 01:46:03 +01:00
parent 0aa36dacd5
commit 753f63104a
2 changed files with 10 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/main/java/net/Indyuce/mmocore/api/experience/source/CraftItemExperienceSource.java
View File

@ -28,7 +28,8 @@ public class CraftItemExperienceSource extends SpecificExperienceSource<Material
return new ExperienceManager<CraftItemExperienceSource>() { return new ExperienceManager<CraftItemExperienceSource>() {
@EventHandler(priority = EventPriority.MONITOR, ignoreCancelled = true) @EventHandler(priority = EventPriority.MONITOR, ignoreCancelled = true)
public void a(CraftItemEvent event) { public void a(CraftItemEvent event) {
if(event.getAction() == InventoryAction.NOTHING) return; if(event.getAction() == InventoryAction.NOTHING ||
event.getInventory().getResult() == null) return;
PlayerData data = PlayerData.get((Player) event.getWhoClicked()); PlayerData data = PlayerData.get((Player) event.getWhoClicked());
for (CraftItemExperienceSource source : getSources()) for (CraftItemExperienceSource source : getSources())
if (source.matches(data, event.getInventory().getResult().getType())) if (source.matches(data, event.getInventory().getResult().getType()))

11
src/main/java/net/Indyuce/mmocore/api/player/PlayerData.java
View File

@ -136,10 +136,15 @@ public class PlayerData extends OfflinePlayerData {
try { try {
boundSkills.set(j, getProfess().getSkill(boundSkills.get(j).getSkill())); boundSkills.set(j, getProfess().getSkill(boundSkills.get(j).getSkill()));
j++; j++;
} catch (NullPointerException notFound) { } catch (NullPointerException npe1) {
boundSkills.remove(j); boundSkills.remove(j);
MMOCore.log(Level.SEVERE, "[Userdata] Could not find skill " + boundSkills.get(j).getSkill().getId() + " in class " try {
+ getProfess().getId() + " while refreshing player data."); MMOCore.log(Level.SEVERE, "[Userdata] Could not find skill " + boundSkills.get(j).getSkill().getId() + " in class "
+ getProfess().getId() + " while refreshing player data.");
} catch (NullPointerException npe2) {
MMOCore.log(Level.SEVERE, "[Userdata] Could not find unidentified skill in class "
+ getProfess().getId() + " while refreshing player data.");
}
} }
} }