diff --git a/app/src/main/java/audio/funkwhale/ffa/adapters/FavoritesAdapter.kt b/app/src/main/java/audio/funkwhale/ffa/adapters/FavoritesAdapter.kt index ac9ceda2bcba42827cf5aa45204c2d774ca16c8d..2a840242ff79cc3ec3d6d60c1887d7575716df7b 100644 --- a/app/src/main/java/audio/funkwhale/ffa/adapters/FavoritesAdapter.kt +++ b/app/src/main/java/audio/funkwhale/ffa/adapters/FavoritesAdapter.kt @@ -106,7 +106,7 @@ class FavoritesAdapter( favoriteListener.onToggleFavorite(favorite.id, !favorite.favorite) data.remove(favorite) - notifyItemRemoved(holder.adapterPosition) + notifyItemRemoved(holder.bindingAdapterPosition) } } diff --git a/app/src/main/java/audio/funkwhale/ffa/adapters/PlaylistTracksAdapter.kt b/app/src/main/java/audio/funkwhale/ffa/adapters/PlaylistTracksAdapter.kt index a51f502372f17477ffa6308a34fc8e1817ac62b7..734e85056e76b79929af163215f0a90117c47d99 100644 --- a/app/src/main/java/audio/funkwhale/ffa/adapters/PlaylistTracksAdapter.kt +++ b/app/src/main/java/audio/funkwhale/ffa/adapters/PlaylistTracksAdapter.kt @@ -199,10 +199,10 @@ class PlaylistTracksAdapter( viewHolder: RecyclerView.ViewHolder, target: RecyclerView.ViewHolder ): Boolean { - if (from == -1) from = viewHolder.adapterPosition - to = target.adapterPosition + if (from == -1) from = viewHolder.bindingAdapterPosition + to = target.bindingAdapterPosition - onItemMove(viewHolder.adapterPosition, target.adapterPosition) + onItemMove(viewHolder.bindingAdapterPosition, target.bindingAdapterPosition) return true } diff --git a/app/src/main/java/audio/funkwhale/ffa/adapters/TracksAdapter.kt b/app/src/main/java/audio/funkwhale/ffa/adapters/TracksAdapter.kt index 381b05f51964198ab2fbd3b0f815667ab30f297d..c661460a569b591cf01791996c0f3d19137d0d47 100644 --- a/app/src/main/java/audio/funkwhale/ffa/adapters/TracksAdapter.kt +++ b/app/src/main/java/audio/funkwhale/ffa/adapters/TracksAdapter.kt @@ -230,7 +230,7 @@ class TracksAdapter( if (actionState == ItemTouchHelper.ACTION_STATE_DRAG) { context?.let { viewHolder?.let { - from = viewHolder.adapterPosition + from = viewHolder.bindingAdapterPosition viewHolder.itemView.background = ColorDrawable(context.getColor(R.color.colorSelected)) } } diff --git a/app/src/main/java/audio/funkwhale/ffa/fragments/PlaylistTracksFragment.kt b/app/src/main/java/audio/funkwhale/ffa/fragments/PlaylistTracksFragment.kt index da40ee22091f41f30e00874adfb8335c6916e1ad..8255902cea74e38817c1854f4b5ba62ff1b4b1fb 100644 --- a/app/src/main/java/audio/funkwhale/ffa/fragments/PlaylistTracksFragment.kt +++ b/app/src/main/java/audio/funkwhale/ffa/fragments/PlaylistTracksFragment.kt @@ -12,21 +12,13 @@ import androidx.recyclerview.widget.RecyclerView import audio.funkwhale.ffa.R import audio.funkwhale.ffa.adapters.PlaylistTracksAdapter import audio.funkwhale.ffa.databinding.FragmentTracksBinding -import audio.funkwhale.ffa.repositories.FavoritesRepository -import audio.funkwhale.ffa.repositories.ManagementPlaylistsRepository -import audio.funkwhale.ffa.repositories.PlaylistTracksRepository -import audio.funkwhale.ffa.utils.Command -import audio.funkwhale.ffa.utils.CommandBus import audio.funkwhale.ffa.model.Playlist import audio.funkwhale.ffa.model.PlaylistTrack -import audio.funkwhale.ffa.utils.Request -import audio.funkwhale.ffa.utils.RequestBus -import audio.funkwhale.ffa.utils.Response import audio.funkwhale.ffa.model.Track -import audio.funkwhale.ffa.utils.maybeLoad -import audio.funkwhale.ffa.utils.maybeNormalizeUrl -import audio.funkwhale.ffa.utils.toast -import audio.funkwhale.ffa.utils.wait +import audio.funkwhale.ffa.repositories.FavoritesRepository +import audio.funkwhale.ffa.repositories.ManagementPlaylistsRepository +import audio.funkwhale.ffa.repositories.PlaylistTracksRepository +import audio.funkwhale.ffa.utils.* import com.squareup.picasso.Picasso import jp.wasabeef.picasso.transformations.RoundedCornersTransformation import kotlinx.coroutines.Dispatchers.Main @@ -183,15 +175,14 @@ class PlaylistTracksFragment : FFAFragment<PlaylistTrack, PlaylistTracksAdapter> else -> RoundedCornersTransformation.CornerType.TOP_LEFT } - imageView?.let { view -> - lifecycleScope.launch(Main) { - Picasso.get() - .maybeLoad(maybeNormalizeUrl(url)) - .fit() - .centerCrop() - .transform(RoundedCornersTransformation(16, 0, corner)) - .into(view) - } + + lifecycleScope.launch(Main) { + Picasso.get() + .maybeLoad(maybeNormalizeUrl(url)) + .fit() + .centerCrop() + .transform(RoundedCornersTransformation(16, 0, corner)) + .into(imageView) } } } @@ -229,7 +220,7 @@ class PlaylistTracksFragment : FFAFragment<PlaylistTrack, PlaylistTracksAdapter> override fun onRemoveTrackFromPlaylist(track: Track, index: Int) { lifecycleScope.launch(Main) { - playlistsRepository.remove(albumId, track, index) + playlistsRepository.remove(albumId, index) update() } } diff --git a/app/src/main/java/audio/funkwhale/ffa/fragments/QueueFragment.kt b/app/src/main/java/audio/funkwhale/ffa/fragments/QueueFragment.kt index 5fcf876b5be4c5a6081bc031f3a1d38e70884a28..02e1b2554fb2acc099f0db3dc11bd823f9fc90f5 100644 --- a/app/src/main/java/audio/funkwhale/ffa/fragments/QueueFragment.kt +++ b/app/src/main/java/audio/funkwhale/ffa/fragments/QueueFragment.kt @@ -78,7 +78,7 @@ class QueueFragment : BottomSheetDialogFragment() { super.onResume() binding.included.queue.visibility = View.GONE - binding.included.placeholder?.visibility = View.VISIBLE + binding.included.placeholder.visibility = View.VISIBLE binding.included.queueShuffle.setOnClickListener { CommandBus.send(Command.ShuffleQueue) diff --git a/app/src/main/java/audio/funkwhale/ffa/repositories/PlaylistsRepository.kt b/app/src/main/java/audio/funkwhale/ffa/repositories/PlaylistsRepository.kt index 951f293625d7d74cce4d0302c29b7d7946c1d694..752d60f97643ba48c414ffb34ef8c1f60144a44c 100644 --- a/app/src/main/java/audio/funkwhale/ffa/repositories/PlaylistsRepository.kt +++ b/app/src/main/java/audio/funkwhale/ffa/repositories/PlaylistsRepository.kt @@ -103,11 +103,11 @@ class ManagementPlaylistsRepository(override val context: Context?) : } } - suspend fun remove(id: Int, track: Track, index: Int) { + suspend fun remove(albumId: Int, index: Int) { context?.let { val body = mapOf("index" to index) - val request = Fuel.post(mustNormalizeUrl("/api/v1/playlists/$id/remove/")).apply { + val request = Fuel.post(mustNormalizeUrl("/api/v1/playlists/$albumId/remove/")).apply { if (!Settings.isAnonymous()) { authorize(context, oAuth) header("Authorization", "Bearer ${oAuth.state().accessToken}") diff --git a/app/src/main/java/audio/funkwhale/ffa/utils/Util.kt b/app/src/main/java/audio/funkwhale/ffa/utils/Util.kt index 687b536137dd4806db37c254426b1a002680a5b1..9769e71f50af71da2ccefe884da92c58203bcc5a 100644 --- a/app/src/main/java/audio/funkwhale/ffa/utils/Util.kt +++ b/app/src/main/java/audio/funkwhale/ffa/utils/Util.kt @@ -3,8 +3,6 @@ package audio.funkwhale.ffa.utils import android.content.Context import android.widget.Toast import com.google.android.exoplayer2.util.Log -import com.google.android.exoplayer2.util.Log.LOG_LEVEL_ERROR -import com.google.android.exoplayer2.util.Log.LOG_LEVEL_INFO import com.preference.PowerPreference import java.net.URI @@ -33,10 +31,10 @@ private fun logClassName(): String { return "UNKNOWN" } -enum class LogLevel(value: Int) { - INFO(LOG_LEVEL_INFO), - DEBUG(Log.LOG_LEVEL_ALL), - ERROR(LOG_LEVEL_ERROR) +enum class LogLevel { + INFO, + DEBUG, + ERROR } fun Any?.logError(prefix: String? = null) = this.log(prefix, LogLevel.ERROR)