beetbox / beets

music library manager and MusicBrainz tagger
http://beets.io/
MIT License
12.78k stars 1.82k forks source link

fetchart: defer file removal config option evaluation #5244

Closed mgoltzsche closed 2 weeks ago

mgoltzsche commented 4 months ago

Description

Defer the evaluation of the source file removal options (import.delete and import.move) to the point where the fetchart plugin is actually called instead of only evaluating those configuration options on plugin initialization. This is to allow other plugins (such as the ytimport plugin) to invoke the import directly (within the same python process; implicitly invoking the fetchart plugin) with temporarily overwritten configuration options.

Addresses https://github.com/beetbox/beets/issues/5167#issuecomment-2106465172

To Do

mgoltzsche commented 3 months ago

I adjusted the test correspondingly now. Please re-review.