forked from External/mage
text improvements
This commit is contained in:
parent
52530b173d
commit
fc3551fc0a
6 changed files with 38 additions and 31 deletions
|
|
@ -55,14 +55,12 @@ public class AddCountersPlayersEffect extends OneShotEffect {
|
|||
case ANY:
|
||||
return game.getState().getPlayersInRange(source.getControllerId(), game);
|
||||
case YOU:
|
||||
return Arrays.asList(source.getControllerId());
|
||||
return Collections.singletonList(source.getControllerId());
|
||||
case CONTROLLER_ATTACHED_TO:
|
||||
List<UUID> list = new ArrayList<>();
|
||||
Optional.ofNullable(source.getSourcePermanentOrLKI(game))
|
||||
.filter(Objects::nonNull)
|
||||
.map(Permanent::getAttachedTo)
|
||||
.map(game::getControllerId)
|
||||
.filter(Objects::nonNull)
|
||||
.ifPresent(list::add);
|
||||
return list;
|
||||
default:
|
||||
|
|
@ -102,23 +100,26 @@ public class AddCountersPlayersEffect extends OneShotEffect {
|
|||
StringBuilder sb = new StringBuilder();
|
||||
switch (targetController) {
|
||||
case OPPONENT:
|
||||
sb.append("each opponent gets");
|
||||
sb.append("each opponent gets ");
|
||||
break;
|
||||
case ANY:
|
||||
case EACH_PLAYER:
|
||||
sb.append("each player gets");
|
||||
sb.append("each player gets ");
|
||||
break;
|
||||
case YOU:
|
||||
sb.append("you get");
|
||||
sb.append("you get ");
|
||||
break;
|
||||
case CONTROLLER_ATTACHED_TO:
|
||||
sb.append("its controller gets");
|
||||
sb.append("its controller gets ");
|
||||
break;
|
||||
default:
|
||||
throw new UnsupportedOperationException(targetController + " not supported");
|
||||
}
|
||||
sb.append(' ');
|
||||
sb.append(counter.getDescription());
|
||||
if (amount.toString().equals("X")) {
|
||||
sb.append("X ").append(counter.getName()).append(" counters");
|
||||
} else {
|
||||
sb.append(counter.getDescription());
|
||||
}
|
||||
return sb.toString();
|
||||
}
|
||||
}
|
||||
|
|
|
|||
|
|
@ -23,7 +23,7 @@ import java.util.UUID;
|
|||
public class AddCountersTargetEffect extends OneShotEffect {
|
||||
|
||||
private Counter counter;
|
||||
private DynamicValue amount;
|
||||
private final DynamicValue amount;
|
||||
|
||||
public AddCountersTargetEffect(Counter counter) {
|
||||
this(counter, counter.getName().equals(CounterType.M1M1.getName()) ? Outcome.UnboostCreature : Outcome.Benefit);
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue