more text fixes

This commit is contained in:
Evan Kranzler 2021-02-02 16:30:12 -05:00
parent 0d44eea16b
commit bb1a085962
14 changed files with 39 additions and 44 deletions

View file

@ -126,7 +126,7 @@ public class ChampionAbility extends StaticAbility {
@Override
public String getRule() {
StringBuilder sb = new StringBuilder("Champion ").append(objectDescription);
sb.append("<i>(When this enters the battlefield, sacrifice it unless you exile another ");
sb.append(" <i>(When this enters the battlefield, sacrifice it unless you exile another ");
sb.append(objectDescription);
sb.append(" you control. When this leaves the battlefield, that card returns to the battlefield.)</i>");
return sb.toString();

View file

@ -34,7 +34,7 @@ public class FadingAbility extends EntersBattlefieldAbility {
ruleText = "Fading " + fadeCounter
+ (shortRuleText ? ""
: " <i>(This permanent enters the battlefield with " + fadeCounter + " fade counters on it."
+ " At the beginning of your upkeep, remove a fade counter from this permanent. If you can't, sacrifice the permanent.</i>");
+ " At the beginning of your upkeep, remove a fade counter from this permanent. If you can't, sacrifice the permanent.)</i>");
}
public FadingAbility(final FadingAbility ability) {

View file

@ -1,15 +1,14 @@
package mage.abilities.keyword;
import mage.abilities.common.LimitedTimesPerTurnActivatedAbility;
import mage.abilities.ActivatedAbilityImpl;
import mage.abilities.condition.Condition;
import mage.abilities.condition.common.IsStepCondition;
import mage.abilities.costs.Cost;
import mage.abilities.costs.common.RevealSourceFromYourHandCost;
import mage.abilities.effects.Effect;
import mage.constants.PhaseStep;
import mage.constants.Zone;
import mage.game.Game;
import java.util.UUID;
/**
* 702.56. Forecast 702.56a A forecast ability is a special kind of activated
@ -25,10 +24,14 @@ import java.util.UUID;
*
* @author LevelX2
*/
public class ForecastAbility extends LimitedTimesPerTurnActivatedAbility {
public class ForecastAbility extends ActivatedAbilityImpl {
private static final Condition upkeepCondition = new IsStepCondition(PhaseStep.UPKEEP, true);
public ForecastAbility(Effect effect, Cost cost) {
super(Zone.HAND, effect, cost);
this.maxActivationsPerTurn = 1;
this.condition = upkeepCondition;
this.addCost(new RevealSourceFromYourHandCost());
}
@ -41,18 +44,8 @@ public class ForecastAbility extends LimitedTimesPerTurnActivatedAbility {
return new ForecastAbility(this);
}
@Override
public ActivationStatus canActivate(UUID playerId, Game game) {
// May be activated only during the upkeep step of the card's owner
// Because it can only be activated from a players hand it should be ok to check here with controllerId instead of card.getOwnerId().
if (!game.isActivePlayer(controllerId) || PhaseStep.UPKEEP != game.getStep().getType()) {
return ActivationStatus.getFalse();
}
return super.canActivate(playerId, game);
}
@Override
public String getRule() {
return "Forecast &mdash; " + super.getRule() + " <i>(Activate this ability only during your upkeep.)</i>";
return "Forecast &mdash; " + super.getRule() + " <i>(Activate this ability only during your upkeep and only once each turn)</i>";
}
}