github/mage
1
0
Fork 0
mirror of https://github.com/correl/mage.git synced 2024-11-15 19:19:33 +00:00
Code Issues Projects Releases Packages Wiki Activity

fixed ExchangeControlTargetEffect (exchange effect worked after recasting card again, copy() missed a variable, so game state save was incomplete)

Browse source
This commit is contained in:
LevelX2 2012-10-02 15:44:01 +02:00
parent 52641eafc6
commit 847d55d8a8
1 changed files with 8 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
Mage/src/mage/abilities/effects/common/continious/ExchangeControlTargetEffect.java
View file

@ -62,6 +62,7 @@ public class ExchangeControlTargetEffect extends ContinuousEffectImpl<ExchangeCo
super(effect);
this.rule = effect.rule;
this.withSource = effect.withSource;
this.lockedControllers = effect.lockedControllers;
}
@Override
@ -115,6 +116,7 @@ public class ExchangeControlTargetEffect extends ContinuousEffectImpl<ExchangeCo
@Override
public boolean apply(Game game, Ability source) {
int countChangeControl = 0;
Map<UUID, UUID> remainingLockedControllers = new HashMap<UUID, UUID>();
if (this.lockedControllers != null) {
for (UUID permanentId : targetPointer.getTargets(game, source)) {
Permanent permanent = game.getPermanent(permanentId);
@ -122,6 +124,7 @@ public class ExchangeControlTargetEffect extends ContinuousEffectImpl<ExchangeCo
UUID controllerId = this.lockedControllers.get(permanent.getId());
if (controllerId != null) {
if(permanent.changeControllerId(controllerId, game)) {
remainingLockedControllers.put(permanentId, controllerId);
++countChangeControl;
}
}
@ -133,13 +136,17 @@ public class ExchangeControlTargetEffect extends ContinuousEffectImpl<ExchangeCo
UUID controllerId = this.lockedControllers.get(permanent.getId());
if (controllerId != null) {
if (permanent.changeControllerId(controllerId, game)) {
remainingLockedControllers.put(source.getSourceId(), controllerId);
++countChangeControl;
}
}
}
}
}
if (countChangeControl == 2) {
// if the permanent is not existent (card in hand or graveyard) the control changing effect has to end
// else a previous controlled card that will be enter the battlefield again will be immediately be affected
this.lockedControllers = remainingLockedControllers;
if (countChangeControl > 0) {
return true;
}
return false;