Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support "window" pagination with search and find commands #10

Open
piroxxi opened this issue Aug 10, 2015 · 7 comments
Open

Support "window" pagination with search and find commands #10

piroxxi opened this issue Aug 10, 2015 · 7 comments
Labels
C-enhancement Category: A PR with an enhancement or an issue with an enhancement proposal mpd-0.20 MPD 0.20 compatability

Comments

@piroxxi
Copy link

piroxxi commented Aug 10, 2015

Hi,

I'm currently creating an Android Application communicating with a Mopidy server using an HTTP socket. I'm currently trying to work with the "search" function, using the "window" parameter :
http://www.musicpd.org/doc/protocol/database.html

I've been trying multiple ways to write it, but none seems to be working. Here is a example :

nc 192.168.0.9 6600
search any "Test"
...... works ......
search any "Test" 4:5
ACK [2@0] {search} incorrect arguments
search any "Test" "4:5"
ACK [2@0] {search} incorrect arguments

Am-I doing it wrong ? Can anyone help me with that ?

Thank you,
Raphaël

@jodal
Copy link
Member

jodal commented Aug 10, 2015

From the docs, I think it should be:

search any "Test" window 4:5

Though, I've never seen this before, so I doubt we support it yet. The docs doesn't say anything about what it does. I assume it is for pagination of the search results?

@piroxxi
Copy link
Author

piroxxi commented Aug 10, 2015

Yes, it's a pagination argument :

search any "Test" window 4:5
ACK [2@0] {search} incorrect arguments

Is there any other way to limit the number of returned items when doing a search ? For some search the number of items may be really big...

@jodal
Copy link
Member

jodal commented Aug 10, 2015

No, not in our current MPD protocol implementation. Though, most services limit themselves. E.g. the Spotify backend returns at most 200 each of artists, albums and tracks.

@piroxxi
Copy link
Author

piroxxi commented Aug 10, 2015

Ok :)

Thanks for the quick reply ! Even tho it shouldn't be a big problem for me (I should be able to find another way), do you have any idea when such a functionality could be done ?

@jodal
Copy link
Member

jodal commented Aug 10, 2015

It's always a matter of someone voluntarily picking the issue up and fixing it instead of something else, so no. That said, the chance of it being fixed is way better now that we know of the missing feature :-)

@adamcik
Copy link
Member

adamcik commented Aug 10, 2015

http://git.musicpd.org/cgit/master/mpd.git/tree/NEWS lists this as a feature in the unreleased 0.20 version of MPD. So we are not lagging behind just yet :-)

@piroxxi
Copy link
Author

piroxxi commented Aug 10, 2015

Yeay ! That's great ! :)

Thanks again for the impressively quick answers ! :)

@jodal jodal changed the title Using the "window" parameter on find and seach methods. MPD 0.20 compat: Support "window" pagination with search and find commands Feb 16, 2016
@jodal jodal transferred this issue from mopidy/mopidy Dec 20, 2019
@jodal jodal added the mpd-0.20 MPD 0.20 compatability label Dec 20, 2019
@jodal jodal changed the title MPD 0.20 compat: Support "window" pagination with search and find commands Support "window" pagination with search and find commands Dec 20, 2019
@jodal jodal added the C-enhancement Category: A PR with an enhancement or an issue with an enhancement proposal label Dec 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-enhancement Category: A PR with an enhancement or an issue with an enhancement proposal mpd-0.20 MPD 0.20 compatability
Projects
None yet
Development

No branches or pull requests

3 participants