Bug fix

Null pointer exception fixed in armor spell.
Also added a nice message prompting users to provide any error log that occurrs when enabling volatile code for 1.11 R1.

Added comments to most of the passive triggers regarding their trigger variables.

Author: TheComputerGeek2

src/com/nisovin/magicspells/spells/buff/ArmorSpell.java

@@ -209,9 +209,11 @@ public void onPlayerDeath(PlayerDeathEvent event) {
 			Iterator<ItemStack> drops = event.getDrops().iterator();
 			while (drops.hasNext()) {
 				ItemStack drop = drops.next();
-				List<String> lore = drop.getItemMeta().getLore();
-				if (lore != null && lore.size() > 0 && lore.get(lore.size()-1).equals(strLoreText)) {
-					drops.remove();
+				if (drop.hasItemMeta() && drop.getItemMeta() != null) {
+					List<String> lore = drop.getItemMeta().getLore();
+					if (lore != null && lore.size() > 0 && lore.get(lore.size()-1).equals(strLoreText)) {
+						drops.remove();
+					}
 				}
 			}
 		}

src/com/nisovin/magicspells/spells/passive/BlockBreakListener.java

@@ -19,6 +19,7 @@
 import com.nisovin.magicspells.materials.MagicMaterial;
 import com.nisovin.magicspells.spells.PassiveSpell;
 
+// optional trigger variable of a comma separated list of blocks to accept
 public class BlockBreakListener extends PassiveListener {
 
 	Set<Material> materials = new HashSet<Material>();

src/com/nisovin/magicspells/spells/passive/BlockPlaceListener.java

@@ -19,6 +19,7 @@
 import com.nisovin.magicspells.materials.MagicMaterial;
 import com.nisovin.magicspells.spells.PassiveSpell;
 
+// optional trigger variable of comma separated list of blocks to accept
 public class BlockPlaceListener extends PassiveListener {
 
 	Set<Material> materials = new HashSet<Material>();

src/com/nisovin/magicspells/spells/passive/BuffListener.java

@@ -21,6 +21,7 @@
 import com.nisovin.magicspells.spells.BuffSpell;
 import com.nisovin.magicspells.spells.PassiveSpell;
 
+// no trigger variable currently used
 public class BuffListener extends PassiveListener {
 
 	List<PassiveSpell> spells = new ArrayList<PassiveSpell>();

src/com/nisovin/magicspells/spells/passive/CraftListener.java

@@ -19,6 +19,7 @@
 import com.nisovin.magicspells.materials.MagicMaterial;
 import com.nisovin.magicspells.spells.PassiveSpell;
 
+// optional trigger variable of a comma separated list of items to accept as the result
 public class CraftListener extends PassiveListener {
 
 	Set<Material> materials = new HashSet<Material>();

src/com/nisovin/magicspells/spells/passive/DeathListener.java

@@ -10,6 +10,7 @@
 import com.nisovin.magicspells.Spellbook;
 import com.nisovin.magicspells.spells.PassiveSpell;
 
+// no trigger variable used here
 public class DeathListener extends PassiveListener {
 
 	List<PassiveSpell> spells = new ArrayList<PassiveSpell>();

src/com/nisovin/magicspells/spells/passive/DropItemListener.java

@@ -19,6 +19,7 @@
 import com.nisovin.magicspells.materials.MagicMaterial;
 import com.nisovin.magicspells.spells.PassiveSpell;
 
+// optional trigger variable that may contain a comma separated list of items to accept
 public class DropItemListener extends PassiveListener {
 
 	Set<Material> materials = new HashSet<Material>();

src/com/nisovin/magicspells/spells/passive/EnterBedListener.java

@@ -10,6 +10,7 @@
 import com.nisovin.magicspells.Spellbook;
 import com.nisovin.magicspells.spells.PassiveSpell;
 
+// no trigger variable is currently used
 public class EnterBedListener extends PassiveListener {
 
 	List<PassiveSpell> spells = new ArrayList<PassiveSpell>();

src/com/nisovin/magicspells/spells/passive/FatalDamageListener.java

@@ -12,6 +12,7 @@
 import com.nisovin.magicspells.Spellbook;
 import com.nisovin.magicspells.spells.PassiveSpell;
 
+// no trigger variable is used here
 public class FatalDamageListener extends PassiveListener {
 
 	List<PassiveSpell> spells = new ArrayList<PassiveSpell>();

src/com/nisovin/magicspells/spells/passive/FishListener.java

@@ -19,6 +19,10 @@
 import com.nisovin.magicspells.spells.PassiveSpell;
 import com.nisovin.magicspells.util.Util;
 
+// trigger variable is optional
+// if not specified, it triggers in all forms
+// the trigger variable may be a comma separated list containing any of the following
+// ground, fish, fail, <entity type>
 public class FishListener extends PassiveListener {
 
 	Map<EntityType, List<PassiveSpell>> types = new HashMap<EntityType, List<PassiveSpell>>();