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

added replace event to putOntoBattlefield call - moved logic for moving cards to putOntoBattlefield

Browse source
This commit is contained in:
BetaSteward 2012-03-05 09:15:47 -05:00
parent 11a64a1c6a
commit 82f8a7cdee
35 changed files with 50 additions and 75 deletions
Download patch file Download diff file Expand all files Collapse all files

2
Mage.Sets/src/mage/sets/conflux/PathToExile.java
View file

@ -94,7 +94,7 @@ class PathToExileEffect extends OneShotEffect {
if (player.chooseUse(Outcome.PutCardInPlay, "Use Path to Exile effect?", game)) { if (player.chooseUse(Outcome.PutCardInPlay, "Use Path to Exile effect?", game)) {
TargetCardInLibrary target = new TargetCardInLibrary(new FilterBasicLandCard()); TargetCardInLibrary target = new TargetCardInLibrary(new FilterBasicLandCard());
player.searchLibrary(target, game); player.searchLibrary(target, game);
Card card = player.getLibrary().remove(target.getFirstTarget(), game); Card card = player.getLibrary().getCard(target.getFirstTarget(), game);
if (card != null) { if (card != null) {
if (card.putOntoBattlefield(game, Zone.LIBRARY, source.getId(), permanent.getControllerId())) { if (card.putOntoBattlefield(game, Zone.LIBRARY, source.getId(), permanent.getControllerId())) {
Permanent land = game.getPermanent(card.getId()); Permanent land = game.getPermanent(card.getId());

6
Mage.Sets/src/mage/sets/darkascension/SorinLordOfInnistrad.java
View file

@ -153,11 +153,7 @@ class SorinLordOfInnistradEffect extends OneShotEffect<SorinLordOfInnistradEffec
if (game.getState().getZone(targetId) == Zone.GRAVEYARD) { if (game.getState().getZone(targetId) == Zone.GRAVEYARD) {
Card card = game.getCard(targetId); Card card = game.getCard(targetId);
if (card != null) { if (card != null) {
Player owner = game.getPlayer(card.getOwnerId()); card.putOntoBattlefield(game, Zone.GRAVEYARD, source.getId(), player.getId());
if (owner != null) {
owner.removeFromGraveyard(card, game);
card.putOntoBattlefield(game, Zone.GRAVEYARD, source.getId(), player.getId());
}
} }
} }
} }

1
Mage.Sets/src/mage/sets/darkascension/ZombieApocalypse.java
View file

@ -105,7 +105,6 @@ class ZombieApocalypseEffect extends OneShotEffect<ZombieApocalypseEffect> {
Player player = game.getPlayer(source.getControllerId()); Player player = game.getPlayer(source.getControllerId());
for (Card card : player.getGraveyard().getCards(filterZombie, game)) { for (Card card : player.getGraveyard().getCards(filterZombie, game)) {
player.getGraveyard().remove(card);
card.putOntoBattlefield(game, Constants.Zone.GRAVEYARD, source.getId(), source.getControllerId()); card.putOntoBattlefield(game, Constants.Zone.GRAVEYARD, source.getId(), source.getControllerId());
Permanent permanent = game.getPermanent(card.getId()); Permanent permanent = game.getPermanent(card.getId());
if (permanent != null) { if (permanent != null) {

1
Mage.Sets/src/mage/sets/fifthdawn/FoldIntoAEther.java
View file

@ -100,7 +100,6 @@ class FoldIntoAEtherEffect extends OneShotEffect<FoldIntoAEtherEffect> {
&& player.choose(Outcome.PutCreatureInPlay, target, source.getSourceId(), game)) { && player.choose(Outcome.PutCreatureInPlay, target, source.getSourceId(), game)) {
Card card = game.getCard(target.getFirstTarget()); Card card = game.getCard(target.getFirstTarget());
if (card != null) { if (card != null) {
player.removeFromHand(card, game);
card.putOntoBattlefield(game, Zone.HAND, source.getId(), source.getControllerId()); card.putOntoBattlefield(game, Zone.HAND, source.getId(), source.getControllerId());
} }
} }

1
Mage.Sets/src/mage/sets/innistrad/BitterheartWitch.java
View file

@ -110,7 +110,6 @@ class BitterheartWitchEffect extends OneShotEffect<BitterheartWitchEffect> {
player.chooseTarget(Outcome.Detriment, target, source, game); player.chooseTarget(Outcome.Detriment, target, source, game);
Player targetPlayer = game.getPlayer(target.getFirstTarget()); Player targetPlayer = game.getPlayer(target.getFirstTarget());
if (targetPlayer != null) { if (targetPlayer != null) {
player.removeFromLibrary(card, game);
player.shuffleLibrary(game); player.shuffleLibrary(game);
card.putOntoBattlefield(game, Zone.LIBRARY, source.getSourceId(), source.getControllerId()); card.putOntoBattlefield(game, Zone.LIBRARY, source.getSourceId(), source.getControllerId());
return targetPlayer.addAttachment(card.getId(), game); return targetPlayer.addAttachment(card.getId(), game);

2
Mage.Sets/src/mage/sets/innistrad/CaravanVigil.java
View file

@ -95,7 +95,7 @@ class CaravanVigilEffect extends OneShotEffect<CaravanVigilEffect> {
TargetCardInLibrary target = new TargetCardInLibrary(new FilterBasicLandCard()); TargetCardInLibrary target = new TargetCardInLibrary(new FilterBasicLandCard());
if (player.searchLibrary(target, game)) { if (player.searchLibrary(target, game)) {
Cards cards = new CardsImpl(); Cards cards = new CardsImpl();
Card card = player.getLibrary().remove(target.getFirstTarget(), game); Card card = player.getLibrary().getCard(target.getFirstTarget(), game);
if (card != null) { if (card != null) {
cards.add(card); cards.add(card);
if (MorbidCondition.getInstance().apply(game, source) if (MorbidCondition.getInstance().apply(game, source)

2
Mage.Sets/src/mage/sets/innistrad/GhostQuarter.java
View file

@ -102,7 +102,7 @@ class GhostQuarterEffect extends OneShotEffect<GhostQuarterEffect> {
if (player.chooseUse(Outcome.PutLandInPlay, "Do you wish to search for a basic land, put it onto the battlefield and then shuffle your library?", game)) { if (player.chooseUse(Outcome.PutLandInPlay, "Do you wish to search for a basic land, put it onto the battlefield and then shuffle your library?", game)) {
TargetCardInLibrary target = new TargetCardInLibrary(new FilterBasicLandCard()); TargetCardInLibrary target = new TargetCardInLibrary(new FilterBasicLandCard());
if (player.searchLibrary(target, game)) { if (player.searchLibrary(target, game)) {
Card card = player.getLibrary().remove(target.getFirstTarget(), game); Card card = player.getLibrary().getCard(target.getFirstTarget(), game);
if (card != null) { if (card != null) {
card.putOntoBattlefield(game, Zone.HAND, source.getId(), player.getId()); card.putOntoBattlefield(game, Zone.HAND, source.getId(), player.getId());
} }

1
Mage.Sets/src/mage/sets/innistrad/GrimoireOfTheDead.java
View file

@ -104,7 +104,6 @@ class GrimoireOfTheDeadEffect extends OneShotEffect<GrimoireOfTheDeadEffect> {
for (Player player: game.getPlayers().values()) { for (Player player: game.getPlayers().values()) {
for (Card card: player.getGraveyard().getCards(game)) { for (Card card: player.getGraveyard().getCards(game)) {
if (card.getCardType().contains(CardType.CREATURE)) { if (card.getCardType().contains(CardType.CREATURE)) {
player.getGraveyard().remove(card);
card.putOntoBattlefield(game, Zone.GRAVEYARD, source.getId(), source.getControllerId()); card.putOntoBattlefield(game, Zone.GRAVEYARD, source.getId(), source.getControllerId());
game.addEffect(new GrimoireOfTheDeadEffect2(card.getId()), source); game.addEffect(new GrimoireOfTheDeadEffect2(card.getId()), source);
} }

1
Mage.Sets/src/mage/sets/magic2010/ElvishPiper.java
View file

@ -101,7 +101,6 @@ class PutCreatureOnBattlefieldEffect extends OneShotEffect<PutCreatureOnBattlefi
Card card = game.getCard(source.getFirstTarget()); Card card = game.getCard(source.getFirstTarget());
Player player = game.getPlayer(card.getOwnerId()); Player player = game.getPlayer(card.getOwnerId());
if (card != null && player != null) { if (card != null && player != null) {
player.removeFromHand(card, game);
card.putOntoBattlefield(game, Zone.HAND, source.getId(), source.getControllerId()); card.putOntoBattlefield(game, Zone.HAND, source.getId(), source.getControllerId());
return true; return true;
} }

1
Mage.Sets/src/mage/sets/magic2010/LilianaVess.java
View file

@ -98,7 +98,6 @@ class LilianaVessEffect extends OneShotEffect<LilianaVessEffect> {
for (Player player: game.getPlayers().values()) { for (Player player: game.getPlayers().values()) {
for (Card card: player.getGraveyard().getCards(game)) { for (Card card: player.getGraveyard().getCards(game)) {
if (card.getCardType().contains(CardType.CREATURE)) { if (card.getCardType().contains(CardType.CREATURE)) {
player.getGraveyard().remove(card);
card.putOntoBattlefield(game, Zone.GRAVEYARD, source.getId(), source.getControllerId()); card.putOntoBattlefield(game, Zone.GRAVEYARD, source.getId(), source.getControllerId());
} }
} }

2
Mage.Sets/src/mage/sets/magic2010/LurkingPredators.java
View file

@ -130,10 +130,8 @@ class LurkingPredatorsEffect extends OneShotEffect<LurkingPredatorsEffect> {
if (card != null) { if (card != null) {
if (card.getCardType().contains(CardType.CREATURE)) { if (card.getCardType().contains(CardType.CREATURE)) {
player.getLibrary().remove(card.getId(), game);
card.putOntoBattlefield(game, Zone.HAND, source.getId(), source.getControllerId()); card.putOntoBattlefield(game, Zone.HAND, source.getId(), source.getControllerId());
} else if (player.chooseUse(Outcome.Neutral, "Put " + card.getName() + " on the bottom of your library?", game)) { } else if (player.chooseUse(Outcome.Neutral, "Put " + card.getName() + " on the bottom of your library?", game)) {
player.getLibrary().remove(card.getId(), game);
card.moveToZone(Zone.LIBRARY, source.getId(), game, false); card.moveToZone(Zone.LIBRARY, source.getId(), game, false);
} }
} }

2
Mage.Sets/src/mage/sets/magic2011/Cultivate.java
View file

@ -99,7 +99,7 @@ class CultivateEffect extends OneShotEffect<CultivateEffect> {
if (target.getTargets().size() > 0) { if (target.getTargets().size() > 0) {
Cards revealed = new CardsImpl(); Cards revealed = new CardsImpl();
for (UUID cardId: (List<UUID>)target.getTargets()) { for (UUID cardId: (List<UUID>)target.getTargets()) {
Card card = player.getLibrary().remove(cardId, game); Card card = player.getLibrary().getCard(cardId, game);
revealed.add(card); revealed.add(card);
} }
player.revealCards("Cultivate", revealed, game); player.revealCards("Cultivate", revealed, game);

2
Mage.Sets/src/mage/sets/magic2012/ArachnusSpinner.java
View file

@ -124,7 +124,6 @@ class ArachnusSpinnerEffect extends OneShotEffect<ArachnusSpinnerEffect> {
if (player.choose(Outcome.PutCardInPlay, player.getGraveyard(), target, game)) { if (player.choose(Outcome.PutCardInPlay, player.getGraveyard(), target, game)) {
card = game.getCard(target.getFirstTarget()); card = game.getCard(target.getFirstTarget());
if (card != null) { if (card != null) {
player.removeFromGraveyard(card, game);
zone = Zone.GRAVEYARD; zone = Zone.GRAVEYARD;
} }
} }
@ -134,7 +133,6 @@ class ArachnusSpinnerEffect extends OneShotEffect<ArachnusSpinnerEffect> {
if (player.searchLibrary(target, game)) { if (player.searchLibrary(target, game)) {
card = game.getCard(target.getFirstTarget()); card = game.getCard(target.getFirstTarget());
if (card != null) { if (card != null) {
player.removeFromLibrary(card, game);
player.shuffleLibrary(game); player.shuffleLibrary(game);
zone = Zone.LIBRARY; zone = Zone.LIBRARY;
} }

2
Mage.Sets/src/mage/sets/magic2012/DoublingChant.java
View file

@ -108,7 +108,7 @@ class DoublingChantEffect extends OneShotEffect<DoublingChantEffect> {
TargetCardInLibrary target = new TargetCardInLibrary(filter); TargetCardInLibrary target = new TargetCardInLibrary(filter);
if (player.searchLibrary(target, game)) { if (player.searchLibrary(target, game)) {
Card card = player.getLibrary().remove(target.getFirstTarget(), game); Card card = player.getLibrary().getCard(target.getFirstTarget(), game);
if (card != null) { if (card != null) {
chosenCards.add(card); chosenCards.add(card);
} }

2
Mage.Sets/src/mage/sets/magic2012/DruidicSatchel.java
View file

@ -92,7 +92,7 @@ class DruidicSatchelEffect extends OneShotEffect<DruidicSatchelEffect> {
token.putOntoBattlefield(1, game, source.getSourceId(), source.getControllerId()); token.putOntoBattlefield(1, game, source.getSourceId(), source.getControllerId());
} }
if (card.getCardType().contains(CardType.LAND)) { if (card.getCardType().contains(CardType.LAND)) {
player.getLibrary().remove(card.getId(), game); player.getLibrary().getCard(card.getId(), game);
card.putOntoBattlefield(game, Zone.LIBRARY, source.getId(), source.getControllerId()); card.putOntoBattlefield(game, Zone.LIBRARY, source.getId(), source.getControllerId());
} }
if (!card.getCardType().contains(CardType.CREATURE) && !card.getCardType().contains(CardType.LAND)) { if (!card.getCardType().contains(CardType.CREATURE) && !card.getCardType().contains(CardType.LAND)) {

1
Mage.Sets/src/mage/sets/magic2012/QuicksilverAmulet.java
View file

@ -101,7 +101,6 @@ class PutCreatureOnBattlefieldEffect extends OneShotEffect<PutCreatureOnBattlefi
if (player.choose(Outcome.PutCreatureInPlay, target, source.getSourceId(), game)) { if (player.choose(Outcome.PutCreatureInPlay, target, source.getSourceId(), game)) {
Card card = game.getCard(target.getFirstTarget()); Card card = game.getCard(target.getFirstTarget());
if (card != null) { if (card != null) {
player.removeFromHand(card, game);
card.putOntoBattlefield(game, Zone.HAND, source.getId(), source.getControllerId()); card.putOntoBattlefield(game, Zone.HAND, source.getId(), source.getControllerId());
return true; return true;
} }

2
Mage.Sets/src/mage/sets/mirrodinbesieged/GreenSunsZenith.java
View file

@ -93,7 +93,7 @@ class GreenSunsZenithSearchEffect extends OneShotEffect<GreenSunsZenithSearchEff
if (player.searchLibrary(target, game)) { if (player.searchLibrary(target, game)) {
if (target.getTargets().size() > 0) { if (target.getTargets().size() > 0) {
for (UUID cardId : (List<UUID>) target.getTargets()) { for (UUID cardId : (List<UUID>) target.getTargets()) {
Card card = player.getLibrary().remove(cardId, game); Card card = player.getLibrary().getCard(cardId, game);
card.putOntoBattlefield(game, Constants.Zone.HAND, source.getId(), source.getControllerId()); card.putOntoBattlefield(game, Constants.Zone.HAND, source.getId(), source.getControllerId());
} }
player.shuffleLibrary(game); player.shuffleLibrary(game);

21
Mage.Sets/src/mage/sets/mirrodinbesieged/GruesomeEncore.java
View file

@ -99,20 +99,17 @@ class GruesomeEncoreEffect extends OneShotEffect<GruesomeEncoreEffect> {
public boolean apply(Game game, Ability source) { public boolean apply(Game game, Ability source) {
Card card = game.getCard(source.getFirstTarget()); Card card = game.getCard(source.getFirstTarget());
if (card != null) { if (card != null) {
Player opponent = game.getPlayer(card.getOwnerId()); card.addAbility(HasteAbility.getInstance());
if (opponent != null && opponent.removeFromGraveyard(card, game)) { card.putOntoBattlefield(game, Zone.GRAVEYARD, source.getId(), source.getControllerId());
card.addAbility(HasteAbility.getInstance());
card.putOntoBattlefield(game, Zone.GRAVEYARD, source.getId(), source.getControllerId());
ExileTargetEffect exileEffect = new ExileTargetEffect(); ExileTargetEffect exileEffect = new ExileTargetEffect();
exileEffect.setTargetPointer(new FixedTarget(card.getId())); exileEffect.setTargetPointer(new FixedTarget(card.getId()));
DelayedTriggeredAbility delayedAbility = new AtEndOfTurnDelayedTriggeredAbility(exileEffect); DelayedTriggeredAbility delayedAbility = new AtEndOfTurnDelayedTriggeredAbility(exileEffect);
delayedAbility.setSourceId(source.getSourceId()); delayedAbility.setSourceId(source.getSourceId());
delayedAbility.setControllerId(source.getControllerId()); delayedAbility.setControllerId(source.getControllerId());
game.addDelayedTriggeredAbility(delayedAbility); game.addDelayedTriggeredAbility(delayedAbility);
return true; return true;
}
} }
return false; return false;

1
Mage.Sets/src/mage/sets/morningtide/PreeminentCaptain.java
View file

@ -101,7 +101,6 @@ class PreeminentCaptainEffect extends OneShotEffect<PreeminentCaptainEffect> {
UUID cardId = target.getFirstTarget(); UUID cardId = target.getFirstTarget();
Card card = player.getHand().get(cardId, game); Card card = player.getHand().get(cardId, game);
if (card != null) { if (card != null) {
player.getHand().remove(card);
if (card.putOntoBattlefield(game, Zone.HAND, if (card.putOntoBattlefield(game, Zone.HAND,
source.getId(), source.getControllerId())) { source.getId(), source.getControllerId())) {
Permanent permanent = game.getPermanent(card.getId()); Permanent permanent = game.getPermanent(card.getId());

2
Mage.Sets/src/mage/sets/newphyrexia/BirthingPod.java
View file

@ -110,7 +110,7 @@ class BirthingPodEffect extends OneShotEffect<BirthingPodEffect> {
TargetCardInLibrary target = new TargetCardInLibrary(filter); TargetCardInLibrary target = new TargetCardInLibrary(filter);
if (player.searchLibrary(target, game)) { if (player.searchLibrary(target, game)) {
for (UUID cardId : target.getTargets()) { for (UUID cardId : target.getTargets()) {
Card card = player.getLibrary().remove(cardId, game); Card card = player.getLibrary().getCard(cardId, game);
if (card != null) { if (card != null) {
card.putOntoBattlefield(game, Constants.Zone.HAND, source.getId(), source.getControllerId()); card.putOntoBattlefield(game, Constants.Zone.HAND, source.getId(), source.getControllerId());
} }

1
Mage.Sets/src/mage/sets/newphyrexia/OmenMachine.java
View file

@ -159,7 +159,6 @@ class OmenMachineEffect2 extends OneShotEffect<OmenMachineEffect2> {
card.moveToExile(source.getSourceId(), "Omen Machine Exile", source.getId(), game); card.moveToExile(source.getSourceId(), "Omen Machine Exile", source.getId(), game);
if (card.getCardType().contains(CardType.LAND)) { if (card.getCardType().contains(CardType.LAND)) {
card.putOntoBattlefield(game, Zone.EXILED, source.getId(), player.getId()); card.putOntoBattlefield(game, Zone.EXILED, source.getId(), player.getId());
game.getExile().removeCard(card, game);
} }
else { else {
player.cast(card.getSpellAbility(), game, true); player.cast(card.getSpellAbility(), game, true);

2
Mage.Sets/src/mage/sets/newphyrexia/PostmortemLunge.java
View file

@ -110,7 +110,7 @@ class PostmortemLungeEffect extends OneShotEffect<PostmortemLungeEffect> {
public boolean apply(Game game, Ability source) { public boolean apply(Game game, Ability source) {
Card card = game.getCard(source.getFirstTarget()); Card card = game.getCard(source.getFirstTarget());
Player player = game.getPlayer(card.getOwnerId()); Player player = game.getPlayer(card.getOwnerId());
if (card != null && player != null && player.removeFromGraveyard(card, game)) { if (card != null && player != null) {
card.addAbility(HasteAbility.getInstance()); card.addAbility(HasteAbility.getInstance());
card.putOntoBattlefield(game, Zone.GRAVEYARD, source.getId(), source.getControllerId()); card.putOntoBattlefield(game, Zone.GRAVEYARD, source.getId(), source.getControllerId());

1
Mage.Sets/src/mage/sets/scarsofmirrodin/GethLordOfTheVault.java
View file

@ -125,7 +125,6 @@ class GethLordOfTheVaultEffect extends OneShotEffect<GethLordOfTheVaultEffect> {
if (game.getState().getZone(card.getId()).equals(Zone.GRAVEYARD)) { if (game.getState().getZone(card.getId()).equals(Zone.GRAVEYARD)) {
Player player = game.getPlayer(card.getOwnerId()); Player player = game.getPlayer(card.getOwnerId());
if (card.putOntoBattlefield(game, Zone.GRAVEYARD, source.getId(), source.getControllerId())) { if (card.putOntoBattlefield(game, Zone.GRAVEYARD, source.getId(), source.getControllerId())) {
player.getGraveyard().remove(card);
Permanent permanent = game.getPermanent(card.getId()); Permanent permanent = game.getPermanent(card.getId());
if (permanent != null) { if (permanent != null) {

1
Mage.Sets/src/mage/sets/scarsofmirrodin/NimDeathmantle.java
View file

@ -167,7 +167,6 @@ class NimDeathmantleEffect extends OneShotEffect<NimDeathmantleEffect> {
if (card != null && game.getState().getZone(card.getId()).equals(Constants.Zone.GRAVEYARD)) { if (card != null && game.getState().getZone(card.getId()).equals(Constants.Zone.GRAVEYARD)) {
Player owner = game.getPlayer(card.getOwnerId()); Player owner = game.getPlayer(card.getOwnerId());
if (card.putOntoBattlefield(game, Constants.Zone.GRAVEYARD, source.getId(), source.getControllerId())) { if (card.putOntoBattlefield(game, Constants.Zone.GRAVEYARD, source.getId(), source.getControllerId())) {
owner.getGraveyard().remove(card);
Permanent permanent = game.getPermanent(card.getId()); Permanent permanent = game.getPermanent(card.getId());
if (permanent != null) { if (permanent != null) {
permanent.addAttachment(equipment.getId(), game); permanent.addAttachment(equipment.getId(), game);

2
Mage.Sets/src/mage/sets/shardsofalara/ClarionUltimatum.java
View file

@ -118,7 +118,7 @@ class ClarionUltimatumEffect extends OneShotEffect<ClarionUltimatumEffect> {
TargetCardInLibrary target = new TargetCardInLibrary(filter); TargetCardInLibrary target = new TargetCardInLibrary(filter);
if (player.searchLibrary(target, game)) { if (player.searchLibrary(target, game)) {
Card card = player.getLibrary().remove(target.getFirstTarget(), game); Card card = player.getLibrary().getCard(target.getFirstTarget(), game);
if (card != null) { if (card != null) {
chosenCards.add(card); chosenCards.add(card);
} }

2
Mage.Sets/src/mage/sets/shardsofalara/TezzeretTheSeeker.java
View file

@ -127,7 +127,7 @@ class TezzeretTheSeekerEffect2 extends OneShotEffect<TezzeretTheSeekerEffect2> {
TargetCardInLibrary target = new TargetCardInLibrary(filter); TargetCardInLibrary target = new TargetCardInLibrary(filter);
if (player.searchLibrary(target, game)) { if (player.searchLibrary(target, game)) {
Card card = player.getLibrary().remove(target.getFirstTarget(), game); Card card = player.getLibrary().getCard(target.getFirstTarget(), game);
if (card != null) { if (card != null) {
card.putOntoBattlefield(game, Zone.HAND, source.getId(), source.getControllerId()); card.putOntoBattlefield(game, Zone.HAND, source.getId(), source.getControllerId());
} }

1
Mage.Sets/src/mage/sets/zendikar/ExplorersScope.java
View file

@ -134,7 +134,6 @@ class ExplorersScopeEffect extends OneShotEffect<ExplorersScopeEffect> {
if (card.getCardType().contains(CardType.LAND)) { if (card.getCardType().contains(CardType.LAND)) {
String message = "Put " + card.getName() + " onto the battlefield tapped?"; String message = "Put " + card.getName() + " onto the battlefield tapped?";
if (player.chooseUse(Outcome.PutLandInPlay, message, game)) { if (player.chooseUse(Outcome.PutLandInPlay, message, game)) {
player.getLibrary().remove(card.getId(), game);
if (card.putOntoBattlefield(game, Zone.LIBRARY, source.getId(), source.getControllerId())) { if (card.putOntoBattlefield(game, Zone.LIBRARY, source.getId(), source.getControllerId())) {
Permanent permanent = game.getPermanent(card.getId()); Permanent permanent = game.getPermanent(card.getId());
if (permanent != null) { if (permanent != null) {

2
Mage.Sets/src/mage/sets/zendikar/QuestForTheHolyRelic.java
View file

@ -110,7 +110,7 @@ class QuestForTheHolyRelicEffect extends OneShotEffect<QuestForTheHolyRelicEffec
filter.getSubtype().add("Equipment"); filter.getSubtype().add("Equipment");
TargetCardInLibrary target = new TargetCardInLibrary(filter); TargetCardInLibrary target = new TargetCardInLibrary(filter);
if (player.searchLibrary(target, game)) { if (player.searchLibrary(target, game)) {
Card card = player.getLibrary().remove(target.getFirstTarget(), game); Card card = player.getLibrary().getCard(target.getFirstTarget(), game);
if (card != null) { if (card != null) {
card.putOntoBattlefield(game, Zone.LIBRARY, source.getId(), source.getControllerId()); card.putOntoBattlefield(game, Zone.LIBRARY, source.getId(), source.getControllerId());
Permanent equipment = game.getPermanent(card.getId()); Permanent equipment = game.getPermanent(card.getId());

1
Mage.Sets/src/mage/sets/zendikar/WarrenInstigator.java
View file

@ -134,7 +134,6 @@ class WarrenInstigatorEffect extends OneShotEffect<WarrenInstigatorEffect> {
if (player.choose(Outcome.PutCreatureInPlay, target, source.getSourceId(), game)) { if (player.choose(Outcome.PutCreatureInPlay, target, source.getSourceId(), game)) {
Card card = game.getCard(target.getFirstTarget()); Card card = game.getCard(target.getFirstTarget());
if (card != null) { if (card != null) {
player.removeFromHand(card, game);
card.putOntoBattlefield(game, Zone.HAND, source.getId(), source.getControllerId()); card.putOntoBattlefield(game, Zone.HAND, source.getId(), source.getControllerId());
return true; return true;
} }

1
Mage/src/mage/abilities/effects/common/ReturnFromGraveyardToBattlefieldTargetEffect.java
View file

@ -71,7 +71,6 @@ public class ReturnFromGraveyardToBattlefieldTargetEffect extends OneShotEffect<
if (card != null) { if (card != null) {
Player player = game.getPlayer(card.getOwnerId()); Player player = game.getPlayer(card.getOwnerId());
if (player != null) { if (player != null) {
player.removeFromGraveyard(card, game);
if (card.putOntoBattlefield(game, Zone.GRAVEYARD, source.getId(), source.getControllerId())) { if (card.putOntoBattlefield(game, Zone.GRAVEYARD, source.getId(), source.getControllerId())) {
if (tapped) { if (tapped) {
Permanent permanent = game.getPermanent(card.getId()); Permanent permanent = game.getPermanent(card.getId());

1
Mage/src/mage/abilities/effects/common/ReturnSourceFromGraveyardToBattlefieldEffect.java
View file

@ -70,7 +70,6 @@ public class ReturnSourceFromGraveyardToBattlefieldEffect extends OneShotEffect<
Player player = game.getPlayer(source.getControllerId()); Player player = game.getPlayer(source.getControllerId());
Card card = player.getGraveyard().get(source.getSourceId(), game); Card card = player.getGraveyard().get(source.getSourceId(), game);
if (card != null) { if (card != null) {
player.removeFromGraveyard(card, game);
if (card.putOntoBattlefield(game, Zone.GRAVEYARD, source.getId(), source.getControllerId())) { if (card.putOntoBattlefield(game, Zone.GRAVEYARD, source.getId(), source.getControllerId())) {
if (tapped) { if (tapped) {
Permanent permanent = game.getPermanent(card.getId()); Permanent permanent = game.getPermanent(card.getId());

2
Mage/src/mage/abilities/effects/common/search/SearchLibraryPutInPlayEffect.java
View file

@ -78,7 +78,7 @@ public class SearchLibraryPutInPlayEffect extends SearchEffect<SearchLibraryPutI
if (player.searchLibrary(target, game)) { if (player.searchLibrary(target, game)) {
if (target.getTargets().size() > 0) { if (target.getTargets().size() > 0) {
for (UUID cardId: (List<UUID>)target.getTargets()) { for (UUID cardId: (List<UUID>)target.getTargets()) {
Card card = player.getLibrary().remove(cardId, game); Card card = player.getLibrary().getCard(cardId, game);
if (card != null) { if (card != null) {
if (card.putOntoBattlefield(game, Zone.HAND, source.getId(), source.getControllerId())) { if (card.putOntoBattlefield(game, Zone.HAND, source.getId(), source.getControllerId())) {
if (tapped) { if (tapped) {

34
Mage/src/mage/cards/CardImpl.java
View file

@ -348,13 +348,33 @@ public abstract class CardImpl<T extends CardImpl<T>> extends MageObjectImpl<T>
@Override @Override
public boolean putOntoBattlefield(Game game, Zone fromZone, UUID sourceId, UUID controllerId) { public boolean putOntoBattlefield(Game game, Zone fromZone, UUID sourceId, UUID controllerId) {
PermanentCard permanent = new PermanentCard(this, controllerId); ZoneChangeEvent event = new ZoneChangeEvent(this.objectId, sourceId, controllerId, fromZone, Zone.BATTLEFIELD);
game.getBattlefield().addPermanent(permanent); if (!game.replaceEvent(event)) {
game.setZone(objectId, Zone.BATTLEFIELD); if (fromZone != null) {
game.applyEffects(); switch (fromZone) {
permanent.entersBattlefield(sourceId, game); case GRAVEYARD:
game.fireEvent(new ZoneChangeEvent(permanent, controllerId, fromZone, Zone.BATTLEFIELD)); game.getPlayer(ownerId).removeFromGraveyard(this, game);
return true; break;
case HAND:
game.getPlayer(ownerId).removeFromHand(this, game);
break;
case LIBRARY:
game.getPlayer(ownerId).removeFromLibrary(this, game);
break;
default:
//logger.warning("putOntoBattlefield, not fully implemented: from="+fromZone);
}
game.rememberLKI(objectId, event.getFromZone(), this);
}
PermanentCard permanent = new PermanentCard(this, controllerId);
game.getBattlefield().addPermanent(permanent);
game.setZone(objectId, Zone.BATTLEFIELD);
game.applyEffects();
permanent.entersBattlefield(sourceId, game);
game.fireEvent(new ZoneChangeEvent(permanent, controllerId, fromZone, Zone.BATTLEFIELD));
return true;
}
return false;
} }
@Override @Override

1
Mage/src/mage/game/GameImpl.java
View file

@ -505,7 +505,6 @@ public abstract class GameImpl<T extends GameImpl<T>> implements Game, Serializa
for (Card card: player.getHand().getCards(this)) { for (Card card: player.getHand().getCards(this)) {
if (card.getAbilities().containsKey(LeylineAbility.getInstance().getId())) { if (card.getAbilities().containsKey(LeylineAbility.getInstance().getId())) {
if (player.chooseUse(Outcome.PutCardInPlay, "Do you wish to put " + card.getName() + " on the battlefield?", this)) { if (player.chooseUse(Outcome.PutCardInPlay, "Do you wish to put " + card.getName() + " on the battlefield?", this)) {
player.getHand().remove(card.getId());
card.putOntoBattlefield(this, Zone.HAND, null, player.getId()); card.putOntoBattlefield(this, Zone.HAND, null, player.getId());
} }
} }

18
Mage/src/mage/players/PlayerImpl.java
View file

@ -512,24 +512,6 @@ public abstract class PlayerImpl<T extends PlayerImpl<T>> implements Player, Ser
if (!game.replaceEvent(GameEvent.getEvent(GameEvent.EventType.PLAY_LAND, card.getId(), playerId))) { if (!game.replaceEvent(GameEvent.getEvent(GameEvent.EventType.PLAY_LAND, card.getId(), playerId))) {
int bookmark = game.bookmarkState(); int bookmark = game.bookmarkState();
Zone zone = game.getState().getZone(card.getId()); Zone zone = game.getState().getZone(card.getId());
switch (zone) {
case HAND:
removeFromHand(card, game);
break;
case LIBRARY:
removeFromLibrary(card, game);
break;
case GRAVEYARD:
removeFromGraveyard(card, game);
break;
case EXILED:
game.getExile().removeCard(card, game);
break;
default:
// invalid zone for play land
return false;
}
if (card.putOntoBattlefield(game, zone, null, playerId)) { if (card.putOntoBattlefield(game, zone, null, playerId)) {
landsPlayed++; landsPlayed++;
game.fireEvent(GameEvent.getEvent(GameEvent.EventType.LAND_PLAYED, card.getId(), playerId)); game.fireEvent(GameEvent.getEvent(GameEvent.EventType.LAND_PLAYED, card.getId(), playerId));