External/mage
1
0
Fork 1
mirror of https://github.com/magefree/mage.git synced 2026-01-26 21:29:17 -08:00
Code Issues Projects Releases Packages Wiki Activity Actions

* Chronicler of Heroes - Fixed a bug that wrongly was not only checked at resolving time, if a creature with a +1/+1 counter did exist.

Browse source
This commit is contained in:
LevelX2 2013-11-03 20:56:57 +01:00
parent 261f558c33
commit 7db40252d1
1 changed files with 41 additions and 12 deletions
Download patch file Download diff file Expand all files Collapse all files

53
Mage.Sets/src/mage/sets/theros/ChroniclerOfHeroes.java
View file

@ -29,18 +29,21 @@ package mage.sets.theros;
import java.util.UUID;
import mage.MageInt;
import mage.abilities.Ability;
import mage.abilities.common.EntersBattlefieldTriggeredAbility;
import mage.abilities.condition.common.ControlsPermanentCondition;
import mage.abilities.decorator.ConditionalOneShotEffect;
import mage.abilities.effects.common.DrawCardControllerEffect;
import mage.abilities.effects.OneShotEffect;
import mage.cards.CardImpl;
import mage.constants.CardType;
import mage.constants.Outcome;
import mage.constants.Rarity;
import mage.constants.TargetController;
import mage.counters.CounterType;
import mage.filter.common.FilterCreaturePermanent;
import mage.filter.predicate.permanent.ControllerPredicate;
import mage.filter.predicate.permanent.CounterPredicate;
import mage.game.Game;
import mage.players.Player;
/**
*
@ -48,12 +51,6 @@ import mage.filter.predicate.permanent.CounterPredicate;
*/
public class ChroniclerOfHeroes extends CardImpl<ChroniclerOfHeroes> {
private static final FilterCreaturePermanent filter = new FilterCreaturePermanent("a creature with a +1/+1 counter on it");
static {
filter.add(new ControllerPredicate(TargetController.YOU));
filter.add(new CounterPredicate(CounterType.P1P1));
}
public ChroniclerOfHeroes(UUID ownerId) {
super(ownerId, 190, "Chronicler of Heroes", Rarity.UNCOMMON, new CardType[]{CardType.CREATURE}, "{1}{G}{W}");
this.expansionSetCode = "THS";
@ -66,10 +63,7 @@ public class ChroniclerOfHeroes extends CardImpl<ChroniclerOfHeroes> {
this.toughness = new MageInt(3);
// When Chronicler of Heroes enters the battlefield, draw a card if you control a creature with a +1/+1 counter on it.
this.addAbility(new EntersBattlefieldTriggeredAbility(new ConditionalOneShotEffect(
new DrawCardControllerEffect(1),
new ControlsPermanentCondition(filter, ControlsPermanentCondition.CountType.MORE_THAN, 0),
"draw a card if you control a creature with a +1/+1 counter on it")));
this.addAbility(new EntersBattlefieldTriggeredAbility(new ChroniclerOfHeroesEffect()));
}
public ChroniclerOfHeroes(final ChroniclerOfHeroes card) {
@ -81,3 +75,38 @@ public class ChroniclerOfHeroes extends CardImpl<ChroniclerOfHeroes> {
return new ChroniclerOfHeroes(this);
}
}
class ChroniclerOfHeroesEffect extends OneShotEffect<ChroniclerOfHeroesEffect> {
private static final FilterCreaturePermanent filter = new FilterCreaturePermanent("a creature with a +1/+1 counter on it");
static {
filter.add(new ControllerPredicate(TargetController.YOU));
filter.add(new CounterPredicate(CounterType.P1P1));
}
public ChroniclerOfHeroesEffect() {
super(Outcome.DrawCard);
this.staticText = "draw a card if you control a creature with a +1/+1 counter on it";
}
public ChroniclerOfHeroesEffect(final ChroniclerOfHeroesEffect effect) {
super(effect);
}
@Override
public ChroniclerOfHeroesEffect copy() {
return new ChroniclerOfHeroesEffect(this);
}
@Override
public boolean apply(Game game, Ability source) {
Player controller = game.getPlayer(source.getControllerId());
if (controller != null) {
if (new ControlsPermanentCondition(filter, ControlsPermanentCondition.CountType.MORE_THAN, 0).apply(game, source)) {
controller.drawCards(1, game);
}
return true;
}
return false;
}
}