Sort albums by band instead of album

[MightyCreak]

IMO we remember the bands names more than their albums, so first we find for the band, then for one of their albums. This is also how Deezer sorts the user's "albums" page.

I made a test end I find it way more useful than sorting by album:
https://github.com/MightyCreak/sonerezh/commit/88d89752d43ae75f33b3b7afabaed59332e4ba7c

If one reads this, I encourage him to edit his files locally and try it, it's really better.

Even better would be to have the choice, but the changes would be way more intrusive.

[MightyCreak]

Around the same line you can also increase the amount of albums in the first page (I personally increased it to 60):

-                     'limit' => 36
+                     'limit' => 60

And also you can remove the latest albums line:

-             if ($page == 1) {
-                 $latests = $this->Song->find('all', array(
-                     'fields' => array('Song.id', 'Song.band', 'Song.album', 'Song.cover'),
-                     'conditions' => $subQuery,
-                     'order' => 'Song.created DESC',
-                     'limit' => 6
-                 ));
-             }

[ltguillaume]

Nice catch. I think you need to change the same thing on line 166 and line 197 to support all databases. Furthermore, it's even nicer to use 'order' => ['Song.band', 'Song.album'],, so that it's sorted by artist name first, then by album name.

[MightyCreak]

Thanks! I'll do that! ;)

Any news on the next release?

[ltguillaume]

Looks like you were asking me, but although I sort of have the same username as @lGuillaume124, I'm not him ;) So no idea when there's gonna be an update :P

[MightyCreak]

Indeed, I got confused, sorry ;)

[lGuillaume124]

I propose to add two labels to sort the page "by album", or "by band name".

[MightyCreak]

What's the default sort? (I'd vote for album).
Is the choice saved in the users preferences?

[lGuillaume124]

I can store your choice in the cookie.