search

abarisain / dmix

A modern MPD Client for Android.
Apache License 2.0
589 stars 205 forks source link

Sort by Newest Albums #671

Open hurzl opened 9 years ago

hurzl commented 9 years ago

AbstractMusic should parse Last-Modified as returned by MPD via listallinfo. Then albums could be sorted by last added.

abarisain commented 9 years ago

It’s not as simple as that, it is ?

1/ it needs the cache

2/ We need to keep track of when an album appeared by last-modified so we can show the last added ones.

Or is Last-Modified per album, and not per "listallinfo" command ?

On 27 déc. 2014, at 00:22, hurzl notifications@github.com wrote:

Abstract Music should parse Last-Modified as returned by MPD via listallinfo. Then albums could be sorted by last added.

— Reply to this email directly or view it on GitHub https://github.com/abarisain/dmix/issues/671.

burmar commented 9 years ago

+1 Simply the list of last added to database... it would be very nice

hurzl commented 9 years ago
  1. yes speaking of cache.
  2. album last-modified is last-modified of last track modified ;-) and can be cached.
  3. Either overall setting to always sort albums by last mod. (easier) or option in the AlbumsFragment
abarisain commented 9 years ago

Okay, I understand, and it would be great to to that (or even have a home with "Recently added") , but I don't really want to change the UI for something that works only with the cache.

Is there no way to get that information with the traditional calls, rather than listallinfo ?

hurzl commented 9 years ago

Is there no way to get that information with the traditional calls, rather than listallinfo ?

probably by doing listallinfo on the directory of the album? (all the albums!)

kingosticks commented 9 years ago

Can I stick an oar in here and say listallinfo on directories is unpleasant in Mopidy as the directory structure may not be real and interrogating parts of it may not give you any useful data, or any data at all. There is also a discussion to remove support for listallinfo support entirely from Mopidy (for the same reasons). So using traditional calls would be nice, at least as a fallback perhaps. On 27 Dec 2014 11:46, "hurzl" notifications@github.com wrote:

Is there no way to get that information with the traditional calls, rather than listallinfo ?

probably by doing listallinfo on the directory of the album?

— Reply to this email directly or view it on GitHub https://github.com/abarisain/dmix/issues/671#issuecomment-68176256.

hurzl commented 9 years ago

It's not possible as far as I can see. (If Mopidy wants to mimick MPD it should support the command)

hurzl commented 9 years ago

Somehow use stickers for it? But one has to get the information first anyway. Will it be faster afterwards?

abarisain commented 9 years ago

With what I have in mind (I'd like to show a list of recently added albums), listallinfo on album directories won't work anyway

kingosticks commented 9 years ago

For the record, it doesn't want to mimic MPD. It wants to support the MPD protocol where it makes sense to do so for online catalogues :) On 27 Dec 2014 13:16, "Arnaud Barisain Monrose" notifications@github.com wrote:

With what I have in mind (I'd like to show a list of recently added albums), listallinfo on album directories won't work anyway

— Reply to this email directly or view it on GitHub https://github.com/abarisain/dmix/issues/671#issuecomment-68178269.

hurzl commented 9 years ago

you are both right But there is no other way to get the mod time of files. And there is no other way to sort by last added. :( Or you want to do something like the local covers mechanism with the http server.

hurzl commented 9 years ago

It now seems to work, but the ArrayIndexerAdapter should be disabled, how to? It should be disabled when sorting by album date, too, btw.

-> https://github.com/hurzl/dmix/commit/5ae7512b762f2075b0870f2d35f1384453f34ee5

bartkl commented 8 years ago

As I would love this feature, can somebody please give a status update on whether and when it might get implemented? Thanks!

abarisain commented 8 years ago

Hi, Mpdroid's development is pretty much on hiatus. That said, even if it wasn't, we wouldn't implement that due to the lack of server side support

Sorry about that :(

On Tue, May 17, 2016 at 4:24 AM -0700, "A.L. Kleijngeld" notifications@github.com wrote:

As I would love this feature, can somebody please give a status update on whether and when it might get implemented? Thanks!

— You are receiving this because you commented. Reply to this email directly or view it on GitHub

bartkl commented 8 years ago

That's a shame, thanks for the update and all previous hard work!

On Tue, May 17, 2016 at 10:25 PM, Arnaud Barisain-Monrose < notifications@github.com> wrote:

Hi, Mpdroid's development is pretty much on hiatus. That said, even if it wasn't, we wouldn't implement that due to the lack of server side support

Sorry about that :(

On Tue, May 17, 2016 at 4:24 AM -0700, "A.L. Kleijngeld" < notifications@github.com> wrote:

As I would love this feature, can somebody please give a status update on whether and when it might get implemented? Thanks!

— You are receiving this because you commented. Reply to this email directly or view it on GitHub

— You are receiving this because you commented. Reply to this email directly or view it on GitHub https://github.com/abarisain/dmix/issues/671#issuecomment-219842658

bretello commented 6 years ago

Sorry to revive this, but this would be a pretty slick feature, +1!