Merge pull request #2816 from beetbox/better-release-event

Make sure release events are selected in preferred order.
beetbox · Feb 23, 2018 · 3460723 · 3460723
2 parents d6b6ebb + 5d2e203
commit 3460723
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/beets/autotag/mb.py b/beets/autotag/mb.py
@@ -118,8 +118,8 @@ def _preferred_release_event(release):
     """
     countries = config['match']['preferred']['countries'].as_str_seq()
 
-    for event in release.get('release-event-list', {}):
-        for country in countries:
+    for country in countries:
+        for event in release.get('release-event-list', {}):
             try:
                 if country in event['area']['iso-3166-1-code-list']:
                     return country, event['date']

diff --git a/docs/changelog.rst b/docs/changelog.rst
@@ -56,6 +56,9 @@ Fixes:
   incompatible between the source and target file formats.
   :bug:`2814`
   Thanks to :user:`autrimpo`.
+* Importing a release with multiple release events now selects the
+  event based on the order of your :ref:`preferred` countries rather than
+  the order of release events in MusicBrainz. :bug:`2816`
 
 For developers: