external-repos/forge
1
0
Fork 0
mirror of https://github.com/Card-Forge/forge.git synced 2025-11-20 04:38:00 +00:00
Code Issues Packages Projects Releases Wiki Activity

Prevent crash when resolving Ponder on mobile game

Browse Source
This commit is contained in:
drdev
2014-04-29 04:42:53 +00:00
parent 21a1c26cc7
commit 2394b8fb19
2 changed files with 16 additions and 32 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
forge-gui/src/main/java/forge/SGuiChoose.java
View File

@@ -65,10 +65,12 @@ public class SGuiChoose {
}
public static <T> T one(final String message, final Collection<T> choices) {
if (choices == null || choices.isEmpty())
if (choices == null || choices.isEmpty()) {
return null;
if( choices.size() == 1)
}
if (choices.size() == 1) {
return Iterables.getFirst(choices, null);
}
final List<T> choice = SGuiChoose.getChoices(message, 1, 1, choices);
assert choice.size() == 1;

42
forge-gui/src/main/java/forge/SGuiDialog.java
View File

@@ -1,12 +1,8 @@
package forge;
import forge.FThreads;
import forge.game.card.Card;
import org.apache.commons.lang3.StringUtils;
import java.util.concurrent.Callable;
import java.util.concurrent.FutureTask;
/**
* Holds player interactions using standard windows
*
@@ -25,29 +21,20 @@ public class SGuiDialog {
}
public static boolean confirm(final Card c, final String question, final boolean defaultIsYes, final String[] options) {
Callable<Boolean> confirmTask = new Callable<Boolean>() {
@Override
public Boolean call() throws Exception {
if (c != null) {
if (c != null) {
FThreads.invokeInEdtAndWait(new Runnable() {
@Override
public void run() {
GuiBase.getInterface().setCard(c);
}
final String title = c == null ? "Question" : c.getName() + " - Ability";
String questionToUse = StringUtils.isBlank(question) ? "Activate card's ability?" : question;
String[] opts = options == null ? defaultConfirmOptions : options;
int answer = SOptionPane.showOptionDialog(questionToUse, title, SOptionPane.QUESTION_ICON, opts, defaultIsYes ? 0 : 1);
return answer == 0;
}};
FutureTask<Boolean> future = new FutureTask<Boolean>(confirmTask);
FThreads.invokeInEdtAndWait(future);
try {
return future.get().booleanValue();
});
}
catch (Exception e) { // should be no exception here
e.printStackTrace();
}
return false;
final String title = c == null ? "Question" : c.getName() + " - Ability";
String questionToUse = StringUtils.isBlank(question) ? "Activate card's ability?" : question;
String[] opts = options == null ? defaultConfirmOptions : options;
int answer = SOptionPane.showOptionDialog(questionToUse, title, SOptionPane.QUESTION_ICON, opts, defaultIsYes ? 0 : 1);
return answer == 0;
}
/**
@@ -63,11 +50,6 @@ public class SGuiDialog {
}
public static void message(final String message, final String title) {
FThreads.invokeInEdtAndWait(new Runnable() {
@Override
public void run() {
SOptionPane.showMessageDialog(message, title, null);
}
});
SOptionPane.showMessageDialog(message, title, null);
}
}