Add support for exposing cover art via library.get_images()

mopidy_beets/__init__.py

@@ -8,7 +8,6 @@
 
 
 class BeetsExtension(ext.Extension):
-
     dist_name = "Mopidy-Beets"
     ext_name = "beets"
     version = __version__

mopidy_beets/browsers/albums.py

@@ -5,7 +5,6 @@
 
 
 class AlbumsCategoryBrowser(GenericBrowserBase):
-
     field = None
     sort_fields = None
     label_fields = None

mopidy_beets/client.py

@@ -200,7 +200,7 @@ def quote_and_encode(text):
 
     @cache()
     def get_artists(self):
-        """ returns all artists of one or more tracks """
+        """returns all artists of one or more tracks"""
         names = self._get("/artist/")["artist_names"]
         names.sort()
         # remove empty names
@@ -216,7 +216,7 @@ def get_sorted_unique_album_attributes(self, field):
 
     @cache(ctl=32)
     def _get_unique_attribute_values(self, base_url, field, sort_field):
-        """ returns all artists, genres, ... of tracks or albums """
+        """returns all artists, genres, ... of tracks or albums"""
         if not hasattr(self, "__legacy_beets_api_detected"):
             try:
                 result = self._get(

mopidy_beets/library.py

@@ -22,7 +22,6 @@
 
 
 class BeetsLibraryProvider(backend.LibraryProvider):
-
     root_directory = models.Ref.directory(
         uri="beets:library", name="Beets library"
     )
@@ -95,7 +94,7 @@ def search(self, query=None, uris=None, exact=False):
 
         self._validate_query(query)
         search_list = []
-        for (field, values) in query.items():
+        for field, values in query.items():
             for val in values:
                 # missing / unsupported fields: uri, performer
                 if field == "any":
@@ -177,6 +176,15 @@ def lookup(self, uri=None, uris=None):
             # the newer method (mopidy>=1.0): return a dict of uris and tracks
             return {uri: self.lookup(uri=uri) for uri in uris}
 
+    def get_images(self, uris):
+        images = {}
+        for uri in uris:
+            path, item_id = parse_uri(uri, uri_prefix=self.root_directory.uri)
+            if path == "album" and isinstance(item_id, int):
+                url = self.remote.get_album_art_url(item_id)
+                images[uri] = [models.Image(uri=url)]
+        return images
+
     def get_distinct(self, field, query=None):
         logger.debug("Beets distinct query: %s (uri=%s)", field, query)
         if not self.remote.has_connection:

setup.cfg

@@ -74,6 +74,8 @@ select =
     # pep8-naming rules
     N
 ignore =
+    # B019: Use of `functools.lru_cache` on methods can lead to memory leaks.
+    B019
     # E203: whitespace before ':' (not PEP8 compliant)
     E203
     # E501: line too long (replaced by B950)