- Fixing NPE in IsPlayer()

src/main/java/forge/AIPlayer.java

@@ -67,11 +67,6 @@ public class AIPlayer extends Player {
         return true;
     }
 
-    /** {@inheritDoc} */
-    public boolean isPlayer(Player p1) {
-        return p1.getName().equals(this.name);
-    }
-
     ///////////////
     ///
     /// End transition methods

src/main/java/forge/HumanPlayer.java

@@ -68,7 +68,7 @@ public class HumanPlayer extends Player {
 
     /** {@inheritDoc} */
     public boolean isPlayer(Player p1) {
-        return p1.getName().equals(this.name);
+        return p1 != null && p1.getName().equals(this.name);
     }
 
     ///////////////

src/main/java/forge/Player.java

@@ -127,7 +127,9 @@ public abstract class Player extends MyObservable {
      * @param p1 a {@link forge.Player} object.
      * @return a boolean.
      */
-    public abstract boolean isPlayer(Player p1);
+    public boolean isPlayer(Player p1) {
+        return p1 != null && p1.getName().equals(this.name);
+    }
 
     /**
      * <p>getOpponent.</p>