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

Moves "Current_Deck" from Quest Preferences to Quest Data.

Browse Source 
Without this, the selected deck name was global for all quests - switching to another quest retained the deck name from the previous quest
and if the new quest had a deck with the identical name, selected that one, otherwise selected nothing. (in one weird case, this resulted in quest data that crashed caused bugs when loading forge - as quest was saved with a selected deck from another quest data.)
Preferred behavior is to select the deck that was last selected in THAT QUEST when switching to another quest.
This commit is contained in:
Seravy
2018-02-13 01:32:18 +01:00
parent 67e8046af6
commit 56983b2553
8 changed files with 26 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
forge-gui-desktop/src/main/java/forge/screens/deckeditor/SEditorIO.java
View File

@@ -62,7 +62,7 @@ public class SEditorIO {
VAllDecks.SINGLETON_INSTANCE.getLstDecks().setSelectedString(deckStr);
// Set current quest deck to selected
if (Singletons.getControl().getCurrentScreen() == FScreen.DECK_EDITOR_QUEST) {
FModel.getQuestPreferences().setPref(QPref.CURRENT_DECK, name);
FModel.getQuest().setCurrentDeck(name);
}
}

6
forge-gui-desktop/src/main/java/forge/screens/home/quest/CSubmenuQuestDecks.java
View File

@@ -30,10 +30,10 @@ public enum CSubmenuQuestDecks implements ICDoc {
public void run() {
final DeckProxy deck = VSubmenuQuestDecks.SINGLETON_INSTANCE.getLstDecks().getSelectedItem();
if (deck != null) {
FModel.getQuestPreferences().setPref(QPref.CURRENT_DECK, deck.toString());
FModel.getQuest().setCurrentDeck(deck.toString());
}
else {
FModel.getQuestPreferences().setPref(QPref.CURRENT_DECK, QPref.CURRENT_DECK.getDefault());
FModel.getQuest().setCurrentDeck(QPref.CURRENT_DECK.getDefault());
}
FModel.getQuestPreferences().save();
}
@@ -84,7 +84,7 @@ public enum CSubmenuQuestDecks implements ICDoc {
view.getLstDecks().setup(ItemManagerConfig.QUEST_DECKS);
// Look through list for preferred deck from prefs
final DeckProxy deck = hasQuest ? view.getLstDecks().stringToItem(FModel.getQuestPreferences().getPref(QPref.CURRENT_DECK)) : null;
final DeckProxy deck = hasQuest ? view.getLstDecks().stringToItem(FModel.getQuest().getCurrentDeck()) : null;
if (deck != null) {
view.getLstDecks().setSelectedItem(deck);
}

4
forge-gui-mobile/src/forge/deck/FDeckEditor.java
View File

@@ -1620,11 +1620,11 @@ public class FDeckEditor extends TabPageScreen<FDeckEditor> {
DeckPreferences.setSealedDeck(deckStr);
break;
case Quest:
FModel.getQuestPreferences().setPref(QPref.CURRENT_DECK, model.toString());
FModel.getQuest().setCurrentDeck(model.toString());
FModel.getQuest().save();
break;
case QuestDraft:
FModel.getQuestPreferences().setPref(QPref.CURRENT_DECK, model.toString());
FModel.getQuest().setCurrentDeck(model.toString());
FModel.getQuest().save();
break;
default:

6
forge-gui-mobile/src/forge/screens/quest/QuestDecksScreen.java
View File

@@ -42,10 +42,10 @@ public class QuestDecksScreen extends FScreen {
public void handleEvent(FEvent e) {
DeckProxy deck = lstDecks.getSelectedItem();
if (deck != null) {
FModel.getQuestPreferences().setPref(QPref.CURRENT_DECK, deck.toString());
FModel.getQuest().setCurrentDeck(deck.toString());
}
else {
FModel.getQuestPreferences().setPref(QPref.CURRENT_DECK, QPref.CURRENT_DECK.getDefault());
FModel.getQuest().setCurrentDeck(QPref.CURRENT_DECK.getDefault());
}
FModel.getQuestPreferences().save();
}
@@ -135,7 +135,7 @@ public class QuestDecksScreen extends FScreen {
lstDecks.setup(ItemManagerConfig.QUEST_DECKS);
// Look through list for preferred deck from prefs
final DeckProxy deck = hasQuest ? lstDecks.stringToItem(FModel.getQuestPreferences().getPref(QPref.CURRENT_DECK)) : null;
final DeckProxy deck = hasQuest ? lstDecks.stringToItem(FModel.getQuest().getCurrentDeck()) : null;
if (deck != null) {
lstDecks.setSelectedItem(deck);
}

8
forge-gui/src/main/java/forge/quest/QuestController.java
View File

@@ -577,4 +577,12 @@ public class QuestController {
CardEdition randomLandSet = CardEdition.Predicates.getRandomSetWithAllBasicLands(availableEditions);
return randomLandSet == null ? FModel.getMagicDb().getEditions().get("ZEN") : randomLandSet;
}
public String getCurrentDeck() {
return model.currentDeck;
}
public void setCurrentDeck(String s) {
model.currentDeck = s;
}
}

2
forge-gui/src/main/java/forge/quest/QuestUtil.java
View File

@@ -443,7 +443,7 @@ public class QuestUtil {
if (FModel.getQuest().getAssets() != null) {
d = FModel.getQuest().getMyDecks().get(
FModel.getQuestPreferences().getPref(QPref.CURRENT_DECK));
FModel.getQuest().getCurrentDeck());
}
return d;

4
forge-gui/src/main/java/forge/quest/data/QuestData.java
View File

@@ -42,7 +42,7 @@ import java.util.Map;
*/
public final class QuestData {
/** Holds the latest version of the Quest Data. */
public static final int CURRENT_VERSION_NUMBER = 11;
public static final int CURRENT_VERSION_NUMBER = 12;
// This field places the version number into QD instance,
// but only when the object is created through the constructor
@@ -68,6 +68,8 @@ public final class QuestData {
public HashSet<StarRating> Ratings = new HashSet<StarRating>();
public String currentDeck = "DEFAULT";
public QuestData() { //needed for XML serialization
}

5
forge-gui/src/main/java/forge/quest/io/QuestDataIO.java
View File

@@ -17,6 +17,7 @@
*/
package forge.quest.io;
import forge.quest.data.QuestPreferences.QPref;
import com.thoughtworks.xstream.XStream;
import com.thoughtworks.xstream.converters.Converter;
import com.thoughtworks.xstream.converters.MarshallingContext;
@@ -201,6 +202,10 @@ public class QuestDataIO {
newData.Ratings.clear();
}
if (saveVersion < 12) {
// Current Deck moved from preferences to quest data - it should not be global for all quests!!!
QuestDataIO.setFinalField(QuestData.class, "currentDeck", newData, FModel.getQuestPreferences().getPref(QPref.CURRENT_DECK));
}
if (saveVersion < 13) {
// Migrate DraftTournaments to use new Tournament class
}