Fixed NPE with repairing items that rely on metadata.

This commit is contained in:
GJ 2012-06-06 18:27:52 -04:00
parent 2f15c71412
commit d644b4c331
2 changed files with 4 additions and 10 deletions

View File

@ -21,6 +21,7 @@ Version 1.3.09
= Fixed bug with locale strings when trying to teleport to a non-existent player = Fixed bug with locale strings when trying to teleport to a non-existent player
= Fixed bug with Tree Feller changing durability before checking for axe splintering = Fixed bug with Tree Feller changing durability before checking for axe splintering
= Fixed bug with Repair Mastery permission due to typo = Fixed bug with Repair Mastery permission due to typo
= Fixed bug with repairing items that use metadata
! API methods can now only be used in a static way ! API methods can now only be used in a static way
! Changed Spout settings to be in their own config file (spout.yml) ! Changed Spout settings to be in their own config file (spout.yml)

View File

@ -205,17 +205,9 @@ public class SimpleRepairManager implements RepairManager {
ItemStack[] contents = inventory.getContents(); ItemStack[] contents = inventory.getContents();
for (int i = 0; i < contents.length; i++) { for (int i = 0; i < contents.length; i++) {
ItemStack item = contents[i]; ItemStack item = contents[i];
//DEBUG TIME!
if (item == null) { if (item == null) {
System.out.println("The item is null."); continue;
System.out.println("Content length: " + contents.length);
System.out.println("Current contents: " + i);
}
else {
System.out.println("The item is: " + item.toString());
System.out.println("Content length: " + contents.length);
System.out.println("Current contents: " + i);
} }
if (item.getTypeId() == itemId) { if (item.getTypeId() == itemId) {
@ -223,6 +215,7 @@ public class SimpleRepairManager implements RepairManager {
location = i; location = i;
} }
} }
if (location != -1) { if (location != -1) {
break; break;
} }