Merge pull request #5 from Mindera/dev/ALFMOB-81_add_to_bag

[ALFMOB-81] Add to Bag

Author: artyomromanov

data/src/main/java/au/com/alfie/ecomm/data/bag/BagRepositoryImpl.kt

@@ -0,0 +1,34 @@
+package au.com.alfie.ecomm.data.bag
+
+import au.com.alfie.ecomm.data.toRepositoryResult
+import au.com.alfie.ecomm.repository.bag.BagProduct
+import au.com.alfie.ecomm.repository.bag.BagRepository
+import au.com.alfie.ecomm.repository.result.RepositoryResult
+import kotlinx.coroutines.flow.Flow
+import kotlinx.coroutines.flow.MutableStateFlow
+import kotlinx.coroutines.flow.map
+import javax.inject.Inject
+
+class BagRepositoryImpl @Inject constructor() : BagRepository {
+
+    // TODO consider removing this property when the products in the bag are saved on database or api
+    private val _bag = MutableStateFlow<List<BagProduct>>(listOf())
+
+    // TODO change this implementation to a proper implementation using data base or api to get the products in the bag
+    override fun getBag(): Flow<RepositoryResult<List<BagProduct>>> {
+        return _bag.map { bag ->
+            Result.success(bag).toRepositoryResult()
+        }
+    }
+
+    // TODO change this implementation to a proper implementation using data base or api to save the product
+    override fun addToBag(bagProduct: BagProduct): RepositoryResult<Boolean> {
+        _bag.value = _bag.value.toMutableList().apply { add(bagProduct) }
+        return RepositoryResult.Success(true)
+    }
+
+    override fun removeFromBag(bagProduct: BagProduct): RepositoryResult<Boolean> {
+        _bag.value = _bag.value.toMutableList().apply { remove(bagProduct) }
+        return RepositoryResult.Success(true)
+    }
+}

data/src/main/java/au/com/alfie/ecomm/data/bag/di/BagModule.kt

@@ -0,0 +1,18 @@
+package au.com.alfie.ecomm.data.bag.di
+
+import au.com.alfie.ecomm.data.bag.BagRepositoryImpl
+import au.com.alfie.ecomm.repository.bag.BagRepository
+import dagger.Binds
+import dagger.Module
+import dagger.hilt.InstallIn
+import dagger.hilt.components.SingletonComponent
+import javax.inject.Singleton
+
+@Module
+@InstallIn(SingletonComponent::class)
+internal interface BagModule {
+
+    @Binds
+    @Singleton
+    fun bindBagRepository(bagRepositoryImpl: BagRepositoryImpl): BagRepository
+}

designsystem/src/main/java/au/com/alfie/ecomm/designsystem/component/price/PriceType.kt

@@ -1,16 +1,22 @@
 package au.com.alfie.ecomm.designsystem.component.price
 
+import androidx.compose.runtime.Stable
+
+@Stable
 sealed interface PriceType {
 
+    @Stable
     data class Default(
         val price: String
     ) : PriceType
 
+    @Stable
     data class Sale(
         val fullPrice: String,
         val salePrice: String
     ) : PriceType
 
+    @Stable
     data class Range(
         val startPrice: String,
         val endPrice: String

designsystem/src/main/java/au/com/alfie/ecomm/designsystem/component/productcard/ProductCardType.kt

@@ -16,7 +16,7 @@ sealed interface ProductCardType {
     val image: ImageUI
     val brand: String
     val name: String
-    val price: PriceType?
+    val price: PriceType
     val cardTestTag: String
     val imageTestTag: String
     val brandTestTag: String
@@ -30,6 +30,7 @@ sealed interface ProductCardType {
         override val price: PriceType,
         val color: String,
         val size: String,
+        val onRemoveClick: ClickEvent? = null,
         override val cardTestTag: String = PRODUCT_CARD,
         override val imageTestTag: String = PRODUCT_IMAGE,
         override val brandTestTag: String = PRODUCT_DESIGNER,
@@ -43,7 +44,7 @@ sealed interface ProductCardType {
         override val image: ImageUI,
         override val brand: String,
         override val name: String,
-        override val price: PriceType?,
+        override val price: PriceType,
         override val cardTestTag: String = PRODUCT_CARD,
         override val imageTestTag: String = PRODUCT_IMAGE,
         override val brandTestTag: String = PRODUCT_DESIGNER,

designsystem/src/main/java/au/com/alfie/ecomm/designsystem/component/productcard/size/ProductCardXSmall.kt

@@ -7,10 +7,13 @@ import androidx.compose.foundation.layout.Spacer
 import androidx.compose.foundation.layout.fillMaxWidth
 import androidx.compose.foundation.layout.size
 import androidx.compose.foundation.layout.width
+import androidx.compose.material3.Icon
+import androidx.compose.material3.IconButton
 import androidx.compose.material3.Text
 import androidx.compose.runtime.Composable
 import androidx.compose.ui.Modifier
 import androidx.compose.ui.platform.testTag
+import androidx.compose.ui.res.painterResource
 import androidx.compose.ui.res.stringResource
 import androidx.compose.ui.text.style.TextOverflow
 import androidx.compose.ui.tooling.preview.Preview
@@ -137,6 +140,18 @@ internal fun ProductCardXSmall(
                     .testTag(productCard.priceTestTag)
             )
         }
+        if (isLoading.not() && productCard.onRemoveClick != null) {
+            IconButton(
+                modifier = Modifier.size(Theme.iconSize.large),
+                onClick = productCard.onRemoveClick
+            ) {
+                Icon(
+                    painter = painterResource(id = R.drawable.ic_action_close_dark),
+                    contentDescription = null,
+                    modifier = Modifier.size(Theme.iconSize.small)
+                )
+            }
+        }
     }
 }
 

domain/repository/src/main/java/au/com/alfie/ecomm/repository/bag/BagProduct.kt

@@ -0,0 +1,6 @@
+package au.com.alfie.ecomm.repository.bag
+
+data class BagProduct(
+    val productId: String,
+    val variantSku: String
+)

domain/repository/src/main/java/au/com/alfie/ecomm/repository/bag/BagRepository.kt

@@ -0,0 +1,13 @@
+package au.com.alfie.ecomm.repository.bag
+
+import au.com.alfie.ecomm.repository.result.RepositoryResult
+import kotlinx.coroutines.flow.Flow
+
+interface BagRepository {
+
+    fun getBag(): Flow<RepositoryResult<List<BagProduct>>>
+
+    fun addToBag(bagProduct: BagProduct): RepositoryResult<Boolean>
+
+    fun removeFromBag(bagProduct: BagProduct): RepositoryResult<Boolean>
+}

domain/repository/src/main/java/au/com/alfie/ecomm/repository/product/model/Variant.kt

@@ -7,7 +7,7 @@ import au.com.alfie.ecomm.repository.shared.model.Size
 data class Variant(
     val attributes: List<Attribute>,
     val color: Color?,
-    val media: Media,
+    val media: Media.Image,
     val price: Price,
     val size: Size?,
     val sku: String,

domain/src/main/java/au/com/alfie/ecomm/domain/usecase/bag/AddToBagUseCase.kt

@@ -0,0 +1,28 @@
+package au.com.alfie.ecomm.domain.usecase.bag
+
+import au.com.alfie.ecomm.domain.UseCaseInteractor
+import au.com.alfie.ecomm.domain.UseCaseResult
+import au.com.alfie.ecomm.domain.doOnResult
+import au.com.alfie.ecomm.repository.bag.BagProduct
+import au.com.alfie.ecomm.repository.bag.BagRepository
+import javax.inject.Inject
+
+class AddToBagUseCase @Inject constructor(
+    private val bagRepository: BagRepository
+) : UseCaseInteractor {
+
+    suspend operator fun invoke(
+        productId: String,
+        variantSku: String
+    ) = run(
+        bagRepository.addToBag(
+            BagProduct(
+                productId = productId,
+                variantSku = variantSku
+            )
+        )
+    ).doOnResult(
+        onSuccess = { UseCaseResult.Success(it) },
+        onError = { UseCaseResult.Error(it) }
+    )
+}

domain/src/main/java/au/com/alfie/ecomm/domain/usecase/bag/GetBagUseCase.kt

@@ -0,0 +1,19 @@
+package au.com.alfie.ecomm.domain.usecase.bag
+
+import au.com.alfie.ecomm.domain.UseCaseInteractor
+import au.com.alfie.ecomm.domain.UseCaseResult
+import au.com.alfie.ecomm.repository.bag.BagProduct
+import au.com.alfie.ecomm.repository.bag.BagRepository
+import kotlinx.coroutines.flow.Flow
+import kotlinx.coroutines.flow.map
+import javax.inject.Inject
+
+class GetBagUseCase @Inject constructor(
+    private val bagRepository: BagRepository
+) : UseCaseInteractor {
+
+    suspend operator fun invoke(): Flow<UseCaseResult<List<BagProduct>>> =
+        bagRepository.getBag().map { repositoryResult ->
+            run(repositoryResult)
+        }
+}