Valentin271 / DeezerKodi

Deezer client for Kodi
GNU General Public License v3.0
24 stars 4 forks source link

Do not reverse tracks in playlists #18

Closed psiroky closed 2 years ago

psiroky commented 2 years ago

The reverse() looks intentional, but I could not think of a reason for that - maybe the API previously returned the tracks in wrong order?

In any case, I think the order of the tracks in the playlist should be the same as shown e.g. with Deezer apps, so the reverse should be removed.

Valentin271 commented 2 years ago

Hello,

You're right about the reverse, it is intentional. The Deezer API gives the tracks last added in last (which makes sens written like that) but I like to have my recently added songs in first.

Deezer apps allows to sort by :

I agree that the default order should be the default one from Deezer.

However I'll close this merge request as :

  1. It merges on master and I have implemented a better structure on next (I can't just edit the target branch as there is almost nothing in common)
  2. I think this should be an option to satisfy everyone