Merge pull request #383 from Glitchfinder/master

Fixing a few more NPEs.
This commit is contained in:
Glitchfinder 2012-12-28 21:11:58 -08:00
commit d35e56fd37
2 changed files with 29 additions and 17 deletions

View File

@ -308,7 +308,7 @@ public class Mining {
break;
default:
if (ModChecks.isCustomMiningBlock(block) && ModChecks.getCustomBlock(block) != null) {
if (ModChecks.isCustomMiningBlock(block)) {
xp += ModChecks.getCustomBlock(block).getXpGain();
}
break;

View File

@ -76,11 +76,14 @@ public class ModChecks {
item.setData(new MaterialData(block.getTypeId(), block.getData()));
if (customBlocksEnabled && blocksInstance.customMiningBlocks.contains(item)) {
return true;
}
else {
return false;
for (CustomBlock b : blocksInstance.customBlocks) {
if ((b.getItemID() == block.getTypeId()) && (b.getDataValue() == block.getData())) {
return true;
}
}
}
return false;
}
/**
@ -94,11 +97,14 @@ public class ModChecks {
item.setData(new MaterialData(block.getTypeId(), block.getData()));
if (blocksInstance.customLeaves.contains(item)) {
return true;
}
else {
return false;
for (CustomBlock b : blocksInstance.customBlocks) {
if ((b.getItemID() == block.getTypeId()) && (b.getDataValue() == block.getData())) {
return true;
}
}
}
return false;
}
/**
@ -112,11 +118,14 @@ public class ModChecks {
item.setData(new MaterialData(block.getTypeId(), block.getData()));
if (blocksInstance.customLogs.contains(item)) {
return true;
}
else {
return false;
for (CustomBlock b : blocksInstance.customBlocks) {
if ((b.getItemID() == block.getTypeId()) && (b.getDataValue() == block.getData())) {
return true;
}
}
}
return false;
}
/**
@ -130,11 +139,14 @@ public class ModChecks {
item.setData(new MaterialData(block.getTypeId(), block.getData()));
if (blocksInstance.customOres.contains(item)) {
return true;
}
else {
return false;
for (CustomBlock b : blocksInstance.customBlocks) {
if ((b.getItemID() == block.getTypeId()) && (b.getDataValue() == block.getData())) {
return true;
}
}
}
return false;
}
/**