Skip to content

Commit

Permalink
Merge pull request #12 from CottaCush/Fix/ParseMoneyCrash
Browse files Browse the repository at this point in the history
Fix parseMoneyValueWithLocale
  • Loading branch information
efguydan authored Feb 19, 2020
2 parents 206b9d0 + b4886ff commit 7a80c6c
Showing 1 changed file with 14 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -18,9 +18,15 @@ package com.cottacush.android.currencyedittext
import android.os.Build
import androidx.annotation.RequiresApi
import java.text.NumberFormat
import java.util.*
import java.text.ParseException
import java.util.Locale
import java.util.IllformedLocaleException

internal fun parseMoneyValue(value: String, groupingSeparator: String, currencySymbol: String): String =
internal fun parseMoneyValue(
value: String,
groupingSeparator: String,
currencySymbol: String
): String =
value.replace(groupingSeparator, "").replace(currencySymbol, "")

internal fun parseMoneyValueWithLocale(
Expand All @@ -29,8 +35,13 @@ internal fun parseMoneyValueWithLocale(
groupingSeparator: String,
currencySymbol: String
): Number {

val valueWithoutSeparator = parseMoneyValue(value, groupingSeparator, currencySymbol)
return NumberFormat.getInstance(locale).parse(valueWithoutSeparator)!!
return try {
NumberFormat.getInstance(locale).parse(valueWithoutSeparator)!!
} catch (exception: ParseException) {
0
}
}

@RequiresApi(Build.VERSION_CODES.LOLLIPOP)
Expand Down

0 comments on commit 7a80c6c

Please sign in to comment.