From 5186f690d74904ff185850d64d2dd1cc84a68a64 Mon Sep 17 00:00:00 2001
From: LevelX2 <ludwig.hirth@gmail.com>
Date: Thu, 21 Nov 2013 09:58:22 +0100
Subject: [PATCH] Added IsPhaseCondition, some minor framework formatting.

---
 .../ActivateIfConditionActivatedAbility.java  |  8 ++-
 .../abilities/condition/InvertCondition.java  |  7 ++-
 .../condition/common/IsPhaseCondition.java    | 58 +++++++++++++++++++
 Mage/src/mage/constants/TurnPhase.java        |  2 +-
 Mage/src/mage/target/TargetCard.java          |  1 +
 Mage/src/mage/target/TargetImpl.java          |  1 +
 Mage/src/mage/target/TargetObject.java        |  1 +
 .../target/common/TargetCardInGraveyard.java  | 10 +++-
 8 files changed, 82 insertions(+), 6 deletions(-)
 create mode 100644 Mage/src/mage/abilities/condition/common/IsPhaseCondition.java

diff --git a/Mage/src/mage/abilities/common/ActivateIfConditionActivatedAbility.java b/Mage/src/mage/abilities/common/ActivateIfConditionActivatedAbility.java
index dd365ad1fde..86f8c599b4f 100644
--- a/Mage/src/mage/abilities/common/ActivateIfConditionActivatedAbility.java
+++ b/Mage/src/mage/abilities/common/ActivateIfConditionActivatedAbility.java
@@ -31,6 +31,7 @@ package mage.abilities.common;
 import java.util.UUID;
 import mage.abilities.ActivatedAbilityImpl;
 import mage.abilities.condition.Condition;
+import mage.abilities.condition.InvertCondition;
 import mage.abilities.costs.Cost;
 import mage.abilities.effects.Effect;
 import mage.constants.Zone;
@@ -80,7 +81,12 @@ public class ActivateIfConditionActivatedAbility extends ActivatedAbilityImpl<Ac
 
     @Override
     public String getRule() {
-        StringBuilder sb = new StringBuilder(super.getRule()).append(" Activate this ability only ");
+        StringBuilder sb = new StringBuilder(super.getRule());
+        if (condition instanceof InvertCondition) {
+            sb.append(" You can't activate this ability ");
+        } else {
+            sb.append(" Activate this ability only ");
+        }
         if (condition.toString() != null) {
             if (!condition.toString().startsWith("during")) {
                 sb.append("if ");
diff --git a/Mage/src/mage/abilities/condition/InvertCondition.java b/Mage/src/mage/abilities/condition/InvertCondition.java
index fe742566e95..e83cf8bbb9c 100644
--- a/Mage/src/mage/abilities/condition/InvertCondition.java
+++ b/Mage/src/mage/abilities/condition/InvertCondition.java
@@ -39,7 +39,7 @@ import mage.game.Game;
  */
 public class InvertCondition implements Condition {
 
-    private Condition condition;
+    private final Condition condition;
 
     public InvertCondition ( Condition condition ) {
         this.condition = condition;
@@ -53,4 +53,9 @@ public class InvertCondition implements Condition {
         return !condition.apply(game, source);
     }
 
+    @Override
+    public String toString() {
+        return condition.toString();
+    }
+    
 }
diff --git a/Mage/src/mage/abilities/condition/common/IsPhaseCondition.java b/Mage/src/mage/abilities/condition/common/IsPhaseCondition.java
new file mode 100644
index 00000000000..3739a855f2c
--- /dev/null
+++ b/Mage/src/mage/abilities/condition/common/IsPhaseCondition.java
@@ -0,0 +1,58 @@
+/*
+ *  Copyright 2010 BetaSteward_at_googlemail.com. All rights reserved.
+ *
+ *  Redistribution and use in source and binary forms, with or without modification, are
+ *  permitted provided that the following conditions are met:
+ *
+ *     1. Redistributions of source code must retain the above copyright notice, this list of
+ *        conditions and the following disclaimer.
+ *
+ *     2. Redistributions in binary form must reproduce the above copyright notice, this list
+ *        of conditions and the following disclaimer in the documentation and/or other materials
+ *        provided with the distribution.
+ *
+ *  THIS SOFTWARE IS PROVIDED BY BetaSteward_at_googlemail.com ``AS IS'' AND ANY EXPRESS OR IMPLIED
+ *  WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND
+ *  FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL BetaSteward_at_googlemail.com OR
+ *  CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+ *  CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
+ *  SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON
+ *  ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING
+ *  NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF
+ *  ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ *
+ *  The views and conclusions contained in the software and documentation are those of the
+ *  authors and should not be interpreted as representing official policies, either expressed
+ *  or implied, of BetaSteward_at_googlemail.com.
+ */
+
+package mage.abilities.condition.common;
+
+import mage.abilities.Ability;
+import mage.abilities.condition.Condition;
+import mage.constants.TurnPhase;
+import mage.game.Game;
+
+/**
+ *
+ * @author LevelX2
+ */
+public class IsPhaseCondition implements Condition {
+
+    protected TurnPhase turnPhase;
+
+    public  IsPhaseCondition(TurnPhase turnPhase) {
+        this.turnPhase = turnPhase;
+    }
+
+    @Override
+    public boolean apply(Game game, Ability source) {
+        return game.getTurn().getPhaseType().equals(turnPhase);
+    }
+
+    @Override
+    public String toString() {
+        return new StringBuilder("during ").append(turnPhase).toString();
+    }
+
+}
diff --git a/Mage/src/mage/constants/TurnPhase.java b/Mage/src/mage/constants/TurnPhase.java
index 4ec0953e0c6..e3d1a114536 100644
--- a/Mage/src/mage/constants/TurnPhase.java
+++ b/Mage/src/mage/constants/TurnPhase.java
@@ -11,7 +11,7 @@ public enum TurnPhase {
     POSTCOMBAT_MAIN ("Postcombat Main"),
     END ("End");
 
-    private String text;
+    private final String text;
 
     TurnPhase(String text) {
         this.text = text;
diff --git a/Mage/src/mage/target/TargetCard.java b/Mage/src/mage/target/TargetCard.java
index 558a3fe4a48..a3f0f13c7b9 100644
--- a/Mage/src/mage/target/TargetCard.java
+++ b/Mage/src/mage/target/TargetCard.java
@@ -42,6 +42,7 @@ import java.util.UUID;
 /**
  *
  * @author BetaSteward_at_googlemail.com
+ * @param <T>
  */
 public class TargetCard<T extends TargetCard<T>> extends TargetObject<TargetCard<T>> {
 
diff --git a/Mage/src/mage/target/TargetImpl.java b/Mage/src/mage/target/TargetImpl.java
index 1f5b9d5d120..7967c1bc9fd 100644
--- a/Mage/src/mage/target/TargetImpl.java
+++ b/Mage/src/mage/target/TargetImpl.java
@@ -42,6 +42,7 @@ import java.util.*;
 /**
  *
  * @author BetaSteward_at_googlemail.com
+ * @param <T>
  */
 public abstract class TargetImpl<T extends TargetImpl<T>> implements Target {
 
diff --git a/Mage/src/mage/target/TargetObject.java b/Mage/src/mage/target/TargetObject.java
index 12e664c001c..14d4c63c1d6 100644
--- a/Mage/src/mage/target/TargetObject.java
+++ b/Mage/src/mage/target/TargetObject.java
@@ -38,6 +38,7 @@ import java.util.UUID;
 /**
  *
  * @author BetaSteward_at_googlemail.com
+ * @param <T>
  */
 public abstract class TargetObject<T extends TargetObject<T>> extends TargetImpl<T> {
 
diff --git a/Mage/src/mage/target/common/TargetCardInGraveyard.java b/Mage/src/mage/target/common/TargetCardInGraveyard.java
index f7db3177d71..21e40fa013c 100644
--- a/Mage/src/mage/target/common/TargetCardInGraveyard.java
+++ b/Mage/src/mage/target/common/TargetCardInGraveyard.java
@@ -28,15 +28,14 @@
 
 package mage.target.common;
 
-import mage.constants.Zone;
+import java.util.UUID;
 import mage.abilities.Ability;
 import mage.cards.Card;
+import mage.constants.Zone;
 import mage.filter.FilterCard;
 import mage.game.Game;
 import mage.target.TargetCard;
 
-import java.util.UUID;
-
 /**
  *
  * @author BetaSteward_at_googlemail.com
@@ -47,6 +46,11 @@ public class TargetCardInGraveyard extends TargetCard<TargetCardInGraveyard> {
         this(1, 1, new FilterCard("card from a graveyard"));
     }
 
+    public TargetCardInGraveyard(boolean required) {
+        this();
+        this.setRequired(required);
+    }
+
     public TargetCardInGraveyard(FilterCard filter) {
         this(1, 1, filter);
     }