Merge pull request #36 from Omega-R/image-loading-callback

added image loading callback

Author: anton-knyazev

app/src/main/java/omega_r/com/omegatypesexample/MainActivity.kt

@@ -5,6 +5,7 @@ import android.graphics.Bitmap
 import android.graphics.BitmapFactory
 import android.graphics.drawable.BitmapDrawable
 import android.os.Bundle
+import android.util.Log
 import android.widget.ImageView
 import android.widget.TextView
 import androidx.core.content.res.ResourcesCompat
@@ -75,7 +76,9 @@ class MainActivity : BaseActivity() {
 //            image
 //        }
 
-        imageView.setImage(image)
+        imageView.setImage(image, processor = GlideImagesProcessor(ImageProcessors.current), onImageApplied = {
+            Log.d("TAG","onImageApplied")
+        })
 
 
 

glide/src/main/java/com/omega_r/libs/omegatypes/image/GlideImagesProcessor.kt

@@ -3,11 +3,16 @@ package com.omega_r.libs.omegatypes.image
 import android.content.Context
 import android.graphics.Bitmap
 import android.graphics.drawable.Drawable
+import android.util.Log
 import android.view.View
 import android.widget.ImageView
 import com.bumptech.glide.Glide
 import com.bumptech.glide.RequestBuilder
+import com.bumptech.glide.load.DataSource
+import com.bumptech.glide.load.engine.GlideException
+import com.bumptech.glide.request.RequestListener
 import com.bumptech.glide.request.target.CustomViewTarget
+import com.bumptech.glide.request.target.Target
 import com.bumptech.glide.request.transition.Transition
 import com.omega_r.libs.omegatypes.decoders.BitmapDecoders
 import com.omega_r.libs.omegatypes.decoders.SimpleBitmapDecoders
@@ -53,11 +58,12 @@ open class GlideImagesProcessor(
         }
     }
 
-    override fun Image.applyImage(imageView: ImageView, placeholderResId: Int) {
+    override fun Image.applyImage(imageView: ImageView, placeholderResId: Int, onImageApplied: (() -> Unit)?) {
         Glide.with(imageView)
                 .asDrawable()
                 .createRequestBuilder(this)
                 ?.applyPlaceholder(placeholderResId)
+                ?.listener(GlideImageRequestListener(onImageApplied))
                 ?.into(imageView)
                 ?: applyOld { applyImage(imageView, placeholderResId) }
     }
@@ -117,4 +123,38 @@ open class GlideImagesProcessor(
     protected inline fun <R> applyOld(block: ImageProcessors.() -> R): R {
         return block(oldImagesProcessor)
     }
+}
+
+class GlideImageRequestListener(private val onImageLoaded: (() -> Unit)?) : RequestListener<Drawable> {
+
+    companion object {
+        private val TAG = GlideImageRequestListener::class.java.name
+    }
+
+    override fun onLoadFailed(
+            e: GlideException?,
+            model: Any?,
+            target: Target<Drawable>?,
+            isFirstResource: Boolean
+    ): Boolean {
+        Log.e(TAG, "Image load failed: ", e)
+        if (isFirstResource) {
+            onImageLoaded?.invoke()
+        }
+        return false
+    }
+
+    override fun onResourceReady(
+            resource: Drawable,
+            model: Any?,
+            target: Target<Drawable>,
+            dataSource: DataSource,
+            isFirstResource: Boolean
+    ): Boolean {
+        target.onResourceReady(resource, null)
+        if (isFirstResource) {
+            onImageLoaded?.invoke()
+        }
+        return true
+    }
 }

omegatypes/src/main/java/com/omega_r/libs/omegatypes/image/Image.kt

@@ -138,16 +138,22 @@ suspend fun Image.getStream(
 }
 
 @JvmOverloads
-fun ImageView.setImage(image: Image?, placeholderResId: Int = Image.NO_PLACEHOLDER_RES, processor: ImageProcessors = ImageProcessors.current) {
+fun ImageView.setImage(
+        image: Image?,
+        placeholderResId: Int = Image.NO_PLACEHOLDER_RES,
+        processor: ImageProcessors = ImageProcessors.current,
+        onImageApplied: (() -> Unit)? = null
+) {
     with(processor) {
         if (image != null) {
-            image.applyImage(this@setImage, placeholderResId)
+            image.applyImage(this@setImage, placeholderResId, onImageApplied)
         } else {
             if (placeholderResId == 0) {
                 setImageDrawable(null)
             } else {
                 setImageResource(placeholderResId)
             }
+            onImageApplied?.invoke()
         }
     }
 }

omegatypes/src/main/java/com/omega_r/libs/omegatypes/image/ImageProcessors.kt

@@ -27,7 +27,7 @@ abstract class ImageProcessors : CoroutineScope {
     override val coroutineContext: CoroutineContext
         get() = Dispatchers.Default
 
-    abstract fun Image.applyImage(imageView: ImageView, placeholderResId: Int)
+    abstract fun Image.applyImage(imageView: ImageView, placeholderResId: Int, onImageApplied: (() -> Unit)? = null)
 
     abstract fun Image.applyBackground(view: View, placeholderResId: Int)
 
@@ -43,8 +43,9 @@ abstract class ImageProcessors : CoroutineScope {
             map[imageClass] = imageProcessor
         }
 
-        override fun Image.applyImage(imageView: ImageView, placeholderResId: Int) = with(getImageProcessor()) {
+        override fun Image.applyImage(imageView: ImageView, placeholderResId: Int, onImageApplied: (() -> Unit)?): Unit = with(getImageProcessor()) {
             applyImage(imageView, placeholderResId)
+            onImageApplied?.invoke()
         }
 
         override fun Image.applyBackground(view: View, placeholderResId: Int) = with(getImageProcessor()) {

picasso/src/main/java/com/omega_r/libs/omegatypes/image/PicassoImagesProcessor.kt

@@ -55,7 +55,7 @@ class PicassoImagesProcessor(
         }
     }
 
-    override fun Image.applyImage(imageView: ImageView, placeholderResId: Int) {
+    override fun Image.applyImage(imageView: ImageView, placeholderResId: Int, onImageApplied: (() -> Unit)?) {
         createRequestCreator()?.apply {
             if (placeholderResId != NO_PLACEHOLDER_RES) placeholder(placeholderResId)
             fit()
@@ -69,6 +69,7 @@ class PicassoImagesProcessor(
         } ?: with(oldImagesProcessor) {
             applyImage(imageView, placeholderResId)
         }
+        onImageApplied?.invoke()
     }
 
     override fun Image.applyBackground(view: View, placeholderResId: Int) {