Skip to content

Add fetch_for_asis setting to FetchArt plugin#6815

Open
overhacked wants to merge 3 commits into
beetbox:masterfrom
overhacked:overhacked/fetchart_asis
Open

Add fetch_for_asis setting to FetchArt plugin#6815
overhacked wants to merge 3 commits into
beetbox:masterfrom
overhacked:overhacked/fetchart_asis

Conversation

@overhacked

Copy link
Copy Markdown

Add fetch_for_asis setting to FetchArt plugin

Description

Enable fetching art for imports even when as-is is selected as the metadata source. Allows for the case when files with good metadata but without album art are being imported (e.g. digital download store).

To Do

  • Documentation.
  • Changelog.
  • Tests.

@overhacked overhacked requested a review from a team as a code owner July 2, 2026 12:34
@github-actions github-actions Bot added the fetchart fetchart plugin label Jul 2, 2026
@codecov

codecov Bot commented Jul 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.94%. Comparing base (9acb1ec) to head (6afc0f2).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #6815   +/-   ##
=======================================
  Coverage   74.94%   74.94%           
=======================================
  Files         163      163           
  Lines       21019    21022    +3     
  Branches     3310     3310           
=======================================
+ Hits        15752    15755    +3     
  Misses       4513     4513           
  Partials      754      754           
Files with missing lines Coverage Δ
beetsplug/fetchart.py 74.55% <100.00%> (+0.37%) ⬆️

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Comment thread test/plugins/test_fetchart.py Outdated
Comment thread beetsplug/fetchart.py Outdated
Comment thread test/plugins/test_fetchart.py Outdated
@overhacked overhacked force-pushed the overhacked/fetchart_asis branch 4 times, most recently from 22d3901 to 3b66c19 Compare July 4, 2026 07:57

@snejus snejus left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@snejus

snejus commented Jul 10, 2026

Copy link
Copy Markdown
Member

You just need to rebase now and to resolve conflicts.

Enable fetching art for imports even when as-is is selected as the
metadata source. Allows for the case when files with good metadata
but without album art are being imported (e.g. digital download store).

Signed-off-by: Ross Williams <ross@ross-williams.net>
Signed-off-by: Ross Williams <ross@ross-williams.net>
Test whether fetchart is run during the import phase and, specifically,
whether the `fetch_for_asis` setting is honored. Mock boundary is
between `FetchArtPlugin` and `ArtSource`, an already-existing internal
API boundary.
@overhacked overhacked force-pushed the overhacked/fetchart_asis branch from 3b66c19 to 6afc0f2 Compare July 10, 2026 21:22
@overhacked

Copy link
Copy Markdown
Author

@snejus rebased and ready!

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

Labels

fetchart fetchart plugin

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants