amperfy icon indicating copy to clipboard operation
amperfy copied to clipboard

[BUG] Show recently added - wrong sort order

Open bbsixzz opened this issue 3 years ago • 6 comments

iPhone SE 2016 running iOS 15.2.1, Amperfy 1.0.8 (4)

The sort order does not correspond to the (correct) recently added sort order in Navidrome:

Screenshot 2022-01-25 at 01 27 06

IMG_603243716EC1-1

bbsixzz avatar Jan 25 '22 00:01 bbsixzz

Addendum iSub has the correct order:

IMG_12C09649C182-1

bbsixzz avatar Jan 26 '22 16:01 bbsixzz

I might be misunderstood but I don't believe Amperfy has a recently added section, as you can see in your example albums are sorted alphabetically. There's not that much need for it either way, assuming you're running an Ampache server that is, where you can simply create a smart playlist with recently added songs. image

P.s. some nice music you've got there :) I particularly like Vril and Mumdance :)

sandersantema avatar Feb 09 '22 13:02 sandersantema

I actually just saw the recently added menu item in the screenshots in the readme, which I suppose is the feature you're referring to. Weirdly enough I haven't got that menu :thinking:

sandersantema avatar Feb 09 '22 14:02 sandersantema

Nice, we have similar taste in music :]

I use Navidrome as my server, maybe that's why I have the menu?

713E8B6C-9D33-499D-9199-BE824581A2AD

bbsixzz avatar Feb 09 '22 15:02 bbsixzz

You are right Amperfy currently does not sort via creation date. This has a simple reason. Ampache does not have the creation date in its xml responses. Since I supported both APIs this would be quite an afford to sort for one api that way and for the api in an other way. My table views were not flexible enough during the time I implemented this feature. I will take a look at it again.

BLeeEZ avatar Feb 09 '22 18:02 BLeeEZ

1.1.0 (1) still wrong sort order just in case you changed anything and need feedback.

bbsixzz avatar Apr 09 '22 03:04 bbsixzz

Hello, are you working on a solution to this?

I still open iSub to look for the right order in order to play my albums in Amperfy.

bbsixzz avatar Sep 08 '22 23:09 bbsixzz

I am currently refactoring the complete api handling. After that I will take care of this. In the meantime I did a lot of changes to the table view display system which should now support this feature better. Expect this bug be fixed in the upcoming release.

BLeeEZ avatar Sep 09 '22 05:09 BLeeEZ

Sorting is now fixed in d09da0db50dbabcba65f151d33f776c704d7d822.

BLeeEZ avatar Sep 12 '22 14:09 BLeeEZ

This should be fixed in the latest beta version 1.1.2 (1): https://github.com/BLeeEZ/amperfy/issues/25. Can you give me some feedback?

BLeeEZ avatar Oct 19 '22 05:10 BLeeEZ

This is great now, thank you!

bbsixzz avatar Oct 19 '22 16:10 bbsixzz

This issue has regressed with the new sort options in Amperfy 1.1.2 (2).

It is no longer possible to sort by time added in Recent Albums, Recent Songs is unaffected.

IMG_AA4669ACF292-1

IMG_D348D521B32D-1

bbsixzz avatar Dec 08 '22 20:12 bbsixzz

You are right. This error came in with the revert to tableview. I will remove the sort option in „recent albums“ and default it to time added.

BLeeEZ avatar Dec 08 '22 21:12 BLeeEZ

Fixed in 99956c84adb5bb4b581e91ab15ab93a2fe0f6c64

BLeeEZ avatar Dec 10 '22 19:12 BLeeEZ