Exclude non playables from isSiegeFriend mid victory check.

This commit is contained in:
MobiusDevelopment
2022-04-05 21:53:49 +00:00
parent b62ccc548a
commit 02597f65af
25 changed files with 50 additions and 50 deletions
@@ -1677,13 +1677,13 @@ public class Player extends Playable
// Check first castle mid victory. // Check first castle mid victory.
final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide);
if ((castle != null) && !castle.isFirstMidVictory()) final Player targetPlayer = target.getActingPlayer();
if ((castle != null) && (targetPlayer != null) && !castle.isFirstMidVictory())
{ {
return true; return true;
} }
// If target isn't a player, is self, isn't on same siege or not on same state, not friends. // If target isn't a player, is self, isn't on same siege or not on same state, not friends.
final Player targetPlayer = target.getActingPlayer();
if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState())) if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState()))
{ {
return false; return false;
@@ -1679,13 +1679,13 @@ public class Player extends Playable
// Check first castle mid victory. // Check first castle mid victory.
final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide);
if ((castle != null) && !castle.isFirstMidVictory()) final Player targetPlayer = target.getActingPlayer();
if ((castle != null) && (targetPlayer != null) && !castle.isFirstMidVictory())
{ {
return true; return true;
} }
// If target isn't a player, is self, isn't on same siege or not on same state, not friends. // If target isn't a player, is self, isn't on same siege or not on same state, not friends.
final Player targetPlayer = target.getActingPlayer();
if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState())) if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState()))
{ {
return false; return false;
@@ -1681,13 +1681,13 @@ public class Player extends Playable
// Check first castle mid victory. // Check first castle mid victory.
final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide);
if ((castle != null) && !castle.isFirstMidVictory()) final Player targetPlayer = target.getActingPlayer();
if ((castle != null) && (targetPlayer != null) && !castle.isFirstMidVictory())
{ {
return true; return true;
} }
// If target isn't a player, is self, isn't on same siege or not on same state, not friends. // If target isn't a player, is self, isn't on same siege or not on same state, not friends.
final Player targetPlayer = target.getActingPlayer();
if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState())) if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState()))
{ {
return false; return false;
@@ -1691,13 +1691,13 @@ public class Player extends Playable
// Check first castle mid victory. // Check first castle mid victory.
final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide);
if ((castle != null) && !castle.isFirstMidVictory()) final Player targetPlayer = target.getActingPlayer();
if ((castle != null) && (targetPlayer != null) && !castle.isFirstMidVictory())
{ {
return true; return true;
} }
// If target isn't a player, is self, isn't on same siege or not on same state, not friends. // If target isn't a player, is self, isn't on same siege or not on same state, not friends.
final Player targetPlayer = target.getActingPlayer();
if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState())) if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState()))
{ {
return false; return false;
@@ -1687,13 +1687,13 @@ public class Player extends Playable
// Check first castle mid victory. // Check first castle mid victory.
final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide);
if ((castle != null) && !castle.isFirstMidVictory()) final Player targetPlayer = target.getActingPlayer();
if ((castle != null) && (targetPlayer != null) && !castle.isFirstMidVictory())
{ {
return true; return true;
} }
// If target isn't a player, is self, isn't on same siege or not on same state, not friends. // If target isn't a player, is self, isn't on same siege or not on same state, not friends.
final Player targetPlayer = target.getActingPlayer();
if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState())) if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState()))
{ {
return false; return false;
@@ -1687,13 +1687,13 @@ public class Player extends Playable
// Check first castle mid victory. // Check first castle mid victory.
final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide);
if ((castle != null) && !castle.isFirstMidVictory()) final Player targetPlayer = target.getActingPlayer();
if ((castle != null) && (targetPlayer != null) && !castle.isFirstMidVictory())
{ {
return true; return true;
} }
// If target isn't a player, is self, isn't on same siege or not on same state, not friends. // If target isn't a player, is self, isn't on same siege or not on same state, not friends.
final Player targetPlayer = target.getActingPlayer();
if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState())) if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState()))
{ {
return false; return false;
@@ -1687,13 +1687,13 @@ public class Player extends Playable
// Check first castle mid victory. // Check first castle mid victory.
final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide);
if ((castle != null) && !castle.isFirstMidVictory()) final Player targetPlayer = target.getActingPlayer();
if ((castle != null) && (targetPlayer != null) && !castle.isFirstMidVictory())
{ {
return true; return true;
} }
// If target isn't a player, is self, isn't on same siege or not on same state, not friends. // If target isn't a player, is self, isn't on same siege or not on same state, not friends.
final Player targetPlayer = target.getActingPlayer();
if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState())) if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState()))
{ {
return false; return false;
@@ -1705,13 +1705,13 @@ public class Player extends Playable
// Check first castle mid victory. // Check first castle mid victory.
final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide);
if ((castle != null) && !castle.isFirstMidVictory()) final Player targetPlayer = target.getActingPlayer();
if ((castle != null) && (targetPlayer != null) && !castle.isFirstMidVictory())
{ {
return true; return true;
} }
// If target isn't a player, is self, isn't on same siege or not on same state, not friends. // If target isn't a player, is self, isn't on same siege or not on same state, not friends.
final Player targetPlayer = target.getActingPlayer();
if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState())) if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState()))
{ {
return false; return false;
@@ -1707,13 +1707,13 @@ public class Player extends Playable
// Check first castle mid victory. // Check first castle mid victory.
final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide);
if ((castle != null) && !castle.isFirstMidVictory()) final Player targetPlayer = target.getActingPlayer();
if ((castle != null) && (targetPlayer != null) && !castle.isFirstMidVictory())
{ {
return true; return true;
} }
// If target isn't a player, is self, isn't on same siege or not on same state, not friends. // If target isn't a player, is self, isn't on same siege or not on same state, not friends.
final Player targetPlayer = target.getActingPlayer();
if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState())) if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState()))
{ {
return false; return false;
@@ -1724,13 +1724,13 @@ public class Player extends Playable
// Check first castle mid victory. // Check first castle mid victory.
final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide);
if ((castle != null) && !castle.isFirstMidVictory()) final Player targetPlayer = target.getActingPlayer();
if ((castle != null) && (targetPlayer != null) && !castle.isFirstMidVictory())
{ {
return true; return true;
} }
// If target isn't a player, is self, isn't on same siege or not on same state, not friends. // If target isn't a player, is self, isn't on same siege or not on same state, not friends.
final Player targetPlayer = target.getActingPlayer();
if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState())) if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState()))
{ {
return false; return false;
@@ -1726,13 +1726,13 @@ public class Player extends Playable
// Check first castle mid victory. // Check first castle mid victory.
final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide);
if ((castle != null) && !castle.isFirstMidVictory()) final Player targetPlayer = target.getActingPlayer();
if ((castle != null) && (targetPlayer != null) && !castle.isFirstMidVictory())
{ {
return true; return true;
} }
// If target isn't a player, is self, isn't on same siege or not on same state, not friends. // If target isn't a player, is self, isn't on same siege or not on same state, not friends.
final Player targetPlayer = target.getActingPlayer();
if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState())) if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState()))
{ {
return false; return false;
@@ -1726,13 +1726,13 @@ public class Player extends Playable
// Check first castle mid victory. // Check first castle mid victory.
final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide);
if ((castle != null) && !castle.isFirstMidVictory()) final Player targetPlayer = target.getActingPlayer();
if ((castle != null) && (targetPlayer != null) && !castle.isFirstMidVictory())
{ {
return true; return true;
} }
// If target isn't a player, is self, isn't on same siege or not on same state, not friends. // If target isn't a player, is self, isn't on same siege or not on same state, not friends.
final Player targetPlayer = target.getActingPlayer();
if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState())) if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState()))
{ {
return false; return false;
@@ -1636,13 +1636,13 @@ public class Player extends Playable
// Check first castle mid victory. // Check first castle mid victory.
final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide);
if ((castle != null) && !castle.isFirstMidVictory()) final Player targetPlayer = target.getActingPlayer();
if ((castle != null) && (targetPlayer != null) && !castle.isFirstMidVictory())
{ {
return true; return true;
} }
// If target isn't a player, is self, isn't on same siege or not on same state, not friends. // If target isn't a player, is self, isn't on same siege or not on same state, not friends.
final Player targetPlayer = target.getActingPlayer();
if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState())) if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState()))
{ {
return false; return false;
@@ -1657,13 +1657,13 @@ public class Player extends Playable
// Check first castle mid victory. // Check first castle mid victory.
final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide);
if ((castle != null) && !castle.isFirstMidVictory()) final Player targetPlayer = target.getActingPlayer();
if ((castle != null) && (targetPlayer != null) && !castle.isFirstMidVictory())
{ {
return true; return true;
} }
// If target isn't a player, is self, isn't on same siege or not on same state, not friends. // If target isn't a player, is self, isn't on same siege or not on same state, not friends.
final Player targetPlayer = target.getActingPlayer();
if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState())) if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState()))
{ {
return false; return false;
@@ -1646,13 +1646,13 @@ public class Player extends Playable
// Check first castle mid victory. // Check first castle mid victory.
final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide);
if ((castle != null) && !castle.isFirstMidVictory()) final Player targetPlayer = target.getActingPlayer();
if ((castle != null) && (targetPlayer != null) && !castle.isFirstMidVictory())
{ {
return true; return true;
} }
// If target isn't a player, is self, isn't on same siege or not on same state, not friends. // If target isn't a player, is self, isn't on same siege or not on same state, not friends.
final Player targetPlayer = target.getActingPlayer();
if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState())) if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState()))
{ {
return false; return false;
@@ -1646,13 +1646,13 @@ public class Player extends Playable
// Check first castle mid victory. // Check first castle mid victory.
final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide);
if ((castle != null) && !castle.isFirstMidVictory()) final Player targetPlayer = target.getActingPlayer();
if ((castle != null) && (targetPlayer != null) && !castle.isFirstMidVictory())
{ {
return true; return true;
} }
// If target isn't a player, is self, isn't on same siege or not on same state, not friends. // If target isn't a player, is self, isn't on same siege or not on same state, not friends.
final Player targetPlayer = target.getActingPlayer();
if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState())) if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState()))
{ {
return false; return false;
@@ -1644,13 +1644,13 @@ public class Player extends Playable
// Check first castle mid victory. // Check first castle mid victory.
final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide);
if ((castle != null) && !castle.isFirstMidVictory()) final Player targetPlayer = target.getActingPlayer();
if ((castle != null) && (targetPlayer != null) && !castle.isFirstMidVictory())
{ {
return true; return true;
} }
// If target isn't a player, is self, isn't on same siege or not on same state, not friends. // If target isn't a player, is self, isn't on same siege or not on same state, not friends.
final Player targetPlayer = target.getActingPlayer();
if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState())) if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState()))
{ {
return false; return false;
@@ -1657,13 +1657,13 @@ public class Player extends Playable
// Check first castle mid victory. // Check first castle mid victory.
final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide);
if ((castle != null) && !castle.isFirstMidVictory()) final Player targetPlayer = target.getActingPlayer();
if ((castle != null) && (targetPlayer != null) && !castle.isFirstMidVictory())
{ {
return true; return true;
} }
// If target isn't a player, is self, isn't on same siege or not on same state, not friends. // If target isn't a player, is self, isn't on same siege or not on same state, not friends.
final Player targetPlayer = target.getActingPlayer();
if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState())) if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState()))
{ {
return false; return false;
@@ -1657,13 +1657,13 @@ public class Player extends Playable
// Check first castle mid victory. // Check first castle mid victory.
final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide);
if ((castle != null) && !castle.isFirstMidVictory()) final Player targetPlayer = target.getActingPlayer();
if ((castle != null) && (targetPlayer != null) && !castle.isFirstMidVictory())
{ {
return true; return true;
} }
// If target isn't a player, is self, isn't on same siege or not on same state, not friends. // If target isn't a player, is self, isn't on same siege or not on same state, not friends.
final Player targetPlayer = target.getActingPlayer();
if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState())) if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState()))
{ {
return false; return false;
@@ -1677,13 +1677,13 @@ public class Player extends Playable
// Check first castle mid victory. // Check first castle mid victory.
final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide);
if ((castle != null) && !castle.isFirstMidVictory()) final Player targetPlayer = target.getActingPlayer();
if ((castle != null) && (targetPlayer != null) && !castle.isFirstMidVictory())
{ {
return true; return true;
} }
// If target isn't a player, is self, isn't on same siege or not on same state, not friends. // If target isn't a player, is self, isn't on same siege or not on same state, not friends.
final Player targetPlayer = target.getActingPlayer();
if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState())) if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState()))
{ {
return false; return false;
@@ -1643,13 +1643,13 @@ public class Player extends Playable
// Check first castle mid victory. // Check first castle mid victory.
final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide);
if ((castle != null) && !castle.isFirstMidVictory()) final Player targetPlayer = target.getActingPlayer();
if ((castle != null) && (targetPlayer != null) && !castle.isFirstMidVictory())
{ {
return true; return true;
} }
// If target isn't a player, is self, isn't on same siege or not on same state, not friends. // If target isn't a player, is self, isn't on same siege or not on same state, not friends.
final Player targetPlayer = target.getActingPlayer();
if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState())) if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState()))
{ {
return false; return false;
@@ -1734,13 +1734,13 @@ public class Player extends Playable
// Check first castle mid victory. // Check first castle mid victory.
final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide);
if ((castle != null) && !castle.isFirstMidVictory()) final Player targetPlayer = target.getActingPlayer();
if ((castle != null) && (targetPlayer != null) && !castle.isFirstMidVictory())
{ {
return true; return true;
} }
// If target isn't a player, is self, isn't on same siege or not on same state, not friends. // If target isn't a player, is self, isn't on same siege or not on same state, not friends.
final Player targetPlayer = target.getActingPlayer();
if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState())) if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState()))
{ {
return false; return false;
@@ -1765,13 +1765,13 @@ public class Player extends Playable
// Check first castle mid victory. // Check first castle mid victory.
final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide);
if ((castle != null) && !castle.isFirstMidVictory()) final Player targetPlayer = target.getActingPlayer();
if ((castle != null) && (targetPlayer != null) && !castle.isFirstMidVictory())
{ {
return true; return true;
} }
// If target isn't a player, is self, isn't on same siege or not on same state, not friends. // If target isn't a player, is self, isn't on same siege or not on same state, not friends.
final Player targetPlayer = target.getActingPlayer();
if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState())) if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState()))
{ {
return false; return false;
@@ -1771,13 +1771,13 @@ public class Player extends Playable
// Check first castle mid victory. // Check first castle mid victory.
final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide);
if ((castle != null) && !castle.isFirstMidVictory()) final Player targetPlayer = target.getActingPlayer();
if ((castle != null) && (targetPlayer != null) && !castle.isFirstMidVictory())
{ {
return true; return true;
} }
// If target isn't a player, is self, isn't on same siege or not on same state, not friends. // If target isn't a player, is self, isn't on same siege or not on same state, not friends.
final Player targetPlayer = target.getActingPlayer();
if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState())) if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState()))
{ {
return false; return false;
@@ -1771,13 +1771,13 @@ public class Player extends Playable
// Check first castle mid victory. // Check first castle mid victory.
final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide);
if ((castle != null) && !castle.isFirstMidVictory()) final Player targetPlayer = target.getActingPlayer();
if ((castle != null) && (targetPlayer != null) && !castle.isFirstMidVictory())
{ {
return true; return true;
} }
// If target isn't a player, is self, isn't on same siege or not on same state, not friends. // If target isn't a player, is self, isn't on same siege or not on same state, not friends.
final Player targetPlayer = target.getActingPlayer();
if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState())) if ((targetPlayer == null) || (targetPlayer == this) || (targetPlayer.getSiegeSide() != _siegeSide) || (_siegeState != targetPlayer.getSiegeState()))
{ {
return false; return false;