Fix unit converter aliases
This commit is contained in:
@@ -122,7 +122,7 @@ class CurrencyRepository(
|
||||
* @return The resolved currency symbol or the original symbol if no alias was found
|
||||
*/
|
||||
fun resolveAlias(symbol: String): String {
|
||||
return currencySymbolAliases[symbol] ?: symbol
|
||||
return currencySymbolAliases[symbol] ?: symbol.uppercase()
|
||||
}
|
||||
|
||||
suspend fun convertCurrency(
|
||||
@@ -168,7 +168,7 @@ class CurrencyRepository(
|
||||
}
|
||||
|
||||
suspend fun isValidCurrency(symbol: String): Boolean {
|
||||
val isoSymbol = currencySymbolAliases[symbol] ?: symbol
|
||||
val isoSymbol = currencySymbolAliases[symbol] ?: symbol.uppercase()
|
||||
return withContext(Dispatchers.IO) {
|
||||
AppDatabase.getInstance(context).currencyDao().exists(isoSymbol)
|
||||
}
|
||||
|
||||
@@ -28,7 +28,7 @@ class CurrencyConverter(
|
||||
private val topCurrencies = arrayOf("USD", "EUR", "JPY", "GBP", "AUD")
|
||||
|
||||
override suspend fun isValidUnit(symbol: String): Boolean {
|
||||
return repository.isValidCurrency(symbol.uppercase())
|
||||
return repository.isValidCurrency(symbol)
|
||||
}
|
||||
|
||||
|
||||
@@ -76,7 +76,7 @@ class CurrencyConverter(
|
||||
value: Double,
|
||||
toUnit: String?
|
||||
): UnitConverter {
|
||||
val fromIsoCode = repository.resolveAlias(fromUnit.uppercase())
|
||||
val fromIsoCode = repository.resolveAlias(fromUnit)
|
||||
val toIsoCode = toUnit?.let { repository.resolveAlias(it) }
|
||||
|
||||
val values = repository.convertCurrency(fromIsoCode, value, toIsoCode).map {
|
||||
|
||||
Reference in New Issue
Block a user