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

Feature Request: API should return song/album paths #2177

Closed
velvet-jones opened this issue Aug 29, 2016 · 6 comments
Closed

Feature Request: API should return song/album paths #2177

velvet-jones opened this issue Aug 29, 2016 · 6 comments
Labels
needinfo We need more details or follow-up from the filer before this can be tagged "bug" or "feature."

Comments

@velvet-jones
Copy link

Problem

We're writing a utility to synchronize music libraries and we could really use the ability to get the path to an album. Both for the web API and for the export plugin.

Setup

  • OS: Ubuntu 14.04.3
  • Python version: 2.7.6
  • beets version: 1.3.18
  • Turning off plugins made problem go away (yes/no): N/A

My configuration (output of beet config) is: N/A

@claman
Copy link
Contributor

claman commented Aug 29, 2016

It's not perfect, but using the info command returns the path of the song in the first line

@sampsyo sampsyo added the needinfo We need more details or follow-up from the filer before this can be tagged "bug" or "feature." label Aug 29, 2016
@sampsyo
Copy link
Member

sampsyo commented Aug 29, 2016

Hi! I think someone recently added a path field to the web API, in #2050 (comment). Can you check that out?

@velvet-jones
Copy link
Author

Thanks, we checked it and sure enough, there's a path - for individual tracks. Surely beets knows the path to each entire album in its database? I've been searching the code to try to figure out if the album path is possible to add to '/album'. Any ideas?

@sampsyo
Copy link
Member

sampsyo commented Aug 30, 2016

There's an album.path field in beets, but it's actually just the dirname of the first track on the album. Just stripping the file name off of the item path should work.

@irskep
Copy link

irskep commented Jan 16, 2017

FYI I added paths to tracks in the web API in #2389. If that suffices, given @sampsyo's last comment, then this can probably be closed.

The history of this is a little confusing. #2050 added relative paths originally. Then that was reverted due to #2182. Finally, I added it back behind a config flag in #2389. So people who need it can have it, and people who don't won't see a slowdown.

@sampsyo
Copy link
Member

sampsyo commented Jan 17, 2017

Thanks for following up, @irskep.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needinfo We need more details or follow-up from the filer before this can be tagged "bug" or "feature."
Projects
None yet
Development

No branches or pull requests

4 participants