External/mage
1
0
Fork 1
mirror of https://github.com/magefree/mage.git synced 2025-12-22 03:22:00 -08:00
Code Issues Projects Releases Packages Wiki Activity Actions

SacrificeEffect target required fix

Browse source
This commit is contained in:
magenoxx 2012-05-27 15:36:41 +04:00
parent c4cbc2a34e
commit cb63b900cd
2 changed files with 24 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

23
Mage.Tests/src/test/java/org/Test.java Normal file
View file

@ -0,0 +1,23 @@
package org
import mage.Constants
import mage.game.match.MatchOptions
import mage.game.tournament.LimitedOptions
import mage.view.SeatView
import java.util.ArrayList
import java.util.Date
import java.util.List
import java.util.UUID
/**
* Created by IntelliJ IDEA.
* User: ayratn
* Date: 5/27/12
* Time: 8:44 AM
* To change this template use File | Settings | File Templates.
*/
public interface Test {
}

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

@ -86,6 +86,7 @@ public class SacrificeEffect extends OneShotEffect<SacrificeEffect>{
amount = Math.min(amount, realCount); amount = Math.min(amount, realCount);
Target target = new TargetControlledPermanent(amount, amount, filter, false); Target target = new TargetControlledPermanent(amount, amount, filter, false);
target.setRequired(true);
//A spell or ability could have removed the only legal target this player //A spell or ability could have removed the only legal target this player
//had, if thats the case this ability should fizzle. //had, if thats the case this ability should fizzle.