feat(ad-hoc): Refactor card events (#370)

Author: vitalii-vanziak-cko

sdk/src/main/kotlin/com/processout/sdk/api/model/request/POCardTokenizationPreferredSchemeRequest.kt

@@ -11,6 +11,7 @@ import java.util.UUID
  * @param[issuerInformation] Holds information about card issuing institution that issued the card to the card holder.
  * @param[uuid] Unique identifier of request.
  */
+@Deprecated(message = "Not used.")
 data class POCardTokenizationPreferredSchemeRequest @ProcessOutInternalApi constructor(
     val issuerInformation: POCardIssuerInformation,
     override val uuid: UUID = UUID.randomUUID()

sdk/src/main/kotlin/com/processout/sdk/api/model/request/POCardTokenizationProcessingRequest.kt

@@ -11,6 +11,7 @@ import java.util.UUID
  * @param[saveCard] Indicates whether the user has chosen to save the card for future payments.
  * @param[uuid] Unique identifier of request.
  */
+@Deprecated(message = "Not used.")
 data class POCardTokenizationProcessingRequest(
     val card: POCard,
     val saveCard: Boolean,

sdk/src/main/kotlin/com/processout/sdk/api/model/request/POCardTokenizationShouldContinueRequest.kt

@@ -11,6 +11,7 @@ import java.util.UUID
  * @param[failure] Failure that can be inspected to decide whether the flow should continue or complete.
  * @param[uuid] Unique identifier of request.
  */
+@Deprecated(message = "Not used.")
 data class POCardTokenizationShouldContinueRequest @ProcessOutInternalApi constructor(
     val failure: ProcessOutResult.Failure,
     override val uuid: UUID = UUID.randomUUID()

sdk/src/main/kotlin/com/processout/sdk/api/model/request/POCardUpdateShouldContinueRequest.kt

@@ -12,6 +12,7 @@ import java.util.UUID
  * @param[failure] Failure that can be inspected to decide whether the flow should continue or complete.
  * @param[uuid] Unique identifier of request.
  */
+@Deprecated(message = "Not used.")
 data class POCardUpdateShouldContinueRequest @ProcessOutInternalApi constructor(
     val cardId: String,
     val failure: ProcessOutResult.Failure,

sdk/src/main/kotlin/com/processout/sdk/api/model/response/POCardTokenizationPreferredSchemeResponse.kt

@@ -12,6 +12,7 @@ import java.util.UUID
  * @param[issuerInformation] Holds information about card issuing institution that issued the card to the card holder.
  * @param[preferredScheme] Preferred scheme that will be used by default for card tokenization.
  */
+@Deprecated(message = "Not used.")
 data class POCardTokenizationPreferredSchemeResponse internal constructor(
     override val uuid: UUID,
     val issuerInformation: POCardIssuerInformation,
@@ -23,6 +24,7 @@ data class POCardTokenizationPreferredSchemeResponse internal constructor(
  *
  * @param[preferredScheme] Preferred scheme that will be used by default for card tokenization. Will use a primary scheme if _null_.
  */
+@Deprecated(message = "Not used.")
 fun POCardTokenizationPreferredSchemeRequest.toResponse(
     preferredScheme: String?
 ) = POCardTokenizationPreferredSchemeResponse(uuid, issuerInformation, preferredScheme)

sdk/src/main/kotlin/com/processout/sdk/api/model/response/POCardTokenizationProcessingResponse.kt

@@ -12,6 +12,7 @@ import java.util.UUID
  * @param[uuid] Unique identifier of response that must be equal to UUID of request.
  * @param[result] Result of tokenized card processing.
  */
+@Deprecated(message = "Not used.")
 data class POCardTokenizationProcessingResponse internal constructor(
     override val uuid: UUID,
     val result: ProcessOutResult<POCard>
@@ -22,6 +23,7 @@ data class POCardTokenizationProcessingResponse internal constructor(
  *
  * @param[result] Result of tokenized card processing.
  */
+@Deprecated(message = "Not used.")
 fun POCardTokenizationProcessingRequest.toResponse(
     result: ProcessOutResult<Any>
 ) = POCardTokenizationProcessingResponse(

sdk/src/main/kotlin/com/processout/sdk/api/model/response/POCardTokenizationShouldContinueResponse.kt

@@ -13,6 +13,7 @@ import java.util.UUID
  * @param[failure] Failure that occurred. Must be provided from [POCardTokenizationShouldContinueRequest].
  * @param[shouldContinue] Boolean that indicates whether the flow should continue or complete after the [failure].
  */
+@Deprecated(message = "Not used.")
 data class POCardTokenizationShouldContinueResponse internal constructor(
     override val uuid: UUID,
     val failure: ProcessOutResult.Failure,
@@ -24,6 +25,7 @@ data class POCardTokenizationShouldContinueResponse internal constructor(
  *
  * @param[shouldContinue] Boolean that indicates whether the flow should continue or complete after the failure.
  */
+@Deprecated(message = "Not used.")
 fun POCardTokenizationShouldContinueRequest.toResponse(
     shouldContinue: Boolean
 ) = POCardTokenizationShouldContinueResponse(uuid, failure, shouldContinue)

sdk/src/main/kotlin/com/processout/sdk/api/model/response/POCardUpdateShouldContinueResponse.kt

@@ -13,6 +13,7 @@ import java.util.UUID
  * @param[failure] Failure that occurred. Must be provided from [POCardUpdateShouldContinueRequest].
  * @param[shouldContinue] Boolean that indicates whether the flow should continue or complete after the [failure].
  */
+@Deprecated(message = "Not used.")
 data class POCardUpdateShouldContinueResponse internal constructor(
     override val uuid: UUID,
     val failure: ProcessOutResult.Failure,
@@ -24,6 +25,7 @@ data class POCardUpdateShouldContinueResponse internal constructor(
  *
  * @param[shouldContinue] Boolean that indicates whether the flow should continue or complete after the failure.
  */
+@Deprecated(message = "Not used.")
 fun POCardUpdateShouldContinueRequest.toResponse(
     shouldContinue: Boolean
 ) = POCardUpdateShouldContinueResponse(uuid, failure, shouldContinue)

ui/src/main/kotlin/com/processout/sdk/ui/card/tokenization/CardTokenizationInteractor.kt

@@ -7,8 +7,11 @@ import com.processout.sdk.R
 import com.processout.sdk.api.dispatcher.POEventDispatcher
 import com.processout.sdk.api.model.event.POCardTokenizationEvent
 import com.processout.sdk.api.model.event.POCardTokenizationEvent.*
-import com.processout.sdk.api.model.request.*
-import com.processout.sdk.api.model.response.*
+import com.processout.sdk.api.model.request.POCardTokenizationRequest
+import com.processout.sdk.api.model.request.POContact
+import com.processout.sdk.api.model.response.POCard
+import com.processout.sdk.api.model.response.POCardIssuerInformation
+import com.processout.sdk.api.model.response.POCardScheme
 import com.processout.sdk.api.repository.POCardsRepository
 import com.processout.sdk.core.POFailure.Code.Cancelled
 import com.processout.sdk.core.POFailure.Code.Generic
@@ -26,12 +29,9 @@ import com.processout.sdk.ui.card.tokenization.CardTokenizationEvent.*
 import com.processout.sdk.ui.card.tokenization.CardTokenizationInteractorState.*
 import com.processout.sdk.ui.card.tokenization.CardTokenizationSideEffect.CardScanner
 import com.processout.sdk.ui.card.tokenization.POCardTokenizationConfiguration.BillingAddressConfiguration.CollectionMode.*
-import com.processout.sdk.ui.card.tokenization.delegate.CardTokenizationEligibilityRequest
-import com.processout.sdk.ui.card.tokenization.delegate.CardTokenizationEligibilityResponse
-import com.processout.sdk.ui.card.tokenization.delegate.POCardTokenizationEligibility
+import com.processout.sdk.ui.card.tokenization.delegate.*
 import com.processout.sdk.ui.card.tokenization.delegate.POCardTokenizationEligibility.Eligible
 import com.processout.sdk.ui.card.tokenization.delegate.POCardTokenizationEligibility.NotEligible
-import com.processout.sdk.ui.card.tokenization.delegate.POCardTokenizationState
 import com.processout.sdk.ui.core.state.POAvailableValue
 import com.processout.sdk.ui.shared.extension.currentAppLocale
 import com.processout.sdk.ui.shared.extension.distinctUntilChangedByMultiple
@@ -88,10 +88,10 @@ internal class CardTokenizationInteractor(
 
     private var issuerInformationJob: Job? = null
 
-    private var latestProcessingRequest: POCardTokenizationProcessingRequest? = null
+    private var latestProcessingRequest: CardTokenizationProcessingRequest? = null
     private var latestEligibilityRequest: CardTokenizationEligibilityRequest? = null
-    private var latestPreferredSchemeRequest: POCardTokenizationPreferredSchemeRequest? = null
-    private var latestShouldContinueRequest: POCardTokenizationShouldContinueRequest? = null
+    private var latestPreferredSchemeRequest: CardTokenizationPreferredSchemeRequest? = null
+    private var latestShouldContinueRequest: CardTokenizationShouldContinueRequest? = null
 
     //region Initialization
 
@@ -507,15 +507,15 @@ internal class CardTokenizationInteractor(
 
     private fun requestPreferredScheme(issuerInformation: POCardIssuerInformation) {
         interactorScope.launch {
-            val request = POCardTokenizationPreferredSchemeRequest(issuerInformation)
+            val request = CardTokenizationPreferredSchemeRequest(issuerInformation = issuerInformation)
             latestPreferredSchemeRequest = request
             eventDispatcher.send(request)
             POLogger.info("Requested to choose preferred scheme by issuer information: %s", issuerInformation)
         }
     }
 
     private fun collectPreferredScheme() {
-        eventDispatcher.subscribeForResponse<POCardTokenizationPreferredSchemeResponse>(
+        eventDispatcher.subscribeForResponse<CardTokenizationPreferredSchemeResponse>(
             coroutineScope = interactorScope
         ) { response ->
             if (response.uuid == latestPreferredSchemeRequest?.uuid) {
@@ -827,7 +827,7 @@ internal class CardTokenizationInteractor(
     }
 
     private suspend fun requestToProcessTokenizedCard(card: POCard) {
-        val request = POCardTokenizationProcessingRequest(
+        val request = CardTokenizationProcessingRequest(
             card = card,
             saveCard = _state.value.saveCardField.value.text.toBooleanStrictOrNull() ?: false
         )
@@ -840,7 +840,7 @@ internal class CardTokenizationInteractor(
     }
 
     private fun handleCompletion() {
-        eventDispatcher.subscribeForResponse<POCardTokenizationProcessingResponse>(
+        eventDispatcher.subscribeForResponse<CardTokenizationProcessingResponse>(
             coroutineScope = interactorScope
         ) { response ->
             if (response.uuid == latestProcessingRequest?.uuid) {
@@ -865,15 +865,15 @@ internal class CardTokenizationInteractor(
 
     private fun requestIfShouldContinue(failure: ProcessOutResult.Failure) {
         interactorScope.launch {
-            val request = POCardTokenizationShouldContinueRequest(failure)
+            val request = CardTokenizationShouldContinueRequest(failure = failure)
             latestShouldContinueRequest = request
             eventDispatcher.send(request)
             POLogger.info("Requested to decide whether the flow should continue or complete after the failure: %s", failure)
         }
     }
 
     private fun shouldContinueOnFailure() {
-        eventDispatcher.subscribeForResponse<POCardTokenizationShouldContinueResponse>(
+        eventDispatcher.subscribeForResponse<CardTokenizationShouldContinueResponse>(
             coroutineScope = interactorScope
         ) { response ->
             if (response.uuid == latestShouldContinueRequest?.uuid) {

ui/src/main/kotlin/com/processout/sdk/ui/card/tokenization/POCardTokenizationLauncher.kt

@@ -9,16 +9,9 @@ import androidx.lifecycle.lifecycleScope
 import com.processout.sdk.R
 import com.processout.sdk.api.dispatcher.POEventDispatcher
 import com.processout.sdk.api.model.event.POCardTokenizationEvent
-import com.processout.sdk.api.model.request.POCardTokenizationPreferredSchemeRequest
-import com.processout.sdk.api.model.request.POCardTokenizationProcessingRequest
-import com.processout.sdk.api.model.request.POCardTokenizationShouldContinueRequest
 import com.processout.sdk.api.model.response.POCard
-import com.processout.sdk.api.model.response.toResponse
 import com.processout.sdk.core.ProcessOutActivityResult
-import com.processout.sdk.ui.card.tokenization.delegate.CardTokenizationEligibilityRequest
-import com.processout.sdk.ui.card.tokenization.delegate.POCardTokenizationDelegate
-import com.processout.sdk.ui.card.tokenization.delegate.POCardTokenizationState
-import com.processout.sdk.ui.card.tokenization.delegate.toResponse
+import com.processout.sdk.ui.card.tokenization.delegate.*
 import kotlinx.coroutines.CoroutineScope
 import kotlinx.coroutines.launch
 
@@ -136,7 +129,7 @@ class POCardTokenizationLauncher private constructor(
     }
 
     private fun dispatchTokenizedCard() {
-        eventDispatcher.subscribeForRequest<POCardTokenizationProcessingRequest>(
+        eventDispatcher.subscribeForRequest<CardTokenizationProcessingRequest>(
             coroutineScope = scope
         ) { request ->
             scope.launch {
@@ -164,7 +157,7 @@ class POCardTokenizationLauncher private constructor(
     }
 
     private fun dispatchPreferredScheme() {
-        eventDispatcher.subscribeForRequest<POCardTokenizationPreferredSchemeRequest>(
+        eventDispatcher.subscribeForRequest<CardTokenizationPreferredSchemeRequest>(
             coroutineScope = scope
         ) { request ->
             scope.launch {
@@ -175,7 +168,7 @@ class POCardTokenizationLauncher private constructor(
     }
 
     private fun dispatchShouldContinue() {
-        eventDispatcher.subscribeForRequest<POCardTokenizationShouldContinueRequest>(
+        eventDispatcher.subscribeForRequest<CardTokenizationShouldContinueRequest>(
             coroutineScope = scope
         ) { request ->
             scope.launch {