From 0810816c65289baef5debf96d576a8be5533a40d Mon Sep 17 00:00:00 2001 From: Kevin Boulongne Date: Wed, 31 Jan 2024 16:04:14 +0100 Subject: [PATCH] Remove unused `Disposable` return value --- .../java/com/infomaniak/lib/core/utils/Extensions.kt | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/src/main/java/com/infomaniak/lib/core/utils/Extensions.kt b/src/main/java/com/infomaniak/lib/core/utils/Extensions.kt index 32dbe754..6a0c404f 100644 --- a/src/main/java/com/infomaniak/lib/core/utils/Extensions.kt +++ b/src/main/java/com/infomaniak/lib/core/utils/Extensions.kt @@ -66,7 +66,6 @@ import androidx.recyclerview.widget.RecyclerView import androidx.viewbinding.ViewBinding import coil.ImageLoader import coil.load -import coil.request.Disposable import com.github.razir.progressbutton.* import com.github.razir.progressbutton.DrawableButton.Companion.GRAVITY_CENTER import com.github.razir.progressbutton.hideProgress @@ -314,7 +313,7 @@ fun SharedPreferences.transaction(block: SharedPreferences.Editor.() -> Unit) { fun ImageView.loadAvatar( user: User, imageLoader: ImageLoader = context.simpleImageLoader, -): Disposable = loadAvatar(user.id, user.avatar, user.getInitials(), imageLoader) +) = loadAvatar(user.id, user.avatar, user.getInitials(), imageLoader) fun ImageView.loadAvatar( id: Int, @@ -322,9 +321,9 @@ fun ImageView.loadAvatar( initials: String, imageLoader: ImageLoader = context.simpleImageLoader, @ColorInt initialsColor: Int = Color.WHITE, -): Disposable { +) { val backgroundColor = context.getBackgroundColorBasedOnId(id) - return loadAvatar(backgroundColor, avatarUrl, initials, imageLoader, initialsColor) + loadAvatar(backgroundColor, avatarUrl, initials, imageLoader, initialsColor) } fun ImageView.loadAvatar( @@ -333,13 +332,13 @@ fun ImageView.loadAvatar( initials: String, imageLoader: ImageLoader = context.simpleImageLoader, @ColorInt initialsColor: Int = Color.WHITE, -): Disposable { +) { val fallback = context.generateInitialsAvatarDrawable( initials = initials, background = backgroundColor, initialsColor = initialsColor, ) - return load(avatarUrl, imageLoader) { + load(avatarUrl, imageLoader) { error(fallback) fallback(fallback) placeholder(fallback)