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

[FIX] Fixes issue 309. Seems that canChoose and possibleTargets fell out of sync. canChoose was saying that the abilities target could be chosen but when it came to actually select the target no possibilities were returned.

Browse source
This commit is contained in:
maurer.it 2011-10-22 11:21:24 -04:00
parent ffadc3be45
commit 1df242adce
1 changed files with 28 additions and 24 deletions
Download patch file Download diff file Expand all files Collapse all files

52
Mage/src/mage/target/TargetCard.java
View file

@ -134,31 +134,35 @@ public class TargetCard<T extends TargetCard<T>> extends TargetObject<TargetCard
@Override
public Set<UUID> possibleTargets(UUID sourceControllerId, Game game) {
Set<UUID> possibleTargets = new HashSet<UUID>();
Player player = game.getPlayer(sourceControllerId);
if (player != null) {
switch (zone) {
case HAND:
for (Card card: player.getHand().getCards(filter, game)) {
possibleTargets.add(card.getId());
for (UUID playerId: game.getPlayer(sourceControllerId).getInRange()) {
if (filter.matchOwner(playerId)) {
Player player = game.getPlayer(playerId);
if (player != null) {
switch (zone) {
case HAND:
for (Card card: player.getHand().getCards(filter, game)) {
possibleTargets.add(card.getId());
}
break;
case GRAVEYARD:
for (Card card: player.getGraveyard().getCards(filter, game)) {
possibleTargets.add(card.getId());
}
break;
case LIBRARY:
for (Card card: player.getLibrary().getUniqueCards(game)) {
if (filter.match(card))
possibleTargets.add(card.getId());
}
break;
case EXILED:
for (Card card: game.getExile().getPermanentExile().getUniqueCards(game)) {
if (filter.match(card, player.getId(), game))
possibleTargets.add(card.getId());
}
break;
}
break;
case GRAVEYARD:
for (Card card: player.getGraveyard().getCards(filter, game)) {
possibleTargets.add(card.getId());
}
break;
case LIBRARY:
for (Card card: player.getLibrary().getUniqueCards(game)) {
if (filter.match(card))
possibleTargets.add(card.getId());
}
break;
case EXILED:
for (Card card: game.getExile().getPermanentExile().getUniqueCards(game)) {
if (filter.match(card, player.getId(), game))
possibleTargets.add(card.getId());
}
break;
}
}
}
return possibleTargets;