mirror of
https://github.com/correl/mage.git
synced 2024-12-26 03:00:11 +00:00
Merge pull request #1844 from drmDev/master
SilentSkimmer and PyreHound bug fixes
This commit is contained in:
commit
90507f4c55
4 changed files with 6 additions and 4 deletions
|
@ -60,7 +60,6 @@ public class SilentSkimmer extends CardImpl {
|
|||
this.addAbility(FlyingAbility.getInstance());
|
||||
// Whenever Silent Skimmer attacks, defending player loses 2 life.
|
||||
this.addAbility(new AttacksTriggeredAbility(new LoseLifeDefendingPlayerEffect(2, true), false));
|
||||
|
||||
}
|
||||
|
||||
public SilentSkimmer(final SilentSkimmer card) {
|
||||
|
|
|
@ -56,7 +56,7 @@ public class PyreHound extends CardImpl {
|
|||
this.addAbility(TrampleAbility.getInstance());
|
||||
// Whenever you cast an instant or sorcery spell, put a +1/+1 counter on Pyre Hound.
|
||||
this.addAbility(new SpellCastControllerTriggeredAbility(new AddCountersSourceEffect(CounterType.P1P1.createInstance()),
|
||||
new FilterInstantOrSorcerySpell("an instant or sorcery spell"), true));
|
||||
new FilterInstantOrSorcerySpell("an instant or sorcery spell"), false));
|
||||
}
|
||||
|
||||
public PyreHound(final PyreHound card) {
|
||||
|
|
|
@ -36,6 +36,8 @@ public class SiftThroughSandsTest extends CardTestPlayerBase {
|
|||
*
|
||||
*/
|
||||
|
||||
// TODO: This test fails sometimes if building the complete test project
|
||||
// The cause is still unknown
|
||||
// test that The Unspeakable is put onto the battlefield
|
||||
@Test
|
||||
public void testTheUnspeakable() {
|
||||
|
|
|
@ -41,8 +41,8 @@ import mage.players.Player;
|
|||
*/
|
||||
public class LoseLifeDefendingPlayerEffect extends OneShotEffect {
|
||||
|
||||
protected DynamicValue amount;
|
||||
protected boolean attackerIsSource;
|
||||
private DynamicValue amount;
|
||||
private boolean attackerIsSource;
|
||||
|
||||
/**
|
||||
*
|
||||
|
@ -57,6 +57,7 @@ public class LoseLifeDefendingPlayerEffect extends OneShotEffect {
|
|||
public LoseLifeDefendingPlayerEffect(DynamicValue amount, boolean attackerIsSource) {
|
||||
super(Outcome.Damage);
|
||||
this.amount = amount;
|
||||
this.attackerIsSource = attackerIsSource;
|
||||
}
|
||||
|
||||
public LoseLifeDefendingPlayerEffect(final LoseLifeDefendingPlayerEffect effect) {
|
||||
|
|
Loading…
Reference in a new issue