Skip to content

Commit 750078d

Browse files
committed
sort traditional currencies after fiat currencies in combined pull down
1 parent dff7e88 commit 750078d

File tree

3 files changed

+13
-4
lines changed

3 files changed

+13
-4
lines changed

core/src/main/java/haveno/core/locale/CurrencyUtil.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ public static void setBaseCurrencyCode(String baseCurrencyCode) {
9393
public static Collection<TraditionalCurrency> getAllSortedFiatCurrencies(Comparator comparator) {
9494
return getAllSortedTraditionalCurrencies(comparator).stream()
9595
.filter(currency -> CurrencyUtil.isFiatCurrency(currency.getCode()))
96-
.collect(Collectors.toList()); // sorted by currency name
96+
.collect(Collectors.toList()); // sorted by currency name
9797
}
9898

9999
public static List<TradeCurrency> getAllFiatCurrencies() {
@@ -105,11 +105,11 @@ public static List<TradeCurrency> getAllFiatCurrencies() {
105105
public static List<TradeCurrency> getAllSortedFiatCurrencies() {
106106
return getAllSortedTraditionalCurrencies().stream()
107107
.filter(currency -> CurrencyUtil.isFiatCurrency(currency.getCode()))
108-
.collect(Collectors.toList()); // sorted by currency name
108+
.collect(Collectors.toList()); // sorted by currency name
109109
}
110110

111111
public static Collection<TraditionalCurrency> getAllSortedTraditionalCurrencies() {
112-
return traditionalCurrencyMapSupplier.get().values(); // sorted by currency name
112+
return traditionalCurrencyMapSupplier.get().values(); // sorted by currency name
113113
}
114114

115115
public static List<TradeCurrency> getAllTraditionalCurrencies() {

desktop/src/main/java/haveno/desktop/util/CurrencyList.java

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,6 +65,7 @@ public void updateWithCurrencies(List<TradeCurrency> currencies, @Nullable Curre
6565

6666
private List<CurrencyListItem> getPartitionedSortedItems(List<TradeCurrency> currencies) {
6767
Map<TradeCurrency, Integer> tradesPerCurrency = countTrades(currencies);
68+
List<CurrencyListItem> fiatCurrencies = new ArrayList<>();
6869
List<CurrencyListItem> traditionalCurrencies = new ArrayList<>();
6970
List<CurrencyListItem> cryptoCurrencies = new ArrayList<>();
7071

@@ -73,7 +74,9 @@ private List<CurrencyListItem> getPartitionedSortedItems(List<TradeCurrency> cur
7374
Integer count = entry.getValue();
7475
CurrencyListItem item = new CurrencyListItem(currency, count);
7576

76-
if (predicates.isTraditionalCurrency(currency)) {
77+
if (predicates.isFiatCurrency(currency)) {
78+
fiatCurrencies.add(item);
79+
} else if (predicates.isTraditionalCurrency(currency)) {
7780
traditionalCurrencies.add(item);
7881
}
7982

@@ -83,10 +86,12 @@ private List<CurrencyListItem> getPartitionedSortedItems(List<TradeCurrency> cur
8386
}
8487

8588
Comparator<CurrencyListItem> comparator = getComparator();
89+
fiatCurrencies.sort(comparator);
8690
traditionalCurrencies.sort(comparator);
8791
cryptoCurrencies.sort(comparator);
8892

8993
List<CurrencyListItem> result = new ArrayList<>();
94+
result.addAll(fiatCurrencies);
9095
result.addAll(traditionalCurrencies);
9196
result.addAll(cryptoCurrencies);
9297

desktop/src/main/java/haveno/desktop/util/CurrencyPredicates.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,10 @@ boolean isCryptoCurrency(TradeCurrency currency) {
2525
return CurrencyUtil.isCryptoCurrency(currency.getCode());
2626
}
2727

28+
boolean isFiatCurrency(TradeCurrency currency) {
29+
return CurrencyUtil.isFiatCurrency(currency.getCode());
30+
}
31+
2832
boolean isTraditionalCurrency(TradeCurrency currency) {
2933
return CurrencyUtil.isTraditionalCurrency(currency.getCode());
3034
}

0 commit comments

Comments
 (0)