Working on fixing null player... Making factions optional dep.

plugin.yml

@@ -2,7 +2,7 @@ name: BubbleShield
 author: squidicuz
 version: 0.9.92
 main: net.sqdmc.bubbleshield.BubbleShield
-depend: [Factions]
+softdepend: [Factions]
 
 commands:
   bubbleshield:

src/net/sqdmc/bubbleshield/ShieldOwnerPlayer.java

@@ -11,9 +11,10 @@ public class ShieldOwnerPlayer extends ShieldOwner {
 
 	public ShieldOwnerPlayer(String name) {
 		Player player = Bukkit.getPlayer(name);
+		OfflinePlayer offPlayer = Bukkit.getOfflinePlayer(name);
 
 		this.shieldOwner = player;
-		this.shieldOwnerOff = player;
+		this.shieldOwnerOff = offPlayer;
 	}
 
 	public ShieldOwnerPlayer(Player player) {
@@ -23,7 +24,10 @@ public ShieldOwnerPlayer(Player player) {
 
 	@Override
 	public String getOwner() {
-		return shieldOwner.getName();
+		if (this.shieldOwnerOff.isOnline())
+			return this.shieldOwner.getName();
+		else
+			return this.shieldOwnerOff.getName();
 	}
 
 	public void setPlayer(Player player){
@@ -33,8 +37,8 @@ public void setPlayer(Player player){
 
 	@Override
 	public void sendMessage(String message) {
-		if (shieldOwner != null && shieldOwner.isOnline()) {
-			shieldOwner.sendMessage(message);
+		if (this.shieldOwner != null && this.shieldOwnerOff.isOnline()) {
+			this.shieldOwner.sendMessage(message);
 		}
 	}
 
@@ -62,6 +66,6 @@ public boolean equals(Object obj) {
 
 	@Override
 	public String toString() {
-		return shieldOwner.getName();
+		return this.shieldOwner.getName();
 	}
 }

src/net/sqdmc/bubbleshield/Util.java

@@ -53,7 +53,7 @@ public static ShieldBase getShieldBase(Location targetLoc) {
 	public static int getShieldCount(ShieldStorage shieldstorage, String owner) {
 		int count = 0;
 		for (ShieldBase shieldbase : shieldstorage.GetShieldBases()) {
-			if (shieldbase.shield.getShieldOwner().getOwner() == owner) {
+			if (shieldbase.shield.getShieldOwner() != null && shieldbase.shield.getShieldOwner().getOwner() == owner) {
 				count++;
 			}
 		}