Skip to content

Commit

Permalink
fix: make android card validation state consistent with iOS
Browse files Browse the repository at this point in the history
  • Loading branch information
charliecruzan-stripe committed May 23, 2022
1 parent 5780386 commit 1ff8b4e
Showing 1 changed file with 13 additions and 3 deletions.
16 changes: 13 additions & 3 deletions android/src/main/java/com/reactnativestripesdk/CardFieldView.kt
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ import com.stripe.android.model.PaymentMethodCreateParams
import com.stripe.android.view.CardInputListener
import com.stripe.android.view.CardInputWidget
import com.stripe.android.view.CardValidCallback
import com.stripe.android.view.StripeEditText

class CardFieldView(context: ThemedReactContext) : FrameLayout(context) {
private var mCardWidget: CardInputWidget = CardInputWidget(context)
Expand Down Expand Up @@ -228,9 +229,18 @@ class CardFieldView(context: ThemedReactContext) : FrameLayout(context) {
}

mCardWidget.setCardValidCallback { isValid, invalidFields ->
cardDetails["validNumber"] = if (invalidFields.contains(CardValidCallback.Fields.Number)) "Invalid" else "Valid"
cardDetails["validCVC"] = if (invalidFields.contains(CardValidCallback.Fields.Cvc)) "Invalid" else "Valid"
cardDetails["validExpiryDate"] = if (invalidFields.contains(CardValidCallback.Fields.Expiry)) "Invalid" else "Valid"
fun getCardValidationState(field: CardValidCallback.Fields, editTextField: StripeEditText): String {
if (invalidFields.contains(field)) {
return if (editTextField.shouldShowError) "Invalid"
else "Incomplete"
}
return "Valid"
}

cardDetails["validNumber"] = getCardValidationState(CardValidCallback.Fields.Number, cardInputWidgetBinding.cardNumberEditText)
cardDetails["validCVC"] = getCardValidationState(CardValidCallback.Fields.Cvc, cardInputWidgetBinding.cvcEditText)
cardDetails["validExpiryDate"] = getCardValidationState(CardValidCallback.Fields.Expiry, cardInputWidgetBinding.expiryDateEditText)

if (isValid) {
onValidCardChange()
} else {
Expand Down

0 comments on commit 1ff8b4e

Please sign in to comment.