Some Problems with playlists about tempo HOT 6 CLOSED

Hey @dnno thanks for reporting!
I fixed the playlists issues, you can see the code and test if all the issues are now fixed.

from tempo.

This is the commit mentioned above 1eda7ce

from tempo.

There's a remaining NPE, I opened a PR, see #47

from tempo.

Yes, I fixed that problem as well.
I did it a priori, even before giving the adapter class the result of the server call:

playlistEditorViewModel.getPlaylistSongLiveList().observe(requireActivity(), songs -> {
    if (songs != null) {
        playlistDialogSongHorizontalAdapter.setItems(songs);
    }
});

In this way, inside the adapter, the list will never be null, at most empty.

from tempo.

Oh, I'm sorry, my fork sync missed that commit, it seems. bad timing...

from tempo.

No, don't apologize, it was me who pushed online waiting a few minutes between one commit and another

from tempo.