diff --git a/app/src/main/java/com/cappielloantonio/play/repository/PlaylistRepository.java b/app/src/main/java/com/cappielloantonio/play/repository/PlaylistRepository.java index 2d756232..e8254cb6 100644 --- a/app/src/main/java/com/cappielloantonio/play/repository/PlaylistRepository.java +++ b/app/src/main/java/com/cappielloantonio/play/repository/PlaylistRepository.java @@ -2,9 +2,11 @@ package com.cappielloantonio.play.repository; import android.app.Application; +import androidx.annotation.NonNull; import androidx.lifecycle.MutableLiveData; import com.cappielloantonio.play.App; +import com.cappielloantonio.play.model.Album; import com.cappielloantonio.play.model.Playlist; import com.cappielloantonio.play.model.Song; import com.cappielloantonio.play.subsonic.models.ResponseStatus; @@ -20,7 +22,7 @@ import retrofit2.Callback; import retrofit2.Response; public class PlaylistRepository { - private Application application; + private final Application application; public PlaylistRepository(Application application) { this.application = application; @@ -34,8 +36,8 @@ public class PlaylistRepository { .getPlaylists() .enqueue(new Callback() { @Override - public void onResponse(Call call, Response response) { - if (response.body().getStatus().getValue().equals(ResponseStatus.OK)) { + public void onResponse(@NonNull Call call, @NonNull Response response) { + if (response.isSuccessful() && response.body() != null && response.body().getPlaylists() != null) { List playlists = new ArrayList<>(MappingUtil.mapPlaylist(response.body().getPlaylists().getPlaylists())); if(random) { Collections.shuffle(playlists); @@ -48,7 +50,7 @@ public class PlaylistRepository { } @Override - public void onFailure(Call call, Throwable t) { + public void onFailure(@NonNull Call call, @NonNull Throwable t) { } }); @@ -63,15 +65,15 @@ public class PlaylistRepository { .getPlaylist(id) .enqueue(new Callback() { @Override - public void onResponse(Call call, Response response) { - if (response.body().getStatus().getValue().equals(ResponseStatus.OK)) { + public void onResponse(@NonNull Call call, @NonNull Response response) { + if (response.isSuccessful() && response.body() != null && response.body().getPlaylist() != null) { List songs = new ArrayList<>(MappingUtil.mapSong(response.body().getPlaylist().getEntries())); listLivePlaylistSongs.setValue(songs); } } @Override - public void onFailure(Call call, Throwable t) { + public void onFailure(@NonNull Call call, @NonNull Throwable t) { } }); @@ -84,14 +86,12 @@ public class PlaylistRepository { .updatePlaylist(playlistId, null, true, songsId, null) .enqueue(new Callback() { @Override - public void onResponse(Call call, Response response) { - if (response.body().getStatus().getValue().equals(ResponseStatus.OK)) { + public void onResponse(@NonNull Call call, @NonNull Response response) { - } } @Override - public void onFailure(Call call, Throwable t) { + public void onFailure(@NonNull Call call, @NonNull Throwable t) { } }); @@ -103,14 +103,12 @@ public class PlaylistRepository { .createPlaylist(playlistId, name, songsId) .enqueue(new Callback() { @Override - public void onResponse(Call call, Response response) { - if (response.body().getStatus().getValue().equals(ResponseStatus.OK)) { + public void onResponse(@NonNull Call call, @NonNull Response response) { - } } @Override - public void onFailure(Call call, Throwable t) { + public void onFailure(@NonNull Call call, @NonNull Throwable t) { } }); @@ -122,14 +120,12 @@ public class PlaylistRepository { .updatePlaylist(playlistId, name, isPublic, songIdToAdd, songIndexToRemove) .enqueue(new Callback() { @Override - public void onResponse(Call call, Response response) { - if (response.body().getStatus().getValue().equals(ResponseStatus.OK)) { + public void onResponse(@NonNull Call call, @NonNull Response response) { - } } @Override - public void onFailure(Call call, Throwable t) { + public void onFailure(@NonNull Call call, @NonNull Throwable t) { } }); @@ -141,14 +137,12 @@ public class PlaylistRepository { .deletePlaylist(playlistId) .enqueue(new Callback() { @Override - public void onResponse(Call call, Response response) { - if (response.body().getStatus().getValue().equals(ResponseStatus.OK)) { + public void onResponse(@NonNull Call call, @NonNull Response response) { - } } @Override - public void onFailure(Call call, Throwable t) { + public void onFailure(@NonNull Call call, @NonNull Throwable t) { } });