Failure/foul-magics
1
0
Fork 0
forked from External/mage
Code Pull requests Releases Packages Activity Actions

Fixed protector NPE on empty opponents

Browse source
This commit is contained in:
Oleg Agafonov 2023-04-19 18:40:04 +04:00
parent c061c7f436
commit 2e8f24edb3
1 changed files with 9 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

16
Mage/src/main/java/mage/game/permanent/PermanentImpl.java
View file

@ -1677,20 +1677,22 @@ public abstract class PermanentImpl extends CardImpl implements Permanent {
@Override
public void chooseProtector(Game game, Ability source) {
Set<UUID> opponents = game.getOpponents(this.getControllerId());
UUID protectorId;
Player newProtector;
if (opponents.size() > 1) {
TargetPlayer target = new TargetPlayer(new FilterOpponent("protector for " + getName()));
target.setNotTarget(true);
target.setRequired(true);
game.getPlayer(getControllerId()).choose(Outcome.Neutral, target, source, game);
protectorId = target.getFirstTarget();
newProtector = game.getPlayer(target.getFirstTarget());
} else {
protectorId = RandomUtil.randomFromCollection(opponents);
newProtector = game.getPlayer(opponents.iterator().next());
}
if (newProtector != null) {
String protectorName = newProtector.getLogName();
game.informPlayers(protectorName + " has been chosen to protect " + this.getLogName());
this.addInfo("protector", "Protected by " + protectorName, game);
this.setProtectorId(newProtector.getId());
}
String protectorName = game.getPlayer(protectorId).getLogName();
game.informPlayers(protectorName + " has been chosen to protect " + this.getLogName());
this.addInfo("protector", "Protected by " + protectorName, game);
this.setProtectorId(protectorId);
}
@Override