Allow viewing recipes on V1

datasource/src/main/kotlin/gq/kirmanak/mealient/datasource/DataSourceModule.kt

@@ -6,6 +6,9 @@ import dagger.Module
 import dagger.Provides
 import dagger.hilt.InstallIn
 import dagger.hilt.components.SingletonComponent
+import gq.kirmanak.mealient.datasource.v1.MealieDataSourceV1
+import gq.kirmanak.mealient.datasource.v1.MealieDataSourceV1Impl
+import gq.kirmanak.mealient.datasource.v1.MealieServiceV1
 import kotlinx.serialization.ExperimentalSerializationApi
 import kotlinx.serialization.json.Json
 import okhttp3.MediaType.Companion.toMediaType
@@ -52,6 +55,10 @@ interface DataSourceModule {
         fun provideMealieService(retrofit: Retrofit): MealieService =
             retrofit.create()
 
+        @Provides
+        @Singleton
+        fun provideMealieServiceV1(retrofit: Retrofit): MealieServiceV1 =
+            retrofit.create()
     }
 
     @Binds
@@ -65,4 +72,8 @@ interface DataSourceModule {
     @Binds
     @Singleton
     fun bindMealieDataSource(mealientDataSourceImpl: MealieDataSourceImpl): MealieDataSource
+
+    @Binds
+    @Singleton
+    fun bindMealieDataSourceV1(mealientDataSourceImpl: MealieDataSourceV1Impl): MealieDataSourceV1
 }

datasource/src/main/kotlin/gq/kirmanak/mealient/datasource/MealieDataSourceImpl.kt

@@ -73,20 +73,7 @@ class MealieDataSourceImpl @Inject constructor(
         logParameters = { "baseUrl = $baseUrl, token = $token, start = $start, limit = $limit" }
     ).getOrElse {
         val code = (it as? HttpException)?.code() ?: throw it
-        if (code == 404) requestRecipesV1(baseUrl, token, start, limit) else throw it
-    }
-
-    private suspend fun requestRecipesV1(
-        baseUrl: String, token: String?, start: Int, limit: Int
-    ): List<GetRecipeSummaryResponse> {
-        // Imagine start is 30 and limit is 15. It means that we already have page 1 and 2, now we need page 3
-        val perPage = limit
-        val page = start / perPage + 1
-        return makeCall(
-            block = { getRecipeSummaryV1("$baseUrl/api/recipes", token, page, perPage) },
-            logMethod = { "requestRecipesV1" },
-            logParameters = { "baseUrl = $baseUrl, token = $token, start = $start, limit = $limit" }
-        ).map { it.items }.getOrThrowUnauthorized()
+        if (code == 404) throw NetworkError.NotMealie(it) else throw it
     }
 
     override suspend fun requestRecipeInfo(

datasource/src/main/kotlin/gq/kirmanak/mealient/datasource/MealieDataSourceV1Impl.kt

@@ -1,41 +0,0 @@
-package gq.kirmanak.mealient.datasource
-
-import gq.kirmanak.mealient.datasource.models.AddRecipeRequest
-import gq.kirmanak.mealient.datasource.models.GetRecipeResponse
-import gq.kirmanak.mealient.datasource.models.GetRecipeSummaryResponse
-import gq.kirmanak.mealient.datasource.models.VersionResponse
-
-class MealieDataSourceV1Impl : MealieDataSourceV1 {
-    override suspend fun addRecipe(
-        baseUrl: String,
-        token: String?,
-        recipe: AddRecipeRequest
-    ): String {
-        TODO("Not yet implemented")
-    }
-
-    override suspend fun authenticate(baseUrl: String, username: String, password: String): String {
-        TODO("Not yet implemented")
-    }
-
-    override suspend fun getVersionInfo(baseUrl: String): VersionResponse {
-        TODO("Not yet implemented")
-    }
-
-    override suspend fun requestRecipes(
-        baseUrl: String,
-        token: String?,
-        start: Int,
-        limit: Int
-    ): List<GetRecipeSummaryResponse> {
-        TODO("Not yet implemented")
-    }
-
-    override suspend fun requestRecipeInfo(
-        baseUrl: String,
-        token: String?,
-        slug: String
-    ): GetRecipeResponse {
-        TODO("Not yet implemented")
-    }
-}

datasource/src/main/kotlin/gq/kirmanak/mealient/datasource/MealieService.kt

@@ -34,14 +34,6 @@ interface MealieService {
         @Query("limit") limit: Int,
     ): List<GetRecipeSummaryResponse>
 
-    @GET
-    suspend fun getRecipeSummaryV1(
-        @Url url: String,
-        @Header(AUTHORIZATION_HEADER_NAME) token: String?,
-        @Query("page") page: Int,
-        @Query("perPage") perPage: Int,
-    ): GetRecipesResponseV1
-
     @GET
     suspend fun getRecipe(
         @Url url: String,

datasource/src/main/kotlin/gq/kirmanak/mealient/datasource/models/GetRecipeSummaryResponse.kt

@@ -7,7 +7,7 @@ import kotlinx.serialization.Serializable
 
 @Serializable
 data class GetRecipeSummaryResponse(
-    @SerialName("id") val remoteId: String,
+    @SerialName("id") val remoteId: Int,
     @SerialName("name") val name: String,
     @SerialName("slug") val slug: String,
     @SerialName("image") val image: String?,

datasource/src/main/kotlin/gq/kirmanak/mealient/datasource/v1/MealieDataSourceV1.kt

@@ -1,9 +1,9 @@
-package gq.kirmanak.mealient.datasource
+package gq.kirmanak.mealient.datasource.v1
 
 import gq.kirmanak.mealient.datasource.models.AddRecipeRequest
 import gq.kirmanak.mealient.datasource.models.GetRecipeResponse
-import gq.kirmanak.mealient.datasource.models.GetRecipeSummaryResponse
 import gq.kirmanak.mealient.datasource.models.VersionResponse
+import gq.kirmanak.mealient.datasource.v1.models.GetRecipeSummaryResponseV1
 
 interface MealieDataSourceV1 {
 
@@ -31,7 +31,7 @@ interface MealieDataSourceV1 {
         token: String?,
         start: Int,
         limit: Int,
-    ): List<GetRecipeSummaryResponse>
+    ): List<GetRecipeSummaryResponseV1>
 
     suspend fun requestRecipeInfo(
         baseUrl: String,

datasource/src/main/kotlin/gq/kirmanak/mealient/datasource/v1/MealieDataSourceV1Impl.kt

@@ -0,0 +1,90 @@
+package gq.kirmanak.mealient.datasource.v1
+
+import gq.kirmanak.mealient.datasource.models.AddRecipeRequest
+import gq.kirmanak.mealient.datasource.models.GetRecipeResponse
+import gq.kirmanak.mealient.datasource.models.NetworkError
+import gq.kirmanak.mealient.datasource.models.VersionResponse
+import gq.kirmanak.mealient.datasource.v1.models.GetRecipeSummaryResponseV1
+import gq.kirmanak.mealient.logging.Logger
+import kotlinx.serialization.SerializationException
+import kotlinx.serialization.json.Json
+import retrofit2.HttpException
+import java.net.ConnectException
+import java.net.SocketTimeoutException
+import javax.inject.Inject
+import javax.inject.Singleton
+
+@Singleton
+class MealieDataSourceV1Impl @Inject constructor(
+    private val logger: Logger,
+    private val mealieService: MealieServiceV1,
+    private val json: Json,
+) : MealieDataSourceV1 {
+
+    override suspend fun addRecipe(
+        baseUrl: String,
+        token: String?,
+        recipe: AddRecipeRequest
+    ): String {
+        TODO("Not yet implemented")
+    }
+
+    override suspend fun authenticate(baseUrl: String, username: String, password: String): String {
+        TODO("Not yet implemented")
+    }
+
+    override suspend fun getVersionInfo(baseUrl: String): VersionResponse = makeCall(
+        block = { getVersion("$baseUrl/api/app/about") },
+        logMethod = { "getVersionInfo" },
+        logParameters = { "baseUrl = $baseUrl" },
+    ).getOrElse {
+        throw when (it) {
+            is HttpException, is SerializationException -> NetworkError.NotMealie(it)
+            is SocketTimeoutException, is ConnectException -> NetworkError.NoServerConnection(it)
+            else -> NetworkError.MalformedUrl(it)
+        }
+    }
+
+    override suspend fun requestRecipes(
+        baseUrl: String,
+        token: String?,
+        start: Int,
+        limit: Int
+    ): List<GetRecipeSummaryResponseV1> {
+        // Imagine start is 30 and limit is 15. It means that we already have page 1 and 2, now we need page 3
+        val perPage = limit
+        val page = start / perPage + 1
+        return makeCall(
+            block = { getRecipeSummary("$baseUrl/api/recipes", token, page, perPage) },
+            logMethod = { "requestRecipesV1" },
+            logParameters = { "baseUrl = $baseUrl, token = $token, start = $start, limit = $limit" }
+        ).map { it.items }.getOrThrowUnauthorized()
+    }
+
+    override suspend fun requestRecipeInfo(
+        baseUrl: String,
+        token: String?,
+        slug: String
+    ): GetRecipeResponse {
+        TODO("Not yet implemented")
+    }
+
+    private suspend inline fun <T> makeCall(
+        crossinline block: suspend MealieServiceV1.() -> T,
+        crossinline logMethod: () -> String,
+        crossinline logParameters: () -> String,
+    ): Result<T> {
+        logger.v { "${logMethod()} called with: ${logParameters()}" }
+        return mealieService.runCatching { block() }
+            .onFailure { logger.e(it) { "${logMethod()} request failed with: ${logParameters()}" } }
+            .onSuccess { logger.d { "${logMethod()} request succeeded with ${logParameters()}" } }
+    }
+}
+
+private fun <T> Result<T>.getOrThrowUnauthorized(): T = getOrElse {
+    throw if (it is HttpException && it.code() in listOf(401, 403)) {
+        NetworkError.Unauthorized(it)
+    } else {
+        it
+    }
+}

datasource/src/main/kotlin/gq/kirmanak/mealient/datasource/v1/MealieServiceV1.kt

@@ -1,7 +1,8 @@
-package gq.kirmanak.mealient.datasource
+package gq.kirmanak.mealient.datasource.v1
 
 import gq.kirmanak.mealient.datasource.DataSourceModule.Companion.AUTHORIZATION_HEADER_NAME
 import gq.kirmanak.mealient.datasource.models.*
+import gq.kirmanak.mealient.datasource.v1.models.GetRecipesResponseV1
 import retrofit2.http.*
 
 interface MealieServiceV1 {
@@ -28,14 +29,6 @@ interface MealieServiceV1 {
 
     @GET
     suspend fun getRecipeSummary(
-        @Url url: String,
-        @Header(AUTHORIZATION_HEADER_NAME) token: String?,
-        @Query("start") start: Int,
-        @Query("limit") limit: Int,
-    ): List<GetRecipeSummaryResponse>
-
-    @GET
-    suspend fun getRecipeSummaryV1(
         @Url url: String,
         @Header(AUTHORIZATION_HEADER_NAME) token: String?,
         @Query("page") page: Int,

datasource/src/main/kotlin/gq/kirmanak/mealient/datasource/v1/models/GetRecipeSummaryResponseV1.kt

@@ -0,0 +1,24 @@
+package gq.kirmanak.mealient.datasource.v1.models
+
+import kotlinx.datetime.LocalDate
+import kotlinx.datetime.LocalDateTime
+import kotlinx.serialization.SerialName
+import kotlinx.serialization.Serializable
+
+@Serializable
+data class GetRecipeSummaryResponseV1(
+    @SerialName("id") val remoteId: String,
+    @SerialName("name") val name: String,
+    @SerialName("slug") val slug: String,
+    @SerialName("image") val image: String?,
+    @SerialName("description") val description: String = "",
+    @SerialName("recipeCategory") val recipeCategories: List<String>,
+    @SerialName("tags") val tags: List<String>,
+    @SerialName("rating") val rating: Int?,
+    @SerialName("dateAdded") val dateAdded: LocalDate,
+    @SerialName("dateUpdated") val dateUpdated: LocalDateTime
+) {
+    override fun toString(): String {
+        return "GetRecipeSummaryResponseV1(remoteId=$remoteId, name='$name')"
+    }
+}

datasource/src/main/kotlin/gq/kirmanak/mealient/datasource/v1/models/GetRecipesResponseV1.kt

@@ -1,9 +1,9 @@
-package gq.kirmanak.mealient.datasource.models
+package gq.kirmanak.mealient.datasource.v1.models
 
 import kotlinx.serialization.SerialName
 import kotlinx.serialization.Serializable
 
 @Serializable
 data class GetRecipesResponseV1(
-    @SerialName("items") val items: List<GetRecipeSummaryResponse>,
+    @SerialName("items") val items: List<GetRecipeSummaryResponseV1>,
 )