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

fixed text generation for TapTargetEffect to fit with other effects

Browse source
This commit is contained in:
Evan Kranzler 2021-07-05 22:09:16 -04:00
parent bed2e855b6
commit 5b93f25206
37 changed files with 44 additions and 46 deletions
Download patch file Download diff file Expand all files Collapse all files

16
Mage/src/main/java/mage/abilities/effects/common/TapTargetEffect.java
View file

@ -1,8 +1,5 @@
package mage.abilities.effects.common;
import java.util.UUID;
import mage.abilities.Ability;
import mage.abilities.Mode;
import mage.abilities.effects.OneShotEffect;
@ -12,18 +9,20 @@ import mage.game.permanent.Permanent;
import mage.target.Target;
import mage.util.CardUtil;
import java.util.UUID;
/**
* @author BetaSteward_at_googlemail.com
*/
public class TapTargetEffect extends OneShotEffect {
public TapTargetEffect() {
super(Outcome.Tap);
this((String) null);
}
public TapTargetEffect(String text) {
super(Outcome.Tap);
if(text != null) {
if (text != null) {
this.staticText = text;
}
}
@ -50,8 +49,8 @@ public class TapTargetEffect extends OneShotEffect {
@Override
public String getText(Mode mode) {
if (!staticText.isEmpty()) {
return "tap " + staticText;
if (staticText != null && !staticText.isEmpty()) {
return staticText;
}
if (mode.getTargets().isEmpty()) {
@ -65,11 +64,10 @@ public class TapTargetEffect extends OneShotEffect {
} else {
return "tap up to " + CardUtil.numberToText(target.getMaxNumberOfTargets()) + " target " + target.getTargetName() + 's';
}
} else if (target.getMaxNumberOfTargets() == 0){
} else if (target.getMaxNumberOfTargets() == 0) {
return "tap X target " + mode.getTargets().get(0).getTargetName();
} else {
return "tap target " + mode.getTargets().get(0).getTargetName();
}
}
}