refactored mill effect names

Mage/src/main/java/mage/abilities/effects/common/MillCardsTargetEffect.java

@@ -0,0 +1,59 @@
+package mage.abilities.effects.common;
+
+import mage.abilities.Ability;
+import mage.abilities.dynamicvalue.DynamicValue;
+import mage.abilities.dynamicvalue.common.StaticValue;
+import mage.abilities.effects.OneShotEffect;
+import mage.constants.Outcome;
+import mage.game.Game;
+import mage.players.Player;
+import mage.util.CardUtil;
+
+/**
+ * @author LevelX2
+ */
+public class MillCardsTargetEffect extends OneShotEffect {
+
+    private DynamicValue numberCards;
+
+    public MillCardsTargetEffect(int numberCards) {
+        this(StaticValue.get(numberCards));
+    }
+
+    public MillCardsTargetEffect(DynamicValue numberCards) {
+        super(Outcome.Discard);
+        this.numberCards = numberCards;
+        this.staticText = setText();
+    }
+
+    public MillCardsTargetEffect(final MillCardsTargetEffect effect) {
+        super(effect);
+        this.numberCards = effect.numberCards;
+    }
+
+    @Override
+    public MillCardsTargetEffect copy() {
+        return new MillCardsTargetEffect(this);
+    }
+
+    @Override
+    public boolean apply(Game game, Ability source) {
+        Player player = game.getPlayer(targetPointer.getFirst(game, source));
+        if (player != null) {
+            player.millCards(numberCards.calculate(game, source, this), source, game);
+            return true;
+        }
+        return false;
+    }
+
+    private String setText() {
+        StringBuilder sb = new StringBuilder("target player mills ");
+        if (numberCards.toString().equals("1")) {
+            sb.append("a card");
+        } else {
+            sb.append(CardUtil.numberToText(numberCards.toString()));
+            sb.append(" cards");
+        }
+        return sb.toString();
+    }
+}