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

Unable to load MPD playlist #410

Open
ivesdebruycker opened this issue May 11, 2016 · 4 comments
Open

Unable to load MPD playlist #410

ivesdebruycker opened this issue May 11, 2016 · 4 comments

Comments

@ivesdebruycker
Copy link

MPD playlists (/var/lib/mpd/playlists/) are shown in the root of the music library, but when trying to play them, an error is thrown:

info: [1462956944404] ControllerMpd::sendMpdCommandArray
info: Error:
Error:  [50@1] {add} Not found
    at MpdClient.receive (/volumio/node_modules/mpd/index.js:57:17)
    at Socket.<anonymous> (/volumio/node_modules/mpd/index.js:37:12)
    at emitOne (events.js:90:13)
    at Socket.emit (events.js:182:7)
    at readableAddChunk (_stream_readable.js:147:16)
    at Socket.Readable.push (_stream_readable.js:111:10)
    at TCP.onread (net.js:523:20)

The cause is that ControllerMpd.addPlay is called with the name of the playlist (without extension), so command 'add' is sent instead of 'load' (/volumio/node_modules/mpd/index.js:145).

I don't know a reliable way of knowing if is in fact an MPD playlist, except maybe !fileName.startsWith('NAS/').

Perhaps these playlists should also better be located in 'Playlists' instead of 'Music library'?

@volumio
Copy link
Owner

volumio commented May 11, 2016

Since we're using our own playlist method, didn't think to integrate that... But we could add m3u\pls handler after RC2 release...

@ivesdebruycker
Copy link
Author

I was thinking about a plugin that would read m3u-files from a folder and converts them (periodically) to volumio2-playlists, but first i'll need to have a look at the plugin structure.

@volumio
Copy link
Owner

volumio commented May 11, 2016

Good idea... We're finalizing it, hopefully this weekend will be out

@xipmix
Copy link
Contributor

xipmix commented Oct 22, 2016

This may be fixed now, can you give 2.001 a try?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants