diff --git a/src/ca/gibstick/discosheep/DiscoSheep.java b/src/ca/gibstick/discosheep/DiscoSheep.java index 487257f..c16263d 100644 --- a/src/ca/gibstick/discosheep/DiscoSheep.java +++ b/src/ca/gibstick/discosheep/DiscoSheep.java @@ -44,15 +44,18 @@ public final class DiscoSheep extends JavaPlugin { /* * Iterate through all live entities and create default configuration values for them - * excludes bosses and pigmen since they throw NPE for some reason - * excludes horses for 1.5.2 compatibility (also NPE) + * excludes bosses and other mobs that throw NPE */ for (EntityType ent : EntityType.values()) { if (ent.isAlive() && !ent.equals(EntityType.ENDER_DRAGON) && !ent.equals(EntityType.WITHER) && !ent.equals(EntityType.PIG_ZOMBIE) - && !ent.equals(EntityType.HORSE) + && !ent.equals(EntityType.OCELOT) + && !ent.equals(EntityType.CAVE_SPIDER) + && !ent.equals(EntityType.MAGMA_CUBE) + && !ent.equals(EntityType.MUSHROOM_COW) + && !ent.equals(EntityType.IRON_GOLEM) && !ent.equals(EntityType.PLAYER)) { getConfig().addDefault("default.guests." + ent.toString(), 0); getConfig().addDefault("max.guests." + ent.toString(), 0); diff --git a/src/ca/gibstick/discosheep/DiscoSheepCommandExecutor.java b/src/ca/gibstick/discosheep/DiscoSheepCommandExecutor.java index 18efafc..b5f3d5e 100644 --- a/src/ca/gibstick/discosheep/DiscoSheepCommandExecutor.java +++ b/src/ca/gibstick/discosheep/DiscoSheepCommandExecutor.java @@ -156,7 +156,7 @@ public class DiscoSheepCommandExecutor implements CommandExecutor { String[] guests = getNextArgs(args, i + 1); int j = 0; - while (j < guests.length) { + while (j < guests.length - 1) { try { mainParty.setGuestNumber(guests[j], getNextIntArg(guests, j)); } catch (IllegalArgumentException e) { diff --git a/src/ca/gibstick/discosheep/PartyEvents.java b/src/ca/gibstick/discosheep/PartyEvents.java index 4ca8197..cc1ef9d 100644 --- a/src/ca/gibstick/discosheep/PartyEvents.java +++ b/src/ca/gibstick/discosheep/PartyEvents.java @@ -1,6 +1,5 @@ package ca.gibstick.discosheep; -import java.util.logging.Level; import org.bukkit.entity.LivingEntity; import org.bukkit.entity.Sheep; import org.bukkit.event.EventHandler; @@ -18,6 +17,12 @@ public class PartyEvents implements Listener { DiscoSheep parent; DiscoParty party; + /* + * There will be multiple isntances of PartyEvents, + * and each instance will only listen for its own party. + * That way, we don't have multiple instances iterating through + * the entire parties hashmap redunandtly + */ public PartyEvents(DiscoSheep parent, DiscoParty party) { this.parent = parent; @@ -54,8 +59,6 @@ public class PartyEvents implements Listener { party.jump((LivingEntity) e.getEntity()); e.setCancelled(true); } - - parent.getLogger().log(Level.INFO, "Debug: EVENT TRIGGERED"); } // prevent uninvited guests from targetting players