Replace picasso with coil

Author: bubelov

app/build.gradle.kts

@@ -102,7 +102,8 @@ dependencies {
 
     // UI
     implementation(libs.material)
-    implementation(libs.picasso)
+    implementation(libs.coil)
+    implementation(libs.coil.network)
 
     // Parsing
     implementation(libs.jsoup)

app/src/main/kotlin/di/Di.kt

@@ -68,6 +68,7 @@ object Di {
                 get(Db::class.java)
             )
             OpenGraphImagesRepo::class.java -> OpenGraphImagesRepo(
+                context,
                 get(ConfRepo::class.java),
                 get(Db::class.java)
             )

app/src/main/kotlin/entries/EntriesAdapterViewHolder.kt

@@ -5,8 +5,7 @@ import androidx.core.view.isVisible
 import androidx.recyclerview.widget.RecyclerView
 import co.appreactor.news.R
 import co.appreactor.news.databinding.ListItemEntryBinding
-import com.squareup.picasso.Callback
-import com.squareup.picasso.Picasso
+import coil3.load
 
 class EntriesAdapterViewHolder(
     private val binding: ListItemEntryBinding,
@@ -20,7 +19,7 @@ class EntriesAdapterViewHolder(
         val cardHeightMin = root.resources.getDimensionPixelSize(R.dimen.card_height_min)
         val cardHeightMax = root.resources.getDimensionPixelSize(R.dimen.card_height_max)
 
-        Picasso.get().load(null as String?).into(imageView)
+        imageView.load(null)
         imageView.isVisible = false
         imageProgress.isVisible = false
 
@@ -51,21 +50,18 @@ class EntriesAdapterViewHolder(
                 }
             }
 
-            Picasso.get()
-                .load(item.imageUrl)
-                .resize(item.imageWidth, 0)
-                .onlyScaleDown()
-                .into(imageView, object : Callback {
-                    override fun onSuccess() {
+            imageView.load(item.imageUrl) {
+                size(item.imageWidth, 0)
+                listener(
+                    onSuccess = { _, _ ->
                         imageProgress.isVisible = false
-                        imageProgress.isVisible = false
-                    }
-
-                    override fun onError(e: Exception) {
+                    },
+                    onError = { _, _ ->
                         imageView.isVisible = false
                         imageProgress.isVisible = false
                     }
-                })
+                )
+            }
         }
 
         primaryText.text = Html.fromHtml(item.title, Html.FROM_HTML_MODE_COMPACT).toString()

app/src/main/kotlin/entry/TextViewImage.kt

@@ -11,8 +11,8 @@ import android.os.Looper
 import android.text.SpannableStringBuilder
 import android.text.style.ImageSpan
 import android.widget.TextView
-import com.squareup.picasso.Picasso
-import com.squareup.picasso.Target
+import coil3.Image
+import coil3.target.Target
 
 class TextViewImage(
     private val textView: TextView,
@@ -34,7 +34,16 @@ class TextViewImage(
     )
     override fun getOpacity() = PixelFormat.OPAQUE
 
-    override fun onBitmapLoaded(unprocessedBitmap: Bitmap, from: Picasso.LoadedFrom) {
+    override fun onStart(result: Image?) {
+
+    }
+
+    override fun onError(result: Image?) {
+
+    }
+
+    override fun onSuccess(result: Image) {
+        val unprocessedBitmap = (result as BitmapDrawable).bitmap
         val scaleToFullWidth = unprocessedBitmap.width >= textView.width / 5
 
         val bitmap = if (scaleToFullWidth) {
@@ -92,12 +101,4 @@ class TextViewImage(
             textView.text = text
         }
     }
-
-    override fun onBitmapFailed(e: Exception?, errorDrawable: Drawable?) {
-
-    }
-
-    override fun onPrepareLoad(placeHolderDrawable: Drawable?) {
-
-    }
 }

app/src/main/kotlin/entry/TextViewImageGetter.kt

@@ -4,7 +4,8 @@ import android.graphics.drawable.Drawable
 import android.text.Html
 import android.widget.TextView
 import androidx.lifecycle.LifecycleCoroutineScope
-import com.squareup.picasso.Picasso
+import coil3.ImageLoader
+import coil3.request.ImageRequest
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.withContext
 import okhttp3.HttpUrl
@@ -41,11 +42,12 @@ class TextViewImageGetter(
         scope.launchWhenResumed {
             runCatching {
                 withContext(Dispatchers.Main) {
-                    Picasso.get()
-                        .load(url.toString())
-                        .resize(width, 0)
-                        .onlyScaleDown()
-                        .into(drawable)
+                    val request = ImageRequest.Builder(textView.context)
+                        .data(url.toString())
+                        .size(width, 0)
+                        .target(drawable)
+                        .build()
+                    ImageLoader(textView.context).enqueue(request)
                 }
             }
         }

app/src/main/kotlin/opengraph/OpenGraphImagesRepo.kt

@@ -1,10 +1,13 @@
 package opengraph
 
+import android.content.Context
 import android.graphics.Bitmap
 import android.graphics.Color
+import android.graphics.drawable.BitmapDrawable
 import android.util.Log
+import coil3.ImageLoader
+import coil3.request.ImageRequest
 import co.appreactor.feedk.AtomLinkRel
-import com.squareup.picasso.Picasso
 import conf.ConfRepo
 import db.Db
 import http.await
@@ -23,10 +26,13 @@ import java.util.concurrent.TimeUnit
 import kotlin.random.Random
 
 class OpenGraphImagesRepo(
+    private val context: Context,
     private val confRepo: ConfRepo,
     private val db: Db,
 ) {
 
+    private val imageLoader = ImageLoader(context)
+
     private val httpClient = OkHttpClient.Builder()
         .callTimeout(10, TimeUnit.SECONDS)
         .build()
@@ -97,7 +103,12 @@ class OpenGraphImagesRepo(
             }
 
             val bitmap = runCatching {
-                Picasso.get().load(imageUrl).resize(MAX_WIDTH_PX, 0).onlyScaleDown().get()
+                val request = ImageRequest.Builder(context)
+                    .data(imageUrl)
+                    .size(MAX_WIDTH_PX, 0)
+                    .build()
+                (imageLoader.execute(request).image as? BitmapDrawable)?.bitmap
+                    ?: throw IllegalStateException("Failed to load bitmap")
             }.getOrElse {
                 db.entryQueries.updateOgImageChecked(true, entry.id)
                 return@withContext

gradle/libs.versions.toml

@@ -36,8 +36,9 @@ retrofit = "3.0.0"
 # Networking
 # https://github.com/square/okhttp/blob/master/CHANGELOG.md
 okhttp = "5.3.2"
-
-picasso = "2.8"
+# Image manipulation
+# https://github.com/coil-kt/coil/blob/main/CHANGELOG.md
+coil = "3.4.0"
 
 # Parsing
 jsoup = "1.17.2"
@@ -85,7 +86,8 @@ okhttp-mockwebserver = { group = "com.squareup.okhttp3", name = "mockwebserver",
 
 # UI
 material = { group = "com.google.android.material", name = "material", version.ref = "material" }
-picasso = { group = "com.squareup.picasso", name = "picasso", version.ref = "picasso" }
+coil = { group = "io.coil-kt.coil3", name = "coil", version.ref = "coil" }
+coil-network = { group = "io.coil-kt.coil3", name = "coil-network-okhttp", version.ref = "coil" }
 
 # Parsing
 jsoup = { group = "org.jsoup", name = "jsoup", version.ref = "jsoup" }