Disable album art embedding about beets-alternatives

[wisp3rwind]

Yeah, embed should really be a per-alternative option. Maybe, for backwards compatibility, it could default to convert.embed if not set, i.e.

self._embed = config.get(dict).get(
    'embed',
    convert_plugin.config["embed"].get(bool)
)

In fact, the _embed handling should probably be moved to the External() class rather than ExternalConvert(): You're actually correct that art embedding is not handled consistently. ExternalConvert() checks converts embed option when initially adding items to the library, but it inherits External()s methods which will embed the art anyway on the next alt update.

I'll see if I can find some time to fix this, which could then serve as a starting point for your other PR. I won't be able to do so before the weekend, however.

Another approach to the configuration might be to add an alternatives.<name>.sync-art = ["no"|"embed"|"copy"] option.

[pkel]

I think the convert.embed option is considered when a song is added to the alternative directory:

https://github.com/geigerzaehler/beets-alternatives/blob/master/beetsplug/alternatives.py#L339

So yes, my expectation is that also beet alt update considers the convert.embed config option.

Stepping back a little, I think it would be better (and easier to implement) to have a separate alternatives.name.embed option. Similar to the copy_album_art option I propose in #58.

[geigerzaehler]

Stepping back a little, I think it would be better (and easier to implement) to have a separate alternatives.name.embed option

For me this seems to be the right way forward, too.