Only consider release collections in mbcollection plugin (#5856)

Currently, `mbcollection` takes the first collection it finds (if the
user does not explicitely indicates one in the configuration).

The problem is that the user may have collections of types that we are
not interested in (e.g. recording collections, artist collections,
etc.).

Since the `get_collections` response returns the collections types, we
can only keep the `release` collections.

Author: snejus

beetsplug/mbcollection.py

@@ -70,10 +70,14 @@ def _get_collection(self):
         if not collections["collection-list"]:
             raise ui.UserError("no collections exist for user")
 
-        # Get all collection IDs, avoiding event collections
-        collection_ids = [x["id"] for x in collections["collection-list"]]
+        # Get all release collection IDs, avoiding event collections
+        collection_ids = [
+            x["id"]
+            for x in collections["collection-list"]
+            if x["entity-type"] == "release"
+        ]
         if not collection_ids:
-            raise ui.UserError("No collection found.")
+            raise ui.UserError("No release collection found.")
 
         # Check that the collection exists so we can present a nice error
         collection = self.config["collection"].as_str()

docs/changelog.rst

@@ -28,6 +28,8 @@ New features:
   :bug:`5832`
 * :doc:`plugins/playlist`: Support files with the `.m3u8` extension.
   :bug:`5829`
+* :doc:`plugins/mbcollection`: When getting the user collections, only consider
+  collections of releases, and ignore collections of other entity types.
 
 Bug fixes: