Rename ReturnCardAction to PutOntoBattlefieldAction

fixes #1632
master
Ada Joule 2018-11-26 05:19:37 +07:00
parent 3e15d41c16
commit b4e2e742a1
43 changed files with 52 additions and 52 deletions

View File

@ -15,7 +15,7 @@
@Override
public void executeEvent(final MagicGame game, final MagicEvent event) {
final MagicCard died = event.getRefCard();
game.doAction(new ReturnCardAction(
game.doAction(new PutOntoBattlefieldAction(
MagicLocationType.Graveyard,
died,
died.getOwner()

View File

@ -33,7 +33,7 @@
@Override
public void executeEvent(final MagicGame game, final MagicEvent event) {
event.processTargetCard(game, {
game.doAction(new ReturnCardAction(MagicLocationType.OwnersHand,it,event.getPlayer()));
game.doAction(new PutOntoBattlefieldAction(MagicLocationType.OwnersHand,it,event.getPlayer()));
});
}
}

View File

@ -11,7 +11,7 @@ def action = {
final MagicGame game, final MagicEvent event ->
if (event.isYes()) {
event.processTargetCard(game, {
game.doAction(new ReturnCardAction(
game.doAction(new PutOntoBattlefieldAction(
MagicLocationType.OwnersHand,
it,
event.getPlayer(),

View File

@ -25,7 +25,7 @@
for (final MagicCard card : cards) {
game.doAction(new RevealAction(card));
if (card.hasType(MagicType.Creature)) {
game.doAction(new ReturnCardAction(MagicLocationType.OwnersLibrary, card, player));
game.doAction(new PutOntoBattlefieldAction(MagicLocationType.OwnersLibrary, card, player));
} else {
game.doAction(new ShiftCardAction(card, MagicLocationType.OwnersLibrary, MagicLocationType.Graveyard));
game.logAppendMessage(player, "${player.getName()} puts ${card.getName()} into the graveyard.")

View File

@ -19,7 +19,7 @@
final MagicCard topCard = it.getOwner().getLibrary().getCardAtTop();
game.doAction(new RevealAction(topCard));
if (topCard.isPermanentCard()) {
game.doAction(new ReturnCardAction(MagicLocationType.OwnersLibrary, topCard, it.getOwner()));
game.doAction(new PutOntoBattlefieldAction(MagicLocationType.OwnersLibrary, topCard, it.getOwner()));
}
});
}

View File

@ -14,7 +14,7 @@
for (final MagicCard card : event.getPlayer().getLibrary().getCardsFromTop(1)) {
game.doAction(new RevealAction(card));
if (card.hasType(MagicType.Land)) {
game.doAction(new ReturnCardAction(
game.doAction(new PutOntoBattlefieldAction(
MagicLocationType.OwnersLibrary,
card,
event.getPlayer()

View File

@ -11,7 +11,7 @@ def endOfCombatTrigger = new AtEndOfCombatTrigger() {
public void executeEvent(final MagicGame game, final MagicEvent event) {
final MagicPermanent sourcePermanent = event.getPermanent();
game.doAction(new RemoveFromPlayAction(sourcePermanent, MagicLocationType.Exile));
game.doAction(new ReturnCardAction(
game.doAction(new PutOntoBattlefieldAction(
MagicLocationType.Exile,
sourcePermanent.getCard(),
event.getPlayer(),

View File

@ -20,7 +20,7 @@
if (event.isYes()) {
event.processTargetCard(game, {
final MagicPermanent sn = event.getPermanent();
game.doAction(new ReturnCardAction(MagicLocationType.OwnersHand,it,event.getPlayer(), {
game.doAction(new PutOntoBattlefieldAction(MagicLocationType.OwnersHand,it,event.getPlayer(), {
final MagicPermanent perm ->
final MagicGame G = perm.getGame();
G.doAction(new AttachAction(sn.map(G), perm));

View File

@ -15,7 +15,7 @@
@Override
public void executeEvent(final MagicGame game, final MagicEvent event) {
event.processTargetCard(game, {
game.doAction(new ReturnCardAction(MagicLocationType.Graveyard,it,event.getPlayer(), {
game.doAction(new PutOntoBattlefieldAction(MagicLocationType.Graveyard,it,event.getPlayer(), {
final MagicPermanent perm ->
final MagicGame G = perm.getGame();
if (perm.hasSubType(MagicSubType.Angel)) {

View File

@ -32,7 +32,7 @@
game.doAction(new PlayTokensAction(event.getPlayer(), CardDefinitions.getToken("1/1 green Saproling creature token"), 1));
}
if (card.hasType(MagicType.Land)) {
game.doAction(new ReturnCardAction(MagicLocationType.OwnersLibrary, card, event.getPlayer()));
game.doAction(new PutOntoBattlefieldAction(MagicLocationType.OwnersLibrary, card, event.getPlayer()));
}
if (!card.hasType(MagicType.Creature) && !card.hasType(MagicType.Land)) {
game.doAction(new ChangeLifeAction(event.getPlayer(), 2));

View File

@ -5,10 +5,10 @@ def putOntoBattlefieldAction = {
final MagicCardList rest = new MagicCardList(event.getRefCardList());
event.processChosenCards(game, {
rest.remove(it);
game.doAction(new ReturnCardAction(MagicLocationType.Exile, it, opponent));
game.doAction(new PutOntoBattlefieldAction(MagicLocationType.Exile, it, opponent));
});
rest.each {
game.doAction(new ReturnCardAction(MagicLocationType.Exile, it, controller));
game.doAction(new PutOntoBattlefieldAction(MagicLocationType.Exile, it, controller));
}
}

View File

@ -2,7 +2,7 @@ def action = {
final MagicGame game, final MagicEvent event ->
final MagicCardList topCards = new MagicCardList(event.getRefCardList());
event.processChosenCards(game, {
game.doAction(new ReturnCardAction(
game.doAction(new PutOntoBattlefieldAction(
MagicLocationType.OwnersLibrary,
it,
event.getPlayer(),

View File

@ -3,7 +3,7 @@ def action = {
if (event.isYes()) {
final MagicCard card = event.getRefCard();
final MagicPlayer player = event.getPlayer();
game.doAction(new ReturnCardAction(MagicLocationType.OwnersLibrary, card, event.getPlayer(), MagicPlayMod.TAPPED));
game.doAction(new PutOntoBattlefieldAction(MagicLocationType.OwnersLibrary, card, event.getPlayer(), MagicPlayMod.TAPPED));
game.logAppendMessage(player, "${player} puts (${card}) it onto the battlefield tapped.")
}
}

View File

@ -15,7 +15,7 @@
@Override
public void executeEvent(final MagicGame game, final MagicEvent event) {
event.processTargetCard(game, {
game.doAction(new ReturnCardAction(MagicLocationType.Graveyard,it,event.getPlayer(), {
game.doAction(new PutOntoBattlefieldAction(MagicLocationType.Graveyard,it,event.getPlayer(), {
final MagicPermanent perm ->
final MagicGame G = perm.getGame();
if (perm.hasSubType(MagicSubType.Dragon)) {

View File

@ -2,7 +2,7 @@ def toBattlefield = {
final MagicGame game, final MagicEvent event ->
event.processChosenCards(game, {
final MagicCard card ->
game.doAction(new ReturnCardAction(
game.doAction(new PutOntoBattlefieldAction(
MagicLocationType.OwnersLibrary,
card,
event.getPlayer()

View File

@ -44,7 +44,7 @@
if (it.getGraveyard().contains(card)) {
creatureCards++;
game.doAction(new SacrificeAction(event.getPermanent()));
game.doAction(new ReturnCardAction(
game.doAction(new PutOntoBattlefieldAction(
MagicLocationType.Graveyard,
card,
event.getPlayer()

View File

@ -29,7 +29,7 @@
game.doAction(new ShiftCardAction(card, MagicLocationType.OwnersLibrary, MagicLocationType.Graveyard));
game.logAppendMessage(player, "${player.getName()} puts ${card.getName()} into the graveyard.");
} else {
game.doAction(new ReturnCardAction(
game.doAction(new PutOntoBattlefieldAction(
MagicLocationType.OwnersLibrary,
card,
player,

View File

@ -3,7 +3,7 @@ def action = {
if (event.isYes()) {
final MagicCard card = event.getRefCard();
final MagicPlayer player = event.getPlayer();
game.doAction(new ReturnCardAction(MagicLocationType.OwnersLibrary, card, event.getPlayer()));
game.doAction(new PutOntoBattlefieldAction(MagicLocationType.OwnersLibrary, card, event.getPlayer()));
game.logAppendMessage(player, "${player} puts (${card}) it onto the battlefield.")
}
}

View File

@ -6,7 +6,7 @@ def action = {
final MagicEvent manaCost = new MagicPayManaCostEvent(event.getSource(), "{${cost}}");
if (manaCost.isSatisfied()) {
game.addEvent(manaCost);
game.doAction(new ReturnCardAction(
game.doAction(new PutOntoBattlefieldAction(
MagicLocationType.Graveyard,
card,
event.getPlayer(),

View File

@ -16,12 +16,12 @@
}
@Override
public void executeEvent(final MagicGame game, final MagicEvent event) {
game.doAction(new ReturnCardAction(
game.doAction(new PutOntoBattlefieldAction(
MagicLocationType.Graveyard,
event.getRefPermanent().getCard(),
event.getPlayer()
));
game.doAction(new ReturnCardAction(
game.doAction(new PutOntoBattlefieldAction(
MagicLocationType.Graveyard,
event.getPermanent().getCard(),
event.getPlayer(),

View File

@ -17,7 +17,7 @@ def DelayedTrigger = {
}
@Override
public void executeEvent(final MagicGame game, final MagicEvent event) {
game.doAction(new ReturnCardAction(
game.doAction(new PutOntoBattlefieldAction(
MagicLocationType.Exile,
event.getCard(),
event.getPlayer(),

View File

@ -27,7 +27,7 @@ def action = {
for (final MagicCard card : cards) {
game.doAction(new RevealAction(card));
if (card.hasType(MagicType.Creature) == true) {
game.doAction(new ReturnCardAction(MagicLocationType.OwnersLibrary, card, event.getPlayer()));
game.doAction(new PutOntoBattlefieldAction(MagicLocationType.OwnersLibrary, card, event.getPlayer()));
} else {
game.addEvent(new MagicEvent(
event.getSource(),

View File

@ -23,7 +23,7 @@
game.doAction(new RevealAction(topCard));
amount++;
if (predicate(topCard)) {
game.doAction(new ReturnCardAction(MagicLocationType.OwnersLibrary, topCard, player));
game.doAction(new PutOntoBattlefieldAction(MagicLocationType.OwnersLibrary, topCard, player));
break;
} else {
revealed.add(topCard);

View File

@ -14,7 +14,7 @@
@Override
public void executeEvent(final MagicGame game, final MagicEvent event) {
event.processTargetCard(game, {
game.doAction(new ReturnCardAction(MagicLocationType.Graveyard,it,event.getPlayer(),{
game.doAction(new PutOntoBattlefieldAction(MagicLocationType.Graveyard,it,event.getPlayer(),{
final MagicPermanent perm ->
final MagicGame G = perm.getGame();
G.doAction(new ChangeCountersAction(event.getPlayer().map(G), perm, MagicCounterType.PlusOne, 1));

View File

@ -63,7 +63,7 @@ def AN_EQUIPMENT_YOU_CONTROL = new MagicTargetChoice("an Equipment you control")
if (event.isYes()) {
event.processTargetCard(game, {
final MagicLocationType from = it.isInHand() ? MagicLocationType.OwnersHand : MagicLocationType.Graveyard;
game.doAction(new ReturnCardAction(from,it,event.getPlayer()));
game.doAction(new PutOntoBattlefieldAction(from,it,event.getPlayer()));
});
}
}

View File

@ -18,7 +18,7 @@ def choice = new MagicTargetChoice("an Equipment card from your library");
event.processTargetCard(game, {
game.doAction(new RevealAction(it));
if (it.getName().equals("Hammer of Nazahn")) {
game.doAction(new ReturnCardAction(MagicLocationType.OwnersLibrary,it,event.getPlayer()));
game.doAction(new PutOntoBattlefieldAction(MagicLocationType.OwnersLibrary,it,event.getPlayer()));
} else {
game.doAction(new ShiftCardAction(it, MagicLocationType.OwnersLibrary, MagicLocationType.OwnersHand));
}

View File

@ -18,7 +18,7 @@ def trigger = {
}
@Override
public void executeEvent(final MagicGame game, final MagicEvent event) {
game.doAction(new ReturnCardAction(
game.doAction(new PutOntoBattlefieldAction(
MagicLocationType.Exile,
event.getCard(),
event.getPlayer(),

View File

@ -17,7 +17,7 @@
game.doAction(new RevealAction(cards));
for (final MagicCard card : cards) {
if (card.hasType(MagicType.Land)) {
game.doAction(new ReturnCardAction(MagicLocationType.OwnersLibrary, card, player));
game.doAction(new PutOntoBattlefieldAction(MagicLocationType.OwnersLibrary, card, player));
} else {
game.doAction(new ShiftCardAction(card, MagicLocationType.OwnersLibrary, MagicLocationType.OwnersHand));
}

View File

@ -21,7 +21,7 @@ def DelayedTrigger = {
}
@Override
public void executeEvent(final MagicGame game, final MagicEvent event) {
game.doAction(new ReturnCardAction(
game.doAction(new PutOntoBattlefieldAction(
MagicLocationType.Exile,
event.getCard(),
event.getPlayer(),

View File

@ -18,7 +18,7 @@
public void executeEvent(final MagicGame game, final MagicEvent event) {
if (event.isYes()) {
event.processTargetCard(game,{
game.doAction(new ReturnCardAction(MagicLocationType.OwnersLibrary, it, event.getPlayer()));
game.doAction(new PutOntoBattlefieldAction(MagicLocationType.OwnersLibrary, it, event.getPlayer()));
game.doAction(new ShuffleLibraryAction(event.getPlayer()));
});
}

View File

@ -14,7 +14,7 @@
public void executeEvent(final MagicGame game, final MagicEvent event) {
event.processTargetPermanent(game, {
game.doAction(new RemoveFromPlayAction(it, MagicLocationType.Exile));
game.doAction(new ReturnCardAction(MagicLocationType.Exile, it.getCard(), it.getOwner()));
game.doAction(new PutOntoBattlefieldAction(MagicLocationType.Exile, it.getCard(), it.getOwner()));
if (it.hasSubType(MagicSubType.Pirate)) {
game.doAction(new DrawAction(event.getPlayer()));
}

View File

@ -25,7 +25,7 @@
for (final MagicCard card : cards) {
game.doAction(new RevealAction(card));
if (card.hasType(MagicType.Land)) {
game.doAction(new ReturnCardAction(MagicLocationType.OwnersLibrary, card, player));
game.doAction(new PutOntoBattlefieldAction(MagicLocationType.OwnersLibrary, card, player));
} else {
game.doAction(new ShiftCardAction(card, MagicLocationType.OwnersLibrary, MagicLocationType.OwnersHand));
}

View File

@ -23,7 +23,7 @@
game.doAction(new MillLibraryAction(it,1));
} else {
creatureCards++;
game.doAction(new ReturnCardAction(
game.doAction(new PutOntoBattlefieldAction(
MagicLocationType.OwnersLibrary,
library.getCardAtTop(),
event.getPlayer()

View File

@ -4,7 +4,7 @@ def action = {
for (final MagicCard card : player.getLibrary().getCardsFromTop(1)) {
game.doAction(new RevealAction(card));
if (card.hasType(MagicType.Land)) {
game.doAction(new ReturnCardAction(MagicLocationType.OwnersLibrary,card,event.getPlayer(),MagicPlayMod.TAPPED));
game.doAction(new PutOntoBattlefieldAction(MagicLocationType.OwnersLibrary,card,event.getPlayer(),MagicPlayMod.TAPPED));
} else {
game.doAction(new DrawAction(event.getPlayer()));
}

View File

@ -16,7 +16,7 @@
public void executeEvent(final MagicGame game, final MagicEvent event) {
final MagicCard card = event.getPermanent().getCard();
if (card.isInGraveyard()) {
game.doAction(new ReturnCardAction(
game.doAction(new PutOntoBattlefieldAction(
MagicLocationType.Graveyard,
event.getPermanent().getCard(),
event.getPlayer()

View File

@ -11,7 +11,7 @@
@Override
public void executeEvent(final MagicGame game, final MagicEvent event) {
card(MagicType.Land).from(MagicTargetType.Graveyard).filter(event.getPlayer()) each {
game.doAction(new ReturnCardAction(
game.doAction(new PutOntoBattlefieldAction(
MagicLocationType.Graveyard,
it,
event.getPlayer(),

View File

@ -25,7 +25,7 @@
for (final MagicCard card : event.getPlayer().getLibrary().getCardsFromTop(1)) {
game.doAction(new RevealAction(card));
if (card.hasType(MagicType.Creature)) {
game.doAction(new ReturnCardAction(
game.doAction(new PutOntoBattlefieldAction(
MagicLocationType.OwnersLibrary,
card,
event.getPlayer()

View File

@ -7,21 +7,21 @@ import magic.model.MagicGame;
import magic.model.MagicLocationType;
import magic.model.MagicPlayer;
public class ReturnCardAction extends MagicAction {
public class PutOntoBattlefieldAction extends MagicAction {
private final MagicPlayer controller;
private final MagicCard card;
private final MagicLocationType from;
private final List<? extends MagicPermanentAction> modifications;
public ReturnCardAction(final MagicLocationType aFrom, final MagicCard aCard, final MagicPlayer aController, final List<? extends MagicPermanentAction> aModifications) {
public PutOntoBattlefieldAction(final MagicLocationType aFrom, final MagicCard aCard, final MagicPlayer aController, final List<? extends MagicPermanentAction> aModifications) {
card = aCard;
controller = aController;
from = aFrom;
modifications = aModifications;
}
public ReturnCardAction(final MagicLocationType aFrom, final MagicCard aCard, final MagicPlayer aController, final MagicPermanentAction... aModifications) {
public PutOntoBattlefieldAction(final MagicLocationType aFrom, final MagicCard aCard, final MagicPlayer aController, final MagicPermanentAction... aModifications) {
this(aFrom, aCard, aController, Arrays.asList(aModifications));
}

View File

@ -45,7 +45,7 @@ public class ReturnLinkedExileAction extends MagicAction {
for (final MagicCard card : cardList) {
if (card.isInExile()) {
if (location == MagicLocationType.Battlefield) {
game.doAction(new ReturnCardAction(
game.doAction(new PutOntoBattlefieldAction(
MagicLocationType.Exile,
card,
controller.isValid() ? controller : card.getOwner(),

View File

@ -6,7 +6,7 @@ import magic.model.MagicLocationType;
import magic.model.MagicManaCost;
import magic.model.MagicPayedCost;
import magic.model.action.MagicPlayMod;
import magic.model.action.ReturnCardAction;
import magic.model.action.PutOntoBattlefieldAction;
import magic.model.choice.MagicTargetChoice;
import magic.model.condition.MagicCondition;
@ -52,7 +52,7 @@ public class MagicNinjutsuActivation extends MagicCardAbilityActivation {
@Override
public void executeEvent(final MagicGame game, final MagicEvent event) {
game.doAction(new ReturnCardAction(
game.doAction(new PutOntoBattlefieldAction(
MagicLocationType.OwnersHand,
event.getCard(),
event.getPlayer(),

View File

@ -10,7 +10,7 @@ import magic.model.MagicPlayer;
import magic.model.MagicSource;
import magic.model.MagicTuple;
import magic.model.action.MagicPermanentAction;
import magic.model.action.ReturnCardAction;
import magic.model.action.PutOntoBattlefieldAction;
import magic.model.choice.MagicChoice;
import magic.model.target.MagicGraveyardTargetPicker;
@ -45,7 +45,7 @@ public class MagicPutOntoBattlefieldEvent extends MagicEvent {
event.getPlayer(),
MagicMessage.format("Chosen (%s).", card)
);
game.doAction(new ReturnCardAction(MagicLocationType.OwnersHand,card,event.getPlayer(),tup.getMods()));
game.doAction(new PutOntoBattlefieldAction(MagicLocationType.OwnersHand,card,event.getPlayer(),tup.getMods()));
});
}
};

View File

@ -376,7 +376,7 @@ public enum MagicRuleEventAction {
it,
MagicLocationType.Exile
));
game.doAction(new ReturnCardAction(
game.doAction(new PutOntoBattlefieldAction(
MagicLocationType.Exile,
it.getCard(),
event.getPlayer()
@ -402,7 +402,7 @@ public enum MagicRuleEventAction {
it,
MagicLocationType.Exile
));
game.doAction(new ReturnCardAction(
game.doAction(new PutOntoBattlefieldAction(
MagicLocationType.Exile,
it.getCard(),
it.getOwner(),

View File

@ -10,7 +10,7 @@ import magic.model.MagicSource;
import magic.model.MagicTuple;
import magic.model.action.AIRevealAction;
import magic.model.action.MagicPermanentAction;
import magic.model.action.ReturnCardAction;
import magic.model.action.PutOntoBattlefieldAction;
import magic.model.action.ShuffleLibraryAction;
import magic.model.choice.MagicCardChoiceResult;
import magic.model.choice.MagicChoice;
@ -53,7 +53,7 @@ public class MagicSearchOntoBattlefieldEvent extends MagicEvent {
MagicMessage.format("Found (%s).", card)
);
game.doAction(new AIRevealAction(card));
game.doAction(new ReturnCardAction(card.getLocation(),card,event.getPlayer(),tup.getMods()));
game.doAction(new PutOntoBattlefieldAction(card.getLocation(),card,event.getPlayer(),tup.getMods()));
});
game.doAction(new ShuffleLibraryAction(event.getPlayer()));
} else {
@ -63,7 +63,7 @@ public class MagicSearchOntoBattlefieldEvent extends MagicEvent {
MagicMessage.format("Found (%s).", card)
);
game.doAction(new AIRevealAction(card));
game.doAction(new ReturnCardAction(card.getLocation(),card,event.getPlayer(),tup.getMods()));
game.doAction(new PutOntoBattlefieldAction(card.getLocation(),card,event.getPlayer(),tup.getMods()));
});
game.doAction(new ShuffleLibraryAction(event.getPlayer()));
}