diff --git a/.gitattributes b/.gitattributes index 4f74fad2a4c..9c6da4a8af6 100644 --- a/.gitattributes +++ b/.gitattributes @@ -13771,9 +13771,9 @@ src/main/java/forge/Constant.java svneol=native#text/plain src/main/java/forge/CounterType.java svneol=native#text/plain src/main/java/forge/FThreads.java -text src/main/java/forge/GameEntity.java -text -src/main/java/forge/GameEventType.java -text src/main/java/forge/GameLog.java -text src/main/java/forge/GameLogEntry.java -text +src/main/java/forge/GameLogEntryType.java -text src/main/java/forge/GameLogFormatter.java -text src/main/java/forge/ImageCache.java svneol=native#text/plain src/main/java/forge/ImageLoader.java -text diff --git a/src/main/java/forge/Card.java b/src/main/java/forge/Card.java index a40c170e21a..2d2878a716b 100644 --- a/src/main/java/forge/Card.java +++ b/src/main/java/forge/Card.java @@ -3164,7 +3164,7 @@ public class Card extends GameEntity implements Comparable { */ public final void equipCard(final Card c) { if (c.hasKeyword("CARDNAME can't be equipped.")) { - getGame().getGameLog().add(GameEventType.STACK_RESOLVE, "Trying to equip " + c.getName() + " but it can't be equipped."); + getGame().getGameLog().add(GameLogEntryType.STACK_RESOLVE, "Trying to equip " + c.getName() + " but it can't be equipped."); return; } if (this.hasStartOfKeyword("CantEquip")) { @@ -3173,7 +3173,7 @@ public class Card extends GameEntity implements Comparable { final String[] k = parse.split(" ", 2); final String[] restrictions = k[1].split(","); if (c.isValid(restrictions, this.getController(), this)) { - getGame().getGameLog().add(GameEventType.STACK_RESOLVE, "Trying to equip " + c.getName() + " but it can't be equipped."); + getGame().getGameLog().add(GameLogEntryType.STACK_RESOLVE, "Trying to equip " + c.getName() + " but it can't be equipped."); return; } } @@ -3365,7 +3365,7 @@ public class Card extends GameEntity implements Comparable { */ public final void enchantEntity(final GameEntity entity) { if (entity.hasKeyword("CARDNAME can't be enchanted.")) { - getGame().getGameLog().add(GameEventType.STACK_RESOLVE, "Trying to enchant " + entity.getName() + getGame().getGameLog().add(GameLogEntryType.STACK_RESOLVE, "Trying to enchant " + entity.getName() + " but it can't be enchanted."); return; } @@ -7409,7 +7409,7 @@ public class Card extends GameEntity implements Comparable { game.fireEvent(new GameEventCardDamaged()); } - getGame().getGameLog().add(GameEventType.DAMAGE, String.format("Dealing %d damage to %s. %s", damageToAdd, this.getName(), additionalLog)); + getGame().getGameLog().add(GameLogEntryType.DAMAGE, String.format("Dealing %d damage to %s. %s", damageToAdd, this.getName(), additionalLog)); return true; } diff --git a/src/main/java/forge/GameLog.java b/src/main/java/forge/GameLog.java index d6abf204c96..dbeefbeee58 100644 --- a/src/main/java/forge/GameLog.java +++ b/src/main/java/forge/GameLog.java @@ -62,7 +62,7 @@ public class GameLog extends MyObservable { * @param message the message * @param type the level */ - public void add(final GameEventType type, final String message) { + public void add(final GameLogEntryType type, final String message) { log.add(new GameLogEntry(type, message)); this.updateObservers(); } @@ -72,7 +72,7 @@ public class GameLog extends MyObservable { this.updateObservers(); } - public String getLogText(final GameEventType logLevel) { + public String getLogText(final GameLogEntryType logLevel) { List filteredAndReversed = getLogEntries(logLevel); return StringUtils.join(filteredAndReversed, "\r\n"); } @@ -83,7 +83,7 @@ public class GameLog extends MyObservable { * @param logLevel the log level * @return the log text */ - public List getLogEntries(final GameEventType logLevel) { // null to fetch all + public List getLogEntries(final GameLogEntryType logLevel) { // null to fetch all final List result = new ArrayList(); for (int i = log.size() - 1; i >= 0; i--) { @@ -94,7 +94,7 @@ public class GameLog extends MyObservable { return result; } - public List getLogEntriesExact(final GameEventType logLevel) { // null to fetch all + public List getLogEntriesExact(final GameLogEntryType logLevel) { // null to fetch all final List result = new ArrayList(); for (int i = log.size() - 1; i >= 0; i--) { diff --git a/src/main/java/forge/GameLogEntry.java b/src/main/java/forge/GameLogEntry.java index 51479d33b25..372f2fbaab8 100644 --- a/src/main/java/forge/GameLogEntry.java +++ b/src/main/java/forge/GameLogEntry.java @@ -2,10 +2,10 @@ package forge; public class GameLogEntry { public final String message; - public final GameEventType type; + public final GameLogEntryType type; // might add here date and some other fields - GameLogEntry(final GameEventType type0, final String messageIn) { + GameLogEntry(final GameLogEntryType type0, final String messageIn) { type = type0; message = messageIn; } diff --git a/src/main/java/forge/GameEventType.java b/src/main/java/forge/GameLogEntryType.java similarity index 84% rename from src/main/java/forge/GameEventType.java rename to src/main/java/forge/GameLogEntryType.java index 622e2cd2244..1882eff7ffd 100644 --- a/src/main/java/forge/GameEventType.java +++ b/src/main/java/forge/GameLogEntryType.java @@ -1,6 +1,6 @@ package forge; -public enum GameEventType { +public enum GameLogEntryType { GAME_OUTCOME("Game outcome"), MATCH_RESULTS("Match result"), TURN("Turn"), @@ -18,7 +18,7 @@ public enum GameEventType { PHASE("Phase"); private final String caption; - private GameEventType(String name) { + private GameLogEntryType(String name) { this.caption = name; } diff --git a/src/main/java/forge/GameLogFormatter.java b/src/main/java/forge/GameLogFormatter.java index 5e371cd24ea..12e37486444 100644 --- a/src/main/java/forge/GameLogFormatter.java +++ b/src/main/java/forge/GameLogFormatter.java @@ -36,7 +36,7 @@ public class GameLogFormatter extends IGameEventVisitor.Base { String whoHas = p.getKey().equals(human) ? "You have" : p.getKey().getName() + " has"; String outcome = String.format("%s %s", whoHas, p.getValue().getOutcome().toString()); outcomes.add(outcome); - log.add(GameEventType.GAME_OUTCOME, outcome); + log.add(GameLogEntryType.GAME_OUTCOME, outcome); } return generateSummary(ev.history); @@ -63,7 +63,7 @@ public class GameLogFormatter extends IGameEventVisitor.Base { for(int i = 0; i < wins.length; i++) { sb.append(players[i].getName()).append(": ").append(wins[i]).append(" "); } - return new GameLogEntry(GameEventType.MATCH_RESULTS, sb.toString()); + return new GameLogEntry(GameLogEntryType.MATCH_RESULTS, sb.toString()); } @Override @@ -78,13 +78,13 @@ public class GameLogFormatter extends IGameEventVisitor.Base { else message = String.format("%s is controlled by %s", p.getName(), newController.getName()); - return new GameLogEntry(GameEventType.PLAYER_CONROL, message); + return new GameLogEntry(GameLogEntryType.PLAYER_CONROL, message); } @Override public GameLogEntry visit(GameEventTurnPhase ev) { Player p = ev.playerTurn; - return new GameLogEntry(GameEventType.PHASE, ev.phaseDesc + Lang.getPossesive(p.getName()) + " " + ev.phase.nameForUi); + return new GameLogEntry(GameLogEntryType.PHASE, ev.phaseDesc + Lang.getPossesive(p.getName()) + " " + ev.phase.nameForUi); } @@ -108,7 +108,7 @@ public class GameLogFormatter extends IGameEventVisitor.Base { sb.append(" to attack ").append(defender.toString()).append("."); } - return new GameLogEntry(GameEventType.COMBAT, sb.toString()); + return new GameLogEntry(GameLogEntryType.COMBAT, sb.toString()); } @@ -145,7 +145,7 @@ public class GameLogFormatter extends IGameEventVisitor.Base { } } - return new GameLogEntry(GameEventType.COMBAT, sb.toString()); + return new GameLogEntry(GameLogEntryType.COMBAT, sb.toString()); } diff --git a/src/main/java/forge/card/cardfactory/CardFactoryUtil.java b/src/main/java/forge/card/cardfactory/CardFactoryUtil.java index 7c798bc1b04..e322a6fbd17 100644 --- a/src/main/java/forge/card/cardfactory/CardFactoryUtil.java +++ b/src/main/java/forge/card/cardfactory/CardFactoryUtil.java @@ -35,7 +35,7 @@ import forge.Command; import forge.Constant; import forge.CounterType; import forge.GameEntity; -import forge.GameEventType; +import forge.GameLogEntryType; import forge.Singletons; import forge.card.ColorSet; import forge.card.MagicColor; @@ -207,7 +207,7 @@ public class CardFactoryUtil { public void resolve() { if (sourceCard.turnFaceUp()) { String sb = this.getActivatingPlayer() + " has unmorphed " + sourceCard.getName(); - sourceCard.getGame().getGameLog().add(GameEventType.STACK_RESOLVE, sb); + sourceCard.getGame().getGameLog().add(GameLogEntryType.STACK_RESOLVE, sb); } } @@ -430,7 +430,7 @@ public class CardFactoryUtil { c.addCounter(CounterType.TIME, counters, true); String sb = String.format("%s has suspended %s with %d time counters on it.", this.getActivatingPlayer(), c.getName(), counters); - game.getGameLog().add(GameEventType.STACK_RESOLVE, sb); + game.getGameLog().add(GameLogEntryType.STACK_RESOLVE, sb); } }; final StringBuilder sbDesc = new StringBuilder(); diff --git a/src/main/java/forge/card/replacement/ReplacementHandler.java b/src/main/java/forge/card/replacement/ReplacementHandler.java index 27623233b85..34f2a205ff1 100644 --- a/src/main/java/forge/card/replacement/ReplacementHandler.java +++ b/src/main/java/forge/card/replacement/ReplacementHandler.java @@ -26,7 +26,7 @@ import java.util.Map; import org.apache.commons.lang3.StringUtils; import forge.Card; -import forge.GameEventType; +import forge.GameLogEntryType; import forge.card.ability.AbilityFactory; import forge.card.ability.AbilityUtils; import forge.card.spellability.SpellAbility; @@ -154,7 +154,7 @@ public class ReplacementHandler { chosenRE.setHasRun(false); String message = chosenRE.toString(); if ( !StringUtils.isEmpty(message)) - game.getGameLog().add(GameEventType.EFFECT_REPLACED, chosenRE.toString()); + game.getGameLog().add(GameLogEntryType.EFFECT_REPLACED, chosenRE.toString()); return res; } else { if (possibleReplacers.size() == 0) { diff --git a/src/main/java/forge/game/GameAction.java b/src/main/java/forge/game/GameAction.java index 42a4a087e32..a9b603fdb48 100644 --- a/src/main/java/forge/game/GameAction.java +++ b/src/main/java/forge/game/GameAction.java @@ -39,7 +39,7 @@ import forge.Command; import forge.CounterType; import forge.FThreads; import forge.GameEntity; -import forge.GameEventType; +import forge.GameLogEntryType; import forge.card.CardType; import forge.card.TriggerReplacementBase; import forge.card.ability.AbilityFactory; @@ -1570,7 +1570,7 @@ public class GameAction { p.onMulliganned(); allKept = false; } else { - game.getGameLog().add(GameEventType.MULLIGAN, p.getName() + " has kept a hand of " + p.getZone(ZoneType.Hand).size() + " cards"); + game.getGameLog().add(GameLogEntryType.MULLIGAN, p.getName() + " has kept a hand of " + p.getZone(ZoneType.Hand).size() + " cards"); hasKept[i] = true; } } diff --git a/src/main/java/forge/game/GameNew.java b/src/main/java/forge/game/GameNew.java index 8618913897c..1cdf27453b1 100644 --- a/src/main/java/forge/game/GameNew.java +++ b/src/main/java/forge/game/GameNew.java @@ -18,7 +18,7 @@ import com.google.common.collect.Lists; import forge.Card; import forge.CardLists; import forge.CardPredicates; -import forge.GameEventType; +import forge.GameLogEntryType; import forge.card.trigger.Trigger; import forge.card.trigger.TriggerHandler; import forge.deck.CardPool; @@ -312,7 +312,7 @@ public class GameNew { Predicate goodForAnte = Predicates.not(CardPredicates.Presets.BASIC_LANDS); Card ante = Aggregates.random(Iterables.filter(lib, goodForAnte)); if (ante == null) { - game.getGameLog().add(GameEventType.ANTE, "Only basic lands found. Will ante one of them"); + game.getGameLog().add(GameLogEntryType.ANTE, "Only basic lands found. Will ante one of them"); ante = Aggregates.random(lib); } anteed.add(Pair.of(p, ante)); @@ -324,7 +324,7 @@ public class GameNew { for(Pair kv : cards) { Player p = kv.getKey(); p.getGame().getAction().moveTo(ZoneType.Ante, kv.getValue()); - p.getGame().getGameLog().add(GameEventType.ANTE, p + " anted " + kv.getValue()); + p.getGame().getGameLog().add(GameLogEntryType.ANTE, p + " anted " + kv.getValue()); } } diff --git a/src/main/java/forge/game/phase/PhaseHandler.java b/src/main/java/forge/game/phase/PhaseHandler.java index 439de510ca1..09c426f2f85 100644 --- a/src/main/java/forge/game/phase/PhaseHandler.java +++ b/src/main/java/forge/game/phase/PhaseHandler.java @@ -26,7 +26,7 @@ import org.apache.commons.lang.time.StopWatch; import forge.Card; import forge.CardLists; import forge.FThreads; -import forge.GameEventType; +import forge.GameLogEntryType; import forge.Singletons; import forge.CardPredicates.Presets; import forge.card.trigger.TriggerType; @@ -197,7 +197,7 @@ public class PhaseHandler extends MyObservable implements java.io.Serializable { if (this.phase == PhaseType.UNTAP) { this.turn++; - game.getGameLog().add(GameEventType.TURN, "Turn " + this.turn + " (" + this.getPlayerTurn() + ")"); + game.getGameLog().add(GameLogEntryType.TURN, "Turn " + this.turn + " (" + this.getPlayerTurn() + ")"); } game.fireEvent(new GameEventTurnPhase(this.getPlayerTurn(), this.getPhase(), phaseType)); diff --git a/src/main/java/forge/game/player/HumanPlay.java b/src/main/java/forge/game/player/HumanPlay.java index c20c3fbd296..da7374f205f 100644 --- a/src/main/java/forge/game/player/HumanPlay.java +++ b/src/main/java/forge/game/player/HumanPlay.java @@ -10,7 +10,7 @@ import forge.CardLists; import forge.CardPredicates.Presets; import forge.CounterType; import forge.FThreads; -import forge.GameEventType; +import forge.GameLogEntryType; import forge.Singletons; import forge.card.ability.AbilityUtils; import forge.card.ability.ApiType; @@ -349,7 +349,7 @@ public class HumanPlay { if (false == source.canReceiveCounters(counterType)) { String message = String.format("Won't be able to pay upkeep for %s but it can't have %s counters put on it.", source, counterType.getName()); - p.getGame().getGameLog().add(GameEventType.STACK_RESOLVE, message); + p.getGame().getGameLog().add(GameLogEntryType.STACK_RESOLVE, message); return false; } diff --git a/src/main/java/forge/game/player/Player.java b/src/main/java/forge/game/player/Player.java index aaf8fbebcd0..4204793c0fc 100644 --- a/src/main/java/forge/game/player/Player.java +++ b/src/main/java/forge/game/player/Player.java @@ -39,7 +39,7 @@ import forge.Constant.Preferences; import forge.CounterType; import forge.FThreads; import forge.GameEntity; -import forge.GameEventType; +import forge.GameLogEntryType; import forge.Singletons; import forge.card.MagicColor; import forge.card.ability.AbilityFactory; @@ -673,7 +673,7 @@ public class Player extends GameEntity implements Comparable { runParams.put("IsCombatDamage", isCombat); game.getTriggerHandler().runTrigger(TriggerType.DamageDone, runParams, false); - game.getGameLog().add(GameEventType.DAMAGE, String.format("Dealing %d damage to %s. %s", damageToDo, this.getName(), additionalLog)); + game.getGameLog().add(GameLogEntryType.DAMAGE, String.format("Dealing %d damage to %s. %s", damageToDo, this.getName(), additionalLog)); return true; } @@ -1021,7 +1021,7 @@ public class Player extends GameEntity implements Comparable { this.poisonCounters += num; game.fireEvent(new GameEventPoisonCounter(this, source, num)); - game.getGameLog().add(GameEventType.DAMAGE_POISON, this + " receives a poison counter from " + source); + game.getGameLog().add(GameLogEntryType.DAMAGE_POISON, this + " receives a poison counter from " + source); this.updateObservers(); } @@ -1768,7 +1768,7 @@ public class Player extends GameEntity implements Comparable { game.getAction().checkStateEffects(); // add to log - game.getGameLog().add(GameEventType.LAND, this + " played " + land); + game.getGameLog().add(GameLogEntryType.LAND, this + " played " + land); // play a sound game.fireEvent(new GameEventLandPlayed(this, land)); @@ -3128,7 +3128,7 @@ public class Player extends GameEntity implements Comparable { public void onMulliganned() { game.fireEvent(new GameEventMulligan(this)); // quest listener may interfere here final int newHand = getCardsIn(ZoneType.Hand).size(); - game.getGameLog().add(GameEventType.MULLIGAN, this + " has mulliganed down to " + newHand + " cards."); + game.getGameLog().add(GameLogEntryType.MULLIGAN, this + " has mulliganed down to " + newHand + " cards."); stats.notifyHasMulliganed(); stats.notifyOpeningHandSize(newHand); } diff --git a/src/main/java/forge/game/zone/MagicStack.java b/src/main/java/forge/game/zone/MagicStack.java index 48b670f961c..8ead96df0aa 100644 --- a/src/main/java/forge/game/zone/MagicStack.java +++ b/src/main/java/forge/game/zone/MagicStack.java @@ -34,7 +34,7 @@ import forge.CardLists; import forge.FThreads; import forge.Singletons; import forge.CardPredicates.Presets; -import forge.GameEventType; +import forge.GameLogEntryType; import forge.card.ability.AbilityUtils; import forge.card.cardfactory.CardFactory; import forge.card.cardfactory.CardFactoryUtil; @@ -312,7 +312,7 @@ public class MagicStack extends MyObservable implements Iterable { // TODO - some option to make this configurable is probably desirable // By default, grab everything log level 3 or less. - final List data = model.getLogEntries(GameEventType.DAMAGE); + final List data = model.getLogEntries(GameLogEntryType.DAMAGE); final int size = data.size(); pnl.removeAll();