github/mage
1
0
Fork 0
mirror of https://github.com/correl/mage.git synced 2024-12-01 03:00:09 +00:00
Code Issues Projects Releases Packages Wiki Activity

Revert "introduced StringUtil class with .isEmpty(String input) and .isNotEmp…"

Browse source
This commit is contained in:
LevelX2 2016-09-16 23:55:54 +02:00 committed by GitHub
parent 18e771ccd5
commit d7dd635bca
51 changed files with 96 additions and 294 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
Mage.Client/src/main/java/mage/client/cards/Card.java
View file

@ -87,7 +87,6 @@ import mage.client.util.gui.ArrowBuilder;
import mage.constants.CardType; import mage.constants.CardType;
import mage.constants.EnlargeMode; import mage.constants.EnlargeMode;
import mage.remote.Session; import mage.remote.Session;
import mage.util.StringUtil;
import mage.view.AbilityView; import mage.view.AbilityView;
import mage.view.CardView; import mage.view.CardView;
import mage.view.CounterView; import mage.view.CounterView;
@ -249,7 +248,7 @@ public class Card extends MagePermanent implements MouseMotionListener, MouseLis
for (String rule : getRules()) { for (String rule : getRules()) {
sb.append("\n").append(rule); sb.append("\n").append(rule);
} }
if (StringUtil.isNotEmpty(card.getExpansionSetCode())) { if (card.getExpansionSetCode() != null && card.getExpansionSetCode().length() > 0) {
sb.append("\n").append(card.getCardNumber()).append(" - "); sb.append("\n").append(card.getCardNumber()).append(" - ");
sb.append(Sets.getInstance().get(card.getExpansionSetCode()).getName()).append(" - "); sb.append(Sets.getInstance().get(card.getExpansionSetCode()).getName()).append(" - ");
sb.append(card.getRarity().toString()); sb.append(card.getRarity().toString());

2
Mage.Client/src/main/java/mage/client/cards/CardArea.java
View file

@ -163,7 +163,7 @@ public class CardArea extends JPanel implements MouseListener {
private void loadCardsMany(CardsView showCards, BigCard bigCard, UUID gameId) { private void loadCardsMany(CardsView showCards, BigCard bigCard, UUID gameId) {
int rowsOfCards = 20; int rowsOfCards = 20;
int columns = 1; int columns = 1;
if (showCards != null && !showCards.isEmpty()) { if (showCards != null && showCards.size() > 0) {
Rectangle rectangle = new Rectangle(cardDimension.width, cardDimension.height); Rectangle rectangle = new Rectangle(cardDimension.width, cardDimension.height);
int count = 0; int count = 0;
for (CardView card : showCards.values()) { for (CardView card : showCards.values()) {

3
Mage.Client/src/main/java/mage/client/cards/Permanent.java
View file

@ -54,7 +54,6 @@ import static mage.client.constants.Constants.POWBOX_TEXT_MAX_TOP;
import mage.client.util.Config; import mage.client.util.Config;
import mage.client.util.ImageHelper; import mage.client.util.ImageHelper;
import mage.constants.CardType; import mage.constants.CardType;
import mage.util.StringUtil;
import mage.view.CounterView; import mage.view.CounterView;
import mage.view.PermanentView; import mage.view.PermanentView;
import org.mage.plugins.card.images.ImageCache; import org.mage.plugins.card.images.ImageCache;
@ -125,7 +124,7 @@ public class Permanent extends Card {
for (String rule: getRules()) { for (String rule: getRules()) {
sb.append("\n").append(rule); sb.append("\n").append(rule);
} }
if (StringUtil.isNotEmpty(permanent.getOriginal().getExpansionSetCode())) { if (permanent.getOriginal().getExpansionSetCode().length() > 0) {
sb.append("\n").append(permanent.getCardNumber()).append(" - "); sb.append("\n").append(permanent.getCardNumber()).append(" - ");
sb.append("\n").append(Sets.getInstance().get(permanent.getOriginal().getExpansionSetCode()).getName()).append(" - "); sb.append("\n").append(Sets.getInstance().get(permanent.getOriginal().getExpansionSetCode()).getName()).append(" - ");
sb.append(permanent.getOriginal().getRarity().toString()); sb.append(permanent.getOriginal().getRarity().toString());

7
Mage.Client/src/main/java/mage/client/chat/ChatPanelBasic.java
View file

@ -42,7 +42,6 @@ import javax.swing.JTextField;
import mage.client.MageFrame; import mage.client.MageFrame;
import mage.client.SessionHandler; import mage.client.SessionHandler;
import mage.client.util.GUISizeHelper; import mage.client.util.GUISizeHelper;
import mage.util.StringUtil;
import mage.view.ChatMessage.MessageColor; import mage.view.ChatMessage.MessageColor;
import mage.view.ChatMessage.MessageType; import mage.view.ChatMessage.MessageType;
import org.mage.card.arcane.ManaSymbols; import org.mage.card.arcane.ManaSymbols;
@ -222,13 +221,13 @@ public class ChatPanelBasic extends javax.swing.JPanel {
textColor = MESSAGE_COLOR; textColor = MESSAGE_COLOR;
userSeparator = ": "; userSeparator = ": ";
} }
if (color == MessageColor.ORANGE) { if (color.equals(MessageColor.ORANGE)) {
textColor = "Orange"; textColor = "Orange";
} }
if (color == MessageColor.YELLOW) { if (color.equals(MessageColor.YELLOW)) {
textColor = "Yellow"; textColor = "Yellow";
} }
if (StringUtil.isNotEmpty(username)) { if (username != null && !username.isEmpty()) {
text.append(getColoredText(userColor, username + userSeparator)); text.append(getColoredText(userColor, username + userSeparator));
} }
text.append(getColoredText(textColor, ManaSymbols.replaceSymbolsWithHTML(message, ManaSymbols.Type.CHAT))); text.append(getColoredText(textColor, ManaSymbols.replaceSymbolsWithHTML(message, ManaSymbols.Type.CHAT)));

2
Mage.Client/src/main/java/mage/client/components/MageTextArea.java
View file

@ -51,7 +51,7 @@ public class MageTextArea extends JEditorPane {
text = text.replace("\r\n", "<div style='font-size:5pt'></div>"); text = text.replace("\r\n", "<div style='font-size:5pt'></div>");
final String basicText = ManaSymbols.replaceSymbolsWithHTML(text, ManaSymbols.Type.DIALOG); final String basicText = ManaSymbols.replaceSymbolsWithHTML(text, ManaSymbols.Type.DIALOG);
if (!text.isEmpty()) { if (text.length() > 0) {
buffer.append(basicText); buffer.append(basicText);
} }

2
Mage.Client/src/main/java/mage/client/components/ability/AbilityPicker.java
View file

@ -296,7 +296,7 @@ public class AbilityPicker extends JXPanel implements MouseWheelListener {
text = text.replace("\r\n", "<div style='font-size:5pt'></div>"); text = text.replace("\r\n", "<div style='font-size:5pt'></div>");
//text += "<br>"; //text += "<br>";
if (!text.isEmpty()) { if (text.length() > 0) {
buffer.append(ManaSymbols.replaceSymbolsWithHTML(text, ManaSymbols.Type.DIALOG)); buffer.append(ManaSymbols.replaceSymbolsWithHTML(text, ManaSymbols.Type.DIALOG));
} }

40
Mage.Client/src/main/java/mage/client/dialog/NewTableDialog.java
View file

@ -405,25 +405,33 @@ public class NewTableDialog extends MageDialog {
JOptionPane.showMessageDialog(MageFrame.getDesktop(), "Error creating table.", "Error", JOptionPane.ERROR_MESSAGE); JOptionPane.showMessageDialog(MageFrame.getDesktop(), "Error creating table.", "Error", JOptionPane.ERROR_MESSAGE);
return; return;
} }
if (SessionHandler.joinTable( try {
roomId, if (SessionHandler.joinTable(
table.getTableId(), roomId,
this.player1Panel.getPlayerName(), table.getTableId(),
"Human", 1, this.player1Panel.getPlayerName(),
DeckImporterUtil.importDeck(this.player1Panel.getDeckFile()), "Human", 1,
this.txtPassword.getText())) { DeckImporterUtil.importDeck(this.player1Panel.getDeckFile()),
for (TablePlayerPanel player : players) { this.txtPassword.getText())) {
if (!player.getPlayerType().equals("Human")) { for (TablePlayerPanel player : players) {
if (!player.joinTable(roomId, table.getTableId())) { if (!player.getPlayerType().equals("Human")) {
// error message must be send by the server if (!player.joinTable(roomId, table.getTableId())) {
SessionHandler.removeTable(roomId, table.getTableId()); // error message must be send by the server
table = null; SessionHandler.removeTable(roomId, table.getTableId());
return; table = null;
return;
}
} }
} }
this.hideDialog();
return;
} }
this.hideDialog(); } catch (FileNotFoundException ex) {
return; handleError(ex);
} catch (IOException ex) {
handleError(ex);
} catch (ClassNotFoundException ex) {
handleError(ex);
} }
// JOptionPane.showMessageDialog(MageFrame.getDesktop(), "Error joining table.", "Error", JOptionPane.ERROR_MESSAGE); // JOptionPane.showMessageDialog(MageFrame.getDesktop(), "Error joining table.", "Error", JOptionPane.ERROR_MESSAGE);
SessionHandler.removeTable(roomId, table.getTableId()); SessionHandler.removeTable(roomId, table.getTableId());

2
Mage.Client/src/main/java/mage/client/game/GamePanel.java
View file

@ -2173,7 +2173,7 @@ public final class GamePanel extends javax.swing.JPanel {
choices, choices,
this.chosenHandKey); this.chosenHandKey);
if (newChosenHandKey != null && !newChosenHandKey.isEmpty()) { if (newChosenHandKey != null && newChosenHandKey.length() > 0) {
this.chosenHandKey = newChosenHandKey; this.chosenHandKey = newChosenHandKey;
CardsView cards = handCards.get(chosenHandKey); CardsView cards = handCards.get(chosenHandKey);
handContainer.loadCards(cards, bigCard, gameId); handContainer.loadCards(cards, bigCard, gameId);

4
Mage.Client/src/main/java/mage/client/table/TablePlayerPanel.java
View file

@ -69,7 +69,7 @@ public class TablePlayerPanel extends javax.swing.JPanel {
cbPlayerType.setSelectedIndex(cbPlayerType.getItemCount() - 1); cbPlayerType.setSelectedIndex(cbPlayerType.getItemCount() - 1);
} }
else { else {
int index = Integer.parseInt(Config.defaultOtherPlayerIndex); Integer index = Integer.parseInt(Config.defaultOtherPlayerIndex);
cbPlayerType.setSelectedIndex(index); cbPlayerType.setSelectedIndex(index);
} }
} }
@ -78,7 +78,7 @@ public class TablePlayerPanel extends javax.swing.JPanel {
} }
} }
public boolean joinTable(UUID roomId, UUID tableId) { public boolean joinTable(UUID roomId, UUID tableId) throws FileNotFoundException, IOException, ClassNotFoundException {
if (!this.cbPlayerType.getSelectedItem().equals("Human")) { if (!this.cbPlayerType.getSelectedItem().equals("Human")) {
return SessionHandler.joinTable(roomId, tableId, this.newPlayerPanel.getPlayerName(), (String)this.cbPlayerType.getSelectedItem(), this.newPlayerPanel.getLevel(), DeckImporterUtil.importDeck(this.newPlayerPanel.getDeckFile()),""); return SessionHandler.joinTable(roomId, tableId, this.newPlayerPanel.getPlayerName(), (String)this.cbPlayerType.getSelectedItem(), this.newPlayerPanel.getLevel(), DeckImporterUtil.importDeck(this.newPlayerPanel.getDeckFile()),"");
} }

3
Mage.Client/src/main/java/mage/client/util/gui/GuiDisplayUtil.java
View file

@ -18,7 +18,6 @@ import mage.constants.CardType;
import mage.constants.MageObjectType; import mage.constants.MageObjectType;
import mage.constants.Rarity; import mage.constants.Rarity;
import mage.constants.Zone; import mage.constants.Zone;
import mage.util.StringUtil;
import mage.utils.CardUtil; import mage.utils.CardUtil;
import mage.view.CardView; import mage.view.CardView;
import mage.view.CounterView; import mage.view.CounterView;
@ -340,7 +339,7 @@ public class GuiDisplayUtil {
} }
String legal = rule.toString(); String legal = rule.toString();
if (StringUtil.isNotEmpty(legal)) { if (legal.length() > 0) {
legal = legal.replaceAll("\\{this\\}", card.getName().isEmpty() ? "this" : card.getName()); legal = legal.replaceAll("\\{this\\}", card.getName().isEmpty() ? "this" : card.getName());
legal = legal.replaceAll("\\{source\\}", card.getName().isEmpty() ? "this" : card.getName()); legal = legal.replaceAll("\\{source\\}", card.getName().isEmpty() ? "this" : card.getName());
buffer.append(ManaSymbols.replaceSymbolsWithHTML(legal, ManaSymbols.Type.TOOLTIP)); buffer.append(ManaSymbols.replaceSymbolsWithHTML(legal, ManaSymbols.Type.TOOLTIP));

3
Mage.Client/src/main/java/org/mage/card/arcane/CardPanel.java
View file

@ -32,7 +32,6 @@ import mage.client.plugins.impl.Plugins;
import mage.client.util.audio.AudioManager; import mage.client.util.audio.AudioManager;
import mage.constants.CardType; import mage.constants.CardType;
import mage.constants.EnlargeMode; import mage.constants.EnlargeMode;
import mage.util.StringUtil;
import mage.view.AbilityView; import mage.view.AbilityView;
import mage.view.CardView; import mage.view.CardView;
import mage.view.PermanentView; import mage.view.PermanentView;
@ -726,7 +725,7 @@ public abstract class CardPanel extends MagePermanent implements MouseListener,
for (String rule : card.getRules()) { for (String rule : card.getRules()) {
sb.append("\n").append(rule); sb.append("\n").append(rule);
} }
if (StringUtil.isNotEmpty(card.getExpansionSetCode())) { if (card.getExpansionSetCode() != null && card.getExpansionSetCode().length() > 0) {
sb.append("\n").append(card.getCardNumber()).append(" - "); sb.append("\n").append(card.getCardNumber()).append(" - ");
sb.append(card.getExpansionSetCode()).append(" - "); sb.append(card.getExpansionSetCode()).append(" - ");
sb.append(card.getRarity().toString()); sb.append(card.getRarity().toString());

12
Mage.Common/src/mage/view/ChatMessage.java
View file

@ -45,17 +45,7 @@ public class ChatMessage implements Serializable {
private MessageType messageType; private MessageType messageType;
public enum MessageColor { public enum MessageColor {
BLACK("Black"), RED("Red"), GREEN("Green"), BLUE("Blue"), ORANGE("Orange"), YELLOW("Yellow"); BLACK, RED, GREEN, BLUE, ORANGE, YELLOW;
private String color;
MessageColor(String t){
color = t;
}
public String toString(){
return color;
}
} }
public enum MessageType { public enum MessageType {

16
Mage.Server/src/main/java/mage/server/User.java
View file

@ -597,15 +597,15 @@ public class User {
builder.append(proto.getMatches()); builder.append(proto.getMatches());
List<String> quit = new ArrayList<>(); List<String> quit = new ArrayList<>();
if (proto.getMatchesIdleTimeout() > 0) { if (proto.getMatchesIdleTimeout() > 0) {
quit.add("I:" + proto.getMatchesIdleTimeout());//Integer.toString(proto.getMatchesIdleTimeout())); quit.add("I:" + Integer.toString(proto.getMatchesIdleTimeout()));
} }
if (proto.getMatchesTimerTimeout() > 0) { if (proto.getMatchesTimerTimeout() > 0) {
quit.add("T:" + proto.getMatchesTimerTimeout());//Integer.toString(proto.getMatchesTimerTimeout())); quit.add("T:" + Integer.toString(proto.getMatchesTimerTimeout()));
} }
if (proto.getMatchesQuit() > 0) { if (proto.getMatchesQuit() > 0) {
quit.add("Q:" + proto.getMatchesQuit());//Integer.toString(proto.getMatchesQuit())); quit.add("Q:" + Integer.toString(proto.getMatchesQuit()));
} }
if (!quit.isEmpty()) { if (quit.size() > 0) {
builder.append(" ("); builder.append(" (");
joinStrings(builder, quit, " "); joinStrings(builder, quit, " ");
builder.append(")"); builder.append(")");
@ -629,15 +629,15 @@ public class User {
builder.append(proto.getTourneys()); builder.append(proto.getTourneys());
List<String> quit = new ArrayList<>(); List<String> quit = new ArrayList<>();
if (proto.getTourneysQuitDuringDrafting() > 0) { if (proto.getTourneysQuitDuringDrafting() > 0) {
quit.add("D:" + proto.getTourneysQuitDuringDrafting());//Integer.toString(proto.getTourneysQuitDuringDrafting())); quit.add("D:" + Integer.toString(proto.getTourneysQuitDuringDrafting()));
} }
if (proto.getTourneysQuitDuringConstruction() > 0) { if (proto.getTourneysQuitDuringConstruction() > 0) {
quit.add("C:" + proto.getTourneysQuitDuringConstruction());//Integer.toString(proto.getTourneysQuitDuringConstruction())); quit.add("C:" + Integer.toString(proto.getTourneysQuitDuringConstruction()));
} }
if (proto.getTourneysQuitDuringRound() > 0) { if (proto.getTourneysQuitDuringRound() > 0) {
quit.add("R:" + proto.getTourneysQuitDuringRound());//Integer.toString(proto.getTourneysQuitDuringRound())); quit.add("R:" + Integer.toString(proto.getTourneysQuitDuringRound()));
} }
if (!quit.isEmpty()) { if (quit.size() > 0) {
builder.append(" ("); builder.append(" (");
joinStrings(builder, quit, " "); joinStrings(builder, quit, " ");
builder.append(")"); builder.append(")");

33
Mage.Sets/src/mage/sets/kaladesh/GlimmerOfGenius.java
View file

@ -1,33 +0,0 @@
package mage.sets.kaladesh;
import mage.abilities.effects.common.DrawCardSourceControllerEffect;
import mage.abilities.effects.common.counter.GetEnergyCountersControllerEffect;
import mage.abilities.effects.keyword.ScryEffect;
import mage.cards.CardImpl;
import mage.constants.CardType;
import mage.constants.Rarity;
import java.util.UUID;
/**
* Created by IGOUDT on 16-9-2016.
*/
public class GlimmerOfGenius extends CardImpl {
public GlimmerOfGenius(final UUID ownerId){
super(ownerId,49,"Glimmer of Genius", Rarity.UNCOMMON, new CardType[]{CardType.INSTANT},"{3U}");
this.expansionSetCode = "KLD";
this.getSpellAbility().addEffect(new ScryEffect(2));
this.getSpellAbility().addEffect(new DrawCardSourceControllerEffect(2));
this.getSpellAbility().addEffect(new GetEnergyCountersControllerEffect(2));
}
public GlimmerOfGenius(final GlimmerOfGenius glimmerOfGenius){
super(glimmerOfGenius);
}
public GlimmerOfGenius copy(){
return new GlimmerOfGenius(this);
}
}

46
Mage.Sets/src/mage/sets/kaladesh/RevolutionaryRebuff.java
View file

@ -1,46 +0,0 @@
package mage.sets.kaladesh;
import mage.abilities.costs.mana.GenericManaCost;
import mage.abilities.effects.common.CounterUnlessPaysEffect;
import mage.cards.Card;
import mage.cards.CardImpl;
import mage.constants.CardType;
import mage.constants.Rarity;
import mage.filter.Filter;
import mage.filter.FilterSpell;
import mage.filter.predicate.Predicates;
import mage.filter.predicate.mageobject.SubtypePredicate;
import mage.target.TargetSpell;
import mage.target.common.TargetNonBasicLandPermanent;
import java.util.UUID;
/**
* Created by IGOUDT on 16-9-2016.
*/
public class RevolutionaryRebuff extends CardImpl {
public RevolutionaryRebuff(final UUID ownerId){
super(ownerId, 61, "Revolutionary Rebuff", Rarity.COMMON, new CardType[]{CardType.INSTANT},"{1U}");
this.expansionSetCode = "KLD";
FilterSpell filter = new FilterSpell();
filter.add(Predicates.not(new SubtypePredicate("Artifact")));
this.getSpellAbility().addTarget(new TargetSpell(filter) );
this.getSpellAbility().addEffect(new CounterUnlessPaysEffect(new GenericManaCost(2)));
}
public RevolutionaryRebuff(final RevolutionaryRebuff revolutionaryRebuff){
super(revolutionaryRebuff);
}
@Override
public RevolutionaryRebuff copy() {
return new RevolutionaryRebuff(this);
}
}

42
Mage.Sets/src/mage/sets/kaladesh/RushOfVitality.java
View file

@ -1,42 +0,0 @@
package mage.sets.kaladesh;
import mage.abilities.effects.common.continuous.BoostTargetEffect;
import mage.abilities.effects.common.continuous.GainAbilityTargetEffect;
import mage.abilities.keyword.DeathtouchAbility;
import mage.abilities.keyword.IndestructibleAbility;
import mage.abilities.keyword.LifelinkAbility;
import mage.cards.Card;
import mage.cards.CardImpl;
import mage.constants.CardType;
import mage.constants.Duration;
import mage.constants.Rarity;
import mage.target.common.TargetCreaturePermanent;
import java.util.UUID;
/**
* Created by IGOUDT on 16-9-2016.
*/
public class RushOfVitality extends CardImpl {
public RushOfVitality(final UUID ownerId){
super(ownerId,99,"Rush of Vitality", Rarity.COMMON, new CardType[]{CardType.INSTANT},"{1B}");
this.expansionSetCode = "KLD";
this.getSpellAbility().addTarget(new TargetCreaturePermanent());
this.getSpellAbility().addEffect(new BoostTargetEffect(1, 0, Duration.EndOfTurn));
this.getSpellAbility().addEffect(new GainAbilityTargetEffect(IndestructibleAbility.getInstance(), Duration.EndOfTurn));
this.getSpellAbility().addEffect(new GainAbilityTargetEffect(LifelinkAbility.getInstance(), Duration.EndOfTurn));
}
public RushOfVitality(final RushOfVitality rushOfVitality){
super(rushOfVitality);
}
@Override
public RushOfVitality copy() {
return new RushOfVitality(this);
}
}

41
Mage.Sets/src/mage/sets/kaladesh/VedalkenBlademaster.java
View file

@ -1,41 +0,0 @@
package mage.sets.kaladesh;
import mage.MageInt;
import mage.abilities.keyword.FlyingAbility;
import mage.abilities.keyword.ProwessAbility;
import mage.cards.Card;
import mage.cards.CardImpl;
import mage.constants.CardType;
import mage.constants.Rarity;
import java.util.UUID;
/**
* Created by IGOUDT on 16-9-2016.
*/
public class VedalkenBlademaster extends CardImpl {
public VedalkenBlademaster(final UUID ownerId){
super(ownerId, 68, "Vedalken Blademaster", Rarity.COMMON, new CardType[]{CardType.CREATURE},"{2U}");
this.expansionSetCode = "KLD";
this.subtype.add("Vedalken");
this.subtype.add("Soldier");
this.power = new MageInt(2);
this.toughness = new MageInt(3);
this.addAbility(new ProwessAbility());
}
public VedalkenBlademaster(final VedalkenBlademaster card){
super(card);
}
@Override
public VedalkenBlademaster copy() {
return new VedalkenBlademaster(this);
}
}

7
Mage/src/main/java/mage/ConditionalMana.java
View file

@ -206,21 +206,26 @@ public class ConditionalMana extends Mana implements Serializable {
break; break;
case BLUE: case BLUE:
blue += amount; blue += amount;
;
break; break;
case GREEN: case GREEN:
green += amount; green += amount;
;
break; break;
case RED: case RED:
red += amount; red += amount;
;
break; break;
case WHITE: case WHITE:
white += amount; white += amount;
;
break; break;
case COLORLESS: case COLORLESS:
colorless += amount; colorless += amount;
;
case GENERIC: case GENERIC:
generic += amount; generic += amount;
;
break; break;
} }
} }

6
Mage/src/main/java/mage/abilities/AbilitiesImpl.java
View file

@ -39,7 +39,6 @@ import mage.abilities.keyword.ProtectionAbility;
import mage.abilities.mana.ManaAbility; import mage.abilities.mana.ManaAbility;
import mage.constants.Zone; import mage.constants.Zone;
import mage.game.Game; import mage.game.Game;
import mage.util.StringUtil;
import mage.util.ThreadLocalStringBuilder; import mage.util.ThreadLocalStringBuilder;
import org.apache.log4j.Logger; import org.apache.log4j.Logger;
@ -106,10 +105,11 @@ public class AbilitiesImpl<T extends Ability> extends ArrayList<T> implements Ab
rules.add(sbRule.toString()); rules.add(sbRule.toString());
} }
String rule = ability.getRule(); String rule = ability.getRule();
if (StringUtil.isNotEmpty(rule)){ if (rule != null) {
if (rule.length() > 0) {
rules.add(Character.toUpperCase(rule.charAt(0)) + rule.substring(1)); rules.add(Character.toUpperCase(rule.charAt(0)) + rule.substring(1));
} }
else { // logging so we can still can be made aware of rule problems a card has } else { // logging so we can still can be made aware of rule problems a card has
String cardName = ((SpellAbility) ability).getCardName(); String cardName = ((SpellAbility) ability).getCardName();
logger.fatal("Error in rule text generation of " + cardName + ": Create a bug report or fix the source code"); logger.fatal("Error in rule text generation of " + cardName + ": Create a bug report or fix the source code");
} }

5
Mage/src/main/java/mage/abilities/costs/mana/ManaCostsImpl.java
View file

@ -44,7 +44,6 @@ import mage.players.ManaPool;
import mage.players.Player; import mage.players.Player;
import mage.target.Targets; import mage.target.Targets;
import mage.util.ManaUtil; import mage.util.ManaUtil;
import mage.util.StringUtil;
/** /**
* @author BetaSteward_at_googlemail.com * @author BetaSteward_at_googlemail.com
@ -321,13 +320,13 @@ public class ManaCostsImpl<T extends ManaCost> extends ArrayList<T> implements M
this.add(cost.copy()); this.add(cost.copy());
} }
} else { } else {
if (StringUtil.isNotEmpty(mana)) { if (mana == null || mana.isEmpty()) {
return; return;
} }
String[] symbols = mana.split("^\\{|\\}\\{|\\}$"); String[] symbols = mana.split("^\\{|\\}\\{|\\}$");
int modifierForX = 0; int modifierForX = 0;
for (String symbol : symbols) { for (String symbol : symbols) {
if (StringUtil.isNotEmpty(symbol)) { if (symbol.length() > 0) {
if (symbol.length() == 1 || isNumeric(symbol)) { if (symbol.length() == 1 || isNumeric(symbol)) {
if (Character.isDigit(symbol.charAt(0))) { if (Character.isDigit(symbol.charAt(0))) {
this.add(new GenericManaCost(Integer.valueOf(symbol))); this.add(new GenericManaCost(Integer.valueOf(symbol)));

3
Mage/src/main/java/mage/abilities/decorator/ConditionalReplacementEffect.java
View file

@ -37,7 +37,6 @@ import mage.abilities.effects.ReplacementEffectImpl;
import mage.constants.Duration; import mage.constants.Duration;
import mage.game.Game; import mage.game.Game;
import mage.game.events.GameEvent; import mage.game.events.GameEvent;
import mage.util.StringUtil;
/** /**
* *
@ -144,7 +143,7 @@ public class ConditionalReplacementEffect extends ReplacementEffectImpl {
@Override @Override
public String getText(Mode mode) { public String getText(Mode mode) {
if (StringUtil.isEmpty(staticText) && this.effect != null) { // usefull for conditional night/day card abilities if (staticText == null || staticText.isEmpty() && this.effect != null) { // usefull for conditional night/day card abilities
return effect.getText(mode); return effect.getText(mode);
} }
return staticText; return staticText;

2
Mage/src/main/java/mage/abilities/effects/common/CreateTokenEffect.java
View file

@ -129,7 +129,7 @@ public class CreateTokenEffect extends OneShotEffect {
sb.append(" attacking"); sb.append(" attacking");
} }
String message = amount.getMessage(); String message = amount.getMessage();
if (!message.isEmpty()) { if (message.length() > 0) {
if (amount.toString().equals("X")) { if (amount.toString().equals("X")) {
sb.append(", where X is "); sb.append(", where X is ");
} else { } else {

2
Mage/src/main/java/mage/abilities/effects/common/CreateTokenTargetEffect.java
View file

@ -80,7 +80,7 @@ public class CreateTokenTargetEffect extends OneShotEffect {
sb.append(" attacking"); sb.append(" attacking");
} }
String message = amount.getMessage(); String message = amount.getMessage();
if (!message.isEmpty()) { if (message.length() > 0) {
sb.append(" for each "); sb.append(" for each ");
} }
sb.append(message); sb.append(message);

3
Mage/src/main/java/mage/abilities/effects/common/DamageControllerEffect.java
View file

@ -36,7 +36,6 @@ import mage.abilities.dynamicvalue.common.StaticValue;
import mage.abilities.effects.OneShotEffect; import mage.abilities.effects.OneShotEffect;
import mage.game.Game; import mage.game.Game;
import mage.players.Player; import mage.players.Player;
import mage.util.StringUtil;
/** /**
* *
@ -104,7 +103,7 @@ public class DamageControllerEffect extends OneShotEffect {
sb.append(amount); sb.append(amount);
} }
sb.append(" damage to you"); sb.append(" damage to you");
if (StringUtil.isNotEmpty(message)) { if (message.length() > 0) {
if (message.equals("1")) { if (message.equals("1")) {
sb.append(" equal to the number of "); sb.append(" equal to the number of ");
} else { } else {

7
Mage/src/main/java/mage/abilities/effects/common/DamageTargetEffect.java
View file

@ -37,7 +37,6 @@ import mage.game.Game;
import mage.game.permanent.Permanent; import mage.game.permanent.Permanent;
import mage.players.Player; import mage.players.Player;
import mage.target.Target; import mage.target.Target;
import mage.util.StringUtil;
import java.util.UUID; import java.util.UUID;
@ -143,7 +142,7 @@ public class DamageTargetEffect extends OneShotEffect {
@Override @Override
public String getText(Mode mode) { public String getText(Mode mode) {
if (StringUtil.isNotEmpty(staticText)) { if (staticText != null && !staticText.isEmpty()) {
return staticText; return staticText;
} }
StringBuilder sb = new StringBuilder(); StringBuilder sb = new StringBuilder();
@ -153,12 +152,12 @@ public class DamageTargetEffect extends OneShotEffect {
sb.append(amount); sb.append(amount);
} }
sb.append(" damage to "); sb.append(" damage to ");
if (StringUtil.isNotEmpty(targetDescription)){ if (targetDescription.length() > 0) {
sb.append(targetDescription); sb.append(targetDescription);
} else { } else {
sb.append("target ").append(mode.getTargets().get(0).getTargetName()); sb.append("target ").append(mode.getTargets().get(0).getTargetName());
} }
if (StringUtil.isNotEmpty(message)) { if (message.length() > 0) {
if (message.equals("1")) { if (message.equals("1")) {
sb.append(" equal to the number of "); sb.append(" equal to the number of ");
} else { } else {

2
Mage/src/main/java/mage/abilities/effects/common/DontUntapInControllersNextUntapStepTargetEffect.java
View file

@ -158,7 +158,7 @@ public class DontUntapInControllersNextUntapStepTargetEffect extends ContinuousR
if (staticText != null && !staticText.isEmpty()) { if (staticText != null && !staticText.isEmpty()) {
return staticText; return staticText;
} }
if (targetName != null && !targetName.isEmpty()) { if (targetName != null && targetName.length() > 0) {
if (targetName.equals("Those creatures") || targetName.equals("They")) { if (targetName.equals("Those creatures") || targetName.equals("They")) {
return targetName + " don't untap during their controller's next untap step"; return targetName + " don't untap during their controller's next untap step";
} else } else

3
Mage/src/main/java/mage/abilities/effects/common/DrawCardSourceControllerEffect.java
View file

@ -36,7 +36,6 @@ import mage.constants.Outcome;
import mage.game.Game; import mage.game.Game;
import mage.players.Player; import mage.players.Player;
import mage.util.CardUtil; import mage.util.CardUtil;
import mage.util.StringUtil;
/** /**
* @author BetaSteward_at_googlemail.com * @author BetaSteward_at_googlemail.com
@ -84,7 +83,7 @@ public class DrawCardSourceControllerEffect extends OneShotEffect {
sb.append("s"); sb.append("s");
} }
String message = amount.getMessage(); String message = amount.getMessage();
if (StringUtil.isNotEmpty(message)) { if (message.length() > 0) {
sb.append(" for each "); sb.append(" for each ");
} }
sb.append(message); sb.append(message);

2
Mage/src/main/java/mage/abilities/effects/common/DrawCardTargetEffect.java
View file

@ -136,7 +136,7 @@ public class DrawCardTargetEffect extends OneShotEffect {
sb.append("s"); sb.append("s");
} }
String message = amount.getMessage(); String message = amount.getMessage();
if (!message.isEmpty()) { if (message.length() > 0) {
sb.append(" for each "); sb.append(" for each ");
} }
sb.append(message); sb.append(message);

3
Mage/src/main/java/mage/abilities/effects/common/LoseLifeControllerAttachedEffect.java
View file

@ -36,7 +36,6 @@ import mage.constants.Zone;
import mage.game.Game; import mage.game.Game;
import mage.game.permanent.Permanent; import mage.game.permanent.Permanent;
import mage.players.Player; import mage.players.Player;
import mage.util.StringUtil;
public class LoseLifeControllerAttachedEffect extends OneShotEffect { public class LoseLifeControllerAttachedEffect extends OneShotEffect {
@ -89,7 +88,7 @@ public class LoseLifeControllerAttachedEffect extends OneShotEffect {
StringBuilder sb = new StringBuilder(); StringBuilder sb = new StringBuilder();
sb.append("it's controller loses ").append(amount.toString()).append(" life"); sb.append("it's controller loses ").append(amount.toString()).append(" life");
String message = amount.getMessage(); String message = amount.getMessage();
if (StringUtil.isNotEmpty(message)) { if (message.length() > 0) {
sb.append(" for each "); sb.append(" for each ");
sb.append(message); sb.append(message);
} }

2
Mage/src/main/java/mage/abilities/effects/common/LoseLifeOpponentsEffect.java
View file

@ -87,7 +87,7 @@ public class LoseLifeOpponentsEffect extends OneShotEffect {
sb.append(amount).append(" "); sb.append(amount).append(" ");
} }
sb.append("life"); sb.append("life");
if (!message.isEmpty()) { if (message.length() > 0) {
sb.append(message.equals("1") || message.startsWith("the ") ? " equal to the number of " : " for each "); sb.append(message.equals("1") || message.startsWith("the ") ? " equal to the number of " : " for each ");
sb.append(message); sb.append(message);
} }

3
Mage/src/main/java/mage/abilities/effects/common/LoseLifeSourceControllerEffect.java
View file

@ -35,7 +35,6 @@ import mage.abilities.dynamicvalue.common.StaticValue;
import mage.abilities.effects.OneShotEffect; import mage.abilities.effects.OneShotEffect;
import mage.game.Game; import mage.game.Game;
import mage.players.Player; import mage.players.Player;
import mage.util.StringUtil;
/** /**
* *
@ -79,7 +78,7 @@ public class LoseLifeSourceControllerEffect extends OneShotEffect {
StringBuilder sb = new StringBuilder(); StringBuilder sb = new StringBuilder();
sb.append("you lose ").append(amount.toString()).append(" life"); sb.append("you lose ").append(amount.toString()).append(" life");
String message = amount.getMessage(); String message = amount.getMessage();
if (StringUtil.isNotEmpty(message)) { if (message.length() > 0) {
sb.append(" for each "); sb.append(" for each ");
} }
sb.append(message); sb.append(message);

5
Mage/src/main/java/mage/abilities/effects/common/LoseLifeTargetEffect.java
View file

@ -35,7 +35,6 @@ import mage.abilities.effects.OneShotEffect;
import mage.constants.Outcome; import mage.constants.Outcome;
import mage.game.Game; import mage.game.Game;
import mage.players.Player; import mage.players.Player;
import mage.util.StringUtil;
/** /**
* *
@ -82,7 +81,7 @@ public class LoseLifeTargetEffect extends OneShotEffect {
StringBuilder sb = new StringBuilder(); StringBuilder sb = new StringBuilder();
String message = amount.getMessage(); String message = amount.getMessage();
if (!mode.getTargets().isEmpty()) { if (mode.getTargets().size() > 0) {
sb.append("target ").append(mode.getTargets().get(0).getTargetName()); sb.append("target ").append(mode.getTargets().get(0).getTargetName());
} else { } else {
sb.append("that player"); sb.append("that player");
@ -92,7 +91,7 @@ public class LoseLifeTargetEffect extends OneShotEffect {
sb.append(amount).append(" "); sb.append(amount).append(" ");
} }
sb.append("life"); sb.append("life");
if (StringUtil.isNotEmpty(message)) { if (message.length() > 0) {
if (amount.toString().equals("X")) { if (amount.toString().equals("X")) {
sb.append(", where X is "); sb.append(", where X is ");
} else { } else {

2
Mage/src/main/java/mage/abilities/effects/common/PutLibraryIntoGraveTargetEffect.java
View file

@ -101,7 +101,7 @@ public class PutLibraryIntoGraveTargetEffect extends OneShotEffect {
} }
sb.append("of his or her library into his or her graveyard"); sb.append("of his or her library into his or her graveyard");
if (!message.isEmpty()) { if (message.length() > 0) {
sb.append(", where X is the number of "); sb.append(", where X is the number of ");
sb.append(message); sb.append(message);
} }

3
Mage/src/main/java/mage/abilities/effects/common/SkipNextPlayerUntapStepEffect.java
View file

@ -37,7 +37,6 @@ import mage.abilities.effects.OneShotEffect;
import mage.game.Game; import mage.game.Game;
import mage.game.turn.TurnMod; import mage.game.turn.TurnMod;
import mage.players.Player; import mage.players.Player;
import mage.util.StringUtil;
/** /**
* *
@ -78,7 +77,7 @@ public class SkipNextPlayerUntapStepEffect extends OneShotEffect {
@Override @Override
public String getText(Mode mode) { public String getText(Mode mode) {
StringBuilder sb = new StringBuilder(); StringBuilder sb = new StringBuilder();
if (StringUtil.isNotEmpty(staticText)) { if (staticText.length() > 0) {
sb.append(staticText); sb.append(staticText);
} else { } else {
sb.append("target"); sb.append("target");

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

@ -37,7 +37,6 @@ import mage.game.Game;
import mage.game.permanent.Permanent; import mage.game.permanent.Permanent;
import mage.target.Target; import mage.target.Target;
import mage.util.CardUtil; import mage.util.CardUtil;
import mage.util.StringUtil;
/** /**
* @author BetaSteward_at_googlemail.com * @author BetaSteward_at_googlemail.com
@ -77,7 +76,7 @@ public class TapTargetEffect extends OneShotEffect {
@Override @Override
public String getText(Mode mode) { public String getText(Mode mode) {
if (StringUtil.isNotEmpty(staticText)) { if (staticText.length() > 0) {
return "tap " + staticText; return "tap " + staticText;
} }

3
Mage/src/main/java/mage/abilities/effects/common/TransformTargetEffect.java
View file

@ -14,7 +14,6 @@ import mage.game.permanent.Permanent;
import mage.game.permanent.PermanentCard; import mage.game.permanent.PermanentCard;
import mage.target.Target; import mage.target.Target;
import mage.util.CardUtil; import mage.util.CardUtil;
import mage.util.StringUtil;
/** /**
* *
@ -74,7 +73,7 @@ public class TransformTargetEffect extends OneShotEffect {
@Override @Override
public String getText(Mode mode) { public String getText(Mode mode) {
if (StringUtil.isNotEmpty(staticText)) { if (staticText != null && staticText.length() > 0) {
return staticText; return staticText;
} }
if (mode.getTargets().isEmpty()) { if (mode.getTargets().isEmpty()) {

2
Mage/src/main/java/mage/abilities/effects/common/continuous/AssignNoCombatDamageSourceEffect.java
View file

@ -106,7 +106,7 @@ public class AssignNoCombatDamageSourceEffect extends ReplacementEffectImpl {
text += " this combat"; text += " this combat";
break; break;
default: default:
if (!duration.toString().isEmpty()) { if (duration.toString().length() > 0) {
text += " " + duration.toString(); text += " " + duration.toString();
} }
} }

4
Mage/src/main/java/mage/abilities/effects/common/continuous/BecomesColorOrColorsTargetEffect.java
View file

@ -118,11 +118,11 @@ public class BecomesColorOrColorsTargetEffect extends OneShotEffect {
return staticText; return staticText;
} }
StringBuilder sb = new StringBuilder(); StringBuilder sb = new StringBuilder();
if (!mode.getTargets().isEmpty()) { if (mode.getTargets().size() > 0) {
sb.append("target "); sb.append("target ");
sb.append(mode.getTargets().get(0).getFilter().getMessage()); sb.append(mode.getTargets().get(0).getFilter().getMessage());
sb.append(" becomes the color or colors of your choice"); sb.append(" becomes the color or colors of your choice");
if (!duration.toString().isEmpty()) { if (duration.toString().length() > 0) {
sb.append(" ").append(duration.toString()); sb.append(" ").append(duration.toString());
} }
} }

5
Mage/src/main/java/mage/abilities/effects/common/continuous/BecomesCreatureAllEffect.java
View file

@ -39,7 +39,6 @@ import mage.filter.FilterPermanent;
import mage.game.Game; import mage.game.Game;
import mage.game.permanent.Permanent; import mage.game.permanent.Permanent;
import mage.game.permanent.token.Token; import mage.game.permanent.token.Token;
import mage.util.StringUtil;
/** /**
* @author LevelX2 * @author LevelX2
@ -77,11 +76,13 @@ public class BecomesCreatureAllEffect extends ContinuousEffectImpl {
switch (layer) { switch (layer) {
case TypeChangingEffects_4: case TypeChangingEffects_4:
if (sublayer == SubLayer.NA) { if (sublayer == SubLayer.NA) {
if (token.getCardType().size() > 0) {
for (CardType t : token.getCardType()) { for (CardType t : token.getCardType()) {
if (!permanent.getCardType().contains(t)) { if (!permanent.getCardType().contains(t)) {
permanent.getCardType().add(t); permanent.getCardType().add(t);
} }
} }
}
if (type == null) { if (type == null) {
permanent.getSubtype(game).clear(); permanent.getSubtype(game).clear();
} }
@ -139,7 +140,7 @@ public class BecomesCreatureAllEffect extends ContinuousEffectImpl {
} }
sb.append(filter.getMessage()); sb.append(filter.getMessage());
sb.append(" become a ").append(token.getDescription()); sb.append(" become a ").append(token.getDescription());
if (StringUtil.isNotEmpty(type)) { if (type != null && type.length() > 0) {
sb.append(". They are still ").append(type); sb.append(". They are still ").append(type);
} }
return sb.toString(); return sb.toString();

2
Mage/src/main/java/mage/abilities/effects/common/continuous/BecomesCreatureSourceEffect.java
View file

@ -172,7 +172,7 @@ public class BecomesCreatureSourceEffect extends ContinuousEffectImpl implements
} }
private void setText() { private void setText() {
if (type != null && !type.isEmpty()) { if (type != null && type.length() > 0) {
staticText = duration.toString() + " {this} becomes a " + token.getDescription() + " that's still a " + this.type; staticText = duration.toString() + " {this} becomes a " + token.getDescription() + " that's still a " + this.type;
} else { } else {
staticText = duration.toString() + " {this} becomes a " + token.getDescription(); staticText = duration.toString() + " {this} becomes a " + token.getDescription();

2
Mage/src/main/java/mage/abilities/effects/common/continuous/BoostEnchantedEffect.java
View file

@ -134,7 +134,7 @@ public class BoostEnchantedEffect extends ContinuousEffectImpl {
sb.append(" ").append(duration.toString()); sb.append(" ").append(duration.toString());
} }
String message = power.getMessage(); String message = power.getMessage();
if (!message.isEmpty()) { if (message.length() > 0) {
sb.append(" for each "); sb.append(" for each ");
} }
sb.append(message); sb.append(message);

3
Mage/src/main/java/mage/abilities/effects/common/continuous/BoostEquippedEffect.java
View file

@ -39,7 +39,6 @@ import mage.abilities.effects.ContinuousEffectImpl;
import mage.game.Game; import mage.game.Game;
import mage.game.permanent.Permanent; import mage.game.permanent.Permanent;
import mage.target.targetpointer.FixedTarget; import mage.target.targetpointer.FixedTarget;
import mage.util.StringUtil;
/** /**
* @author BetaSteward_at_googlemail.com * @author BetaSteward_at_googlemail.com
@ -133,7 +132,7 @@ public class BoostEquippedEffect extends ContinuousEffectImpl {
if (duration != Duration.WhileOnBattlefield) if (duration != Duration.WhileOnBattlefield)
sb.append(" ").append(duration.toString()); sb.append(" ").append(duration.toString());
String message = power.getMessage(); String message = power.getMessage();
if (StringUtil.isNotEmpty(message)) { if (message.length() > 0) {
sb.append(" for each "); sb.append(" for each ");
} }
sb.append(message); sb.append(message);

3
Mage/src/main/java/mage/abilities/effects/common/counter/AddCountersAttachedEffect.java
View file

@ -36,7 +36,6 @@ import mage.counters.Counter;
import mage.game.Game; import mage.game.Game;
import mage.game.permanent.Permanent; import mage.game.permanent.Permanent;
import mage.util.CardUtil; import mage.util.CardUtil;
import mage.util.StringUtil;
/** /**
* *
@ -102,7 +101,7 @@ public class AddCountersAttachedEffect extends OneShotEffect {
} }
sb.append(counter.getName().toLowerCase()).append(" counter on "); sb.append(counter.getName().toLowerCase()).append(" counter on ");
sb.append(textEnchanted); sb.append(textEnchanted);
if (StringUtil.isNotEmpty(amount.getMessage())) { if (amount.getMessage().length() > 0) {
sb.append(" for each ").append(amount.getMessage()); sb.append(" for each ").append(amount.getMessage());
} }
staticText = sb.toString(); staticText = sb.toString();

3
Mage/src/main/java/mage/abilities/effects/common/counter/AddCountersTargetEffect.java
View file

@ -43,7 +43,6 @@ import mage.game.permanent.Permanent;
import mage.players.Player; import mage.players.Player;
import mage.target.Target; import mage.target.Target;
import mage.util.CardUtil; import mage.util.CardUtil;
import mage.util.StringUtil;
/** /**
* *
@ -165,7 +164,7 @@ public class AddCountersTargetEffect extends OneShotEffect {
sb.append("that creature"); sb.append("that creature");
} }
if (StringUtil.isNotEmpty(amount.getMessage())) { if (amount.getMessage().length() > 0) {
sb.append(" for each ").append(amount.getMessage()); sb.append(" for each ").append(amount.getMessage());
} }
return sb.toString(); return sb.toString();

3
Mage/src/main/java/mage/abilities/effects/common/discard/DiscardControllerEffect.java
View file

@ -36,7 +36,6 @@ import mage.constants.Outcome;
import mage.game.Game; import mage.game.Game;
import mage.players.Player; import mage.players.Player;
import mage.util.CardUtil; import mage.util.CardUtil;
import mage.util.StringUtil;
/** /**
* *
@ -117,7 +116,7 @@ public class DiscardControllerEffect extends OneShotEffect {
sb.append(" at random"); sb.append(" at random");
} }
String message = amount.getMessage(); String message = amount.getMessage();
if (StringUtil.isNotEmpty(message)) { if (message.length() > 0) {
sb.append(" for each "); sb.append(" for each ");
} }
sb.append(message); sb.append(message);

5
Mage/src/main/java/mage/abilities/effects/common/discard/DiscardHandTargetEffect.java
View file

@ -35,7 +35,6 @@ import mage.abilities.effects.OneShotEffect;
import mage.constants.Outcome; import mage.constants.Outcome;
import mage.game.Game; import mage.game.Game;
import mage.players.Player; import mage.players.Player;
import mage.util.StringUtil;
/** /**
* *
@ -81,11 +80,11 @@ public class DiscardHandTargetEffect extends OneShotEffect {
@Override @Override
public String getText(Mode mode) { public String getText(Mode mode) {
if (StringUtil.isNotEmpty(staticText)) { if (staticText != null && !staticText.isEmpty()) {
return staticText; return staticText;
} }
StringBuilder sb = new StringBuilder(); StringBuilder sb = new StringBuilder();
if (StringUtil.isNotEmpty(targetDescription)) { if (targetDescription.length() > 0) {
sb.append(targetDescription); sb.append(targetDescription);
} else { } else {
sb.append("target ").append(mode.getTargets().get(0).getTargetName()); sb.append("target ").append(mode.getTargets().get(0).getTargetName());

5
Mage/src/main/java/mage/abilities/effects/common/discard/DiscardTargetEffect.java
View file

@ -36,7 +36,6 @@ import mage.constants.Outcome;
import mage.game.Game; import mage.game.Game;
import mage.players.Player; import mage.players.Player;
import mage.util.CardUtil; import mage.util.CardUtil;
import mage.util.StringUtil;
/** /**
* *
@ -96,7 +95,7 @@ public class DiscardTargetEffect extends OneShotEffect {
@Override @Override
public String getText(Mode mode) { public String getText(Mode mode) {
if (StringUtil.isNotEmpty(staticText)) { if (staticText != null && !staticText.isEmpty()) {
return staticText; return staticText;
} }
StringBuilder sb = new StringBuilder(); StringBuilder sb = new StringBuilder();
@ -115,7 +114,7 @@ public class DiscardTargetEffect extends OneShotEffect {
sb.append(" at random"); sb.append(" at random");
} }
String message = amount.getMessage(); String message = amount.getMessage();
if (StringUtil.isNotEmpty(message)) { if (message.length() > 0) {
sb.append(" for each "); sb.append(" for each ");
} }
sb.append(message); sb.append(message);

3
Mage/src/main/java/mage/abilities/effects/keyword/ManifestTargetPlayerEffect.java
View file

@ -44,7 +44,6 @@ import mage.game.Game;
import mage.game.permanent.Permanent; import mage.game.permanent.Permanent;
import mage.players.Player; import mage.players.Player;
import mage.util.CardUtil; import mage.util.CardUtil;
import mage.util.StringUtil;
/** /**
* *
@ -105,7 +104,7 @@ public class ManifestTargetPlayerEffect extends OneShotEffect {
private String setText() { private String setText() {
StringBuilder sb = new StringBuilder(); StringBuilder sb = new StringBuilder();
if (StringUtil.isNotEmpty(prefix)) { if (prefix != null && !prefix.isEmpty()) {
sb.append(prefix).append(" "); sb.append(prefix).append(" ");
} }
sb.append("manifest the top "); sb.append("manifest the top ");

2
Mage/src/main/java/mage/abilities/keyword/CascadeAbility.java
View file

@ -133,7 +133,7 @@ class CascadeEffect extends OneShotEffect {
Cards cardsFromExile = new CardsImpl(); Cards cardsFromExile = new CardsImpl();
Cards cardsToLibrary = new CardsImpl(); Cards cardsToLibrary = new CardsImpl();
cardsFromExile.addAll(exile); cardsFromExile.addAll(exile);
while (!cardsFromExile.isEmpty()) { while (cardsFromExile.size() > 0) {
card = cardsFromExile.getRandom(game); card = cardsFromExile.getRandom(game);
cardsFromExile.remove(card.getId()); cardsFromExile.remove(card.getId());
cardsToLibrary.add(card); cardsToLibrary.add(card);

6
Mage/src/main/java/mage/cards/Sets.java
View file

@ -46,7 +46,6 @@ import mage.constants.ColoredManaSymbol;
import mage.util.ClassScanner; import mage.util.ClassScanner;
import mage.util.RandomUtil; import mage.util.RandomUtil;
import mage.util.StringUtil;
import org.apache.log4j.Logger; import org.apache.log4j.Logger;
/** /**
@ -139,11 +138,10 @@ public class Sets extends HashMap<String, ExpansionSet> {
Map<String, DeckCardInfo> deckCards = new HashMap<>(); Map<String, DeckCardInfo> deckCards = new HashMap<>();
Map<String, DeckCardInfo> sideboard = new HashMap<>(); Map<String, DeckCardInfo> sideboard = new HashMap<>();
try { try {
if (deck.getName() != null && deck.getName().length() > 0) {
if (StringUtil.isNotEmpty(deck.getName())) {
out.println("NAME:" + deck.getName()); out.println("NAME:" + deck.getName());
} }
if (StringUtil.isNotEmpty(deck.getAuthor())) { if (deck.getAuthor() != null && deck.getAuthor().length() > 0) {
out.println("AUTHOR:" + deck.getAuthor()); out.println("AUTHOR:" + deck.getAuthor());
} }
for (DeckCardInfo deckCardInfo: deck.getCards()) { for (DeckCardInfo deckCardInfo: deck.getCards()) {

15
Mage/src/main/java/mage/util/StringUtil.java
View file

@ -1,15 +0,0 @@
package mage.util;
/**
* Created by IGOUDT on 16-9-2016.
*/
public class StringUtil {
public static boolean isEmpty(String input){
return (input == null) || input.isEmpty();
}
public static boolean isNotEmpty(String input){
return !isEmpty(input);
}
}