Upstream/Essentials
1
0
Fork 0
mirror of https://github.com/EssentialsX/Essentials.git synced 2025-03-02 11:11:58 +01:00
Code Issues Projects Releases Wiki Activity

Use ItemData#equals when finding item name

Browse Source 
Avoids accidentally causing an NPE while comparing potion data.
This commit is contained in:
md678685 2018-10-13 16:09:45 +01:00
parent f78887a02e
commit 85111f25cb
1 changed files with 10 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
Essentials/src/com/earth2me/essentials/ItemDb.java
View File

@ -194,8 +194,10 @@ public class ItemDb implements IConf, net.ess3.api.IItemDb {
potion = ((PotionMeta) item.getItemMeta()).getBasePotionData();
}
ItemData data = new ItemData(type, potion);
for (Map.Entry<String, ItemData> entry : items.entrySet()) {
if (entry.getValue().getMaterial().equals(type) && entry.getValue().getPotionData().equals(potion)) {
if (entry.getValue().equals(data)) {
return entry.getKey();
}
}
@ -399,6 +401,11 @@ public class ItemDb implements IConf, net.ess3.api.IItemDb {
private Material material;
private PotionData potionData;
public ItemData(Material material, PotionData potionData) {
this.material = material;
this.potionData = potionData;
}
@Override
public int hashCode() {
return (31 * material.hashCode()) ^ potionData.hashCode();
@ -412,8 +419,8 @@ public class ItemDb implements IConf, net.ess3.api.IItemDb {
if (!(o instanceof ItemData)) {
return false;
}
ItemData pairo = (ItemData) o;
return this.material == pairo.getMaterial() && potionDataEquals(pairo);
ItemData that = (ItemData) o;
return this.material == that.getMaterial() && potionDataEquals(that);
}
public String getItemName() {