[Feature] Store album art as a separate file in the album folder

[artofsound]

I want an option to store the album art as a additional file in the same folder, where the downloaded track is stored.
Saving the album art should be done, after downloading all of the album tracks. Doesn't need to be after every single track.

Probably something like "cover.jpg" should do the trick, but maybe you could add a filename option, so users could change it on their behalf.

[NiciTheNici]

bump, this is insanely useful for rockbox users

[NiciTheNici]

I managed to get this working on my fork. @casualsnek do you think this even needs an option or should it just default to saving the cover art as cover.png all the time like I did in my fork?

https://github.com/NiciTheNici/onthespot/tree/save_cover_as_file

[casualsnek]

I managed to get this working on my fork. @casualsnek do you think this even needs an option or should it just default to saving the cover art as cover.png all the time like I did in my fork?

https://github.com/NiciTheNici/onthespot/tree/save_cover_as_file

This being a default behavior won't be too bad as i have seen a lot of media player use it instead of embedded one or using embedded ones only if cover.png is missing !

Good to know you got it working, normally i wold have suggested you to make a PR, but since i am rewriting onthespot all PR and non critical Issue resolution are paused right now. I may make this a default behavior but configurable using the config files in the rewrite.

[artofsound]

I also have a version that works on my fork, even with a settings option you can tick. I used .jpg tho, because that's what Spotify gives you.
If the file already exists, it skips it.

https://github.com/artofsound/onthespot/tree/cover_image

[NiciTheNici]

I used .jpg tho, because that's what Spotify gives you.

Ah I just used png because that's what the method set_music_thumbnail used a few lines above my text. Not unnecessarily converting between formats is probably more efficient...

If the file already exists, it skips it.

Same!

I may make this a default behavior but configurable using the config files in the rewrite.

Sounds perfect! Thanks!