comments, naming, and docs for part of #509

This commit is contained in:
Adrian Sampson 2014-01-27 22:38:23 -08:00
parent 0466df6215
commit 06b3ee179c
3 changed files with 11 additions and 4 deletions

View file

@ -212,15 +212,19 @@ def batch_fetch_art(lib, albums, force, maxwidth=None):
if album.artpath and not force:
message = 'has album art'
else:
lookuppath = None if force else [album.path]
path = art_for_album(album, lookuppath, maxwidth)
# In ordinary invocations, look for images on the
# filesystem. When forcing, however, always go to the Web
# sources.
local_paths = None if force else [album.path]
path = art_for_album(album, local_paths, maxwidth)
if path:
album.set_art(path, False)
album.store()
message = 'found album art'
else:
message = 'no art found'
log.info(u'{0} - {1}: {2}'.format(album.albumartist, album.album,
message))

View file

@ -39,6 +39,9 @@ Other little fixes:
* Fix a potential crash when using image resizing with the
:doc:`/plugins/fetchart` or :doc:`/plugins/embedart` without ImageMagick
installed.
* :doc:`/plugins/fetchart`: The ``beet fetchart`` command can now associate
local images with albums (unless ``--force`` is provided). Thanks to
brilnius.
1.3.2 (December 22, 2013)

View file

@ -32,8 +32,8 @@ been imported::
By default, the command will only look for album art when the album doesn't
already have it; the ``-f`` or ``--force`` switch makes it search for art
regardless. If you specify a query, only matching albums will be processed;
otherwise, the command processes every album in your library.
in Web databases regardless. If you specify a query, only matching albums will
be processed; otherwise, the command processes every album in your library.
.. _image-resizing: