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

Merge branch 'revertDefaultSort' into 'master'

Browse Source 
Revert Default sort in Deckbuilder to code before Merge Request !1035

See merge request core-developers/forge!1042
This commit is contained in:
Michael Kamensky
2018-10-25 04:44:38 +00:00
parent e7cf359659 af7c5f7506
commit a001cb78c0
1 changed files with 9 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
forge-gui/src/main/java/forge/itemmanager/GroupDef.java
View File

@@ -74,12 +74,12 @@ public enum GroupDef {
return -1;
}
}),
DEFAULT("Default", //Beginning in DDU, "Artifacts" category is added at top of list
new String[] { "Artifacts", "Creatures", "Other Spells", "Lands" },
DEFAULT("Default",
new String[] { "Creatures", "Spells", "Lands" },
new Function<Integer, ColumnDef>() {
@Override
public ColumnDef apply(final Integer groupIndex) {
if (groupIndex == 3) {
if (groupIndex == 2) {
return ColumnDef.NAME; //pile lands by name regardless
}
return null;
@@ -90,22 +90,20 @@ public enum GroupDef {
public Integer apply(final InventoryItem item) {
if (item instanceof PaperCard) {
CardType type = ((PaperCard) item).getRules().getType();
if (type.isArtifact()) { //artifact lands/creatures, too
if (type.isCreature()) {
return 0;
}
if (type.isCreature()) {
return 1;
}
if (type.isLand()) {
return 3;
}
if (type.isEnchantment() || type.isPlaneswalker() || type.isInstant() || type.isSorcery()) {
if (type.isLand()) { //make Artifact Lands appear in Lands group
return 2;
}
if (type.isArtifact() || type.isEnchantment() || type.isPlaneswalker() || type.isInstant() || type.isSorcery()) {
return 1;
}
}
return -1;
}
}),
CARD_TYPE("Type",
new String[] { "Planeswalker", "Creature", "Sorcery", "Instant", "Artifact", "Enchantment", "Land", "Tribal instant" },
new Function<Integer, ColumnDef>() {