Add annotation as a valid inc for browse methods

Also add to lookup event, and parse them in events Fixes #251 Signed-off-by: Alastair Porter <[email protected]>
alastair · Jan 16, 2022 · 95e85ec · 95e85ec
1 parent 795b7de
commit 95e85ec
Show file tree

Hide file tree

Showing 4 changed files with 20 additions and 10 deletions.
diff --git a/musicbrainzngs/mbxml.py b/musicbrainzngs/mbxml.py
@@ -295,7 +295,8 @@ def parse_event(event):
                  "alias-list": parse_alias_list,
                  "tag-list": parse_tag_list,
                  "user-tag-list": parse_tag_list,
-                 "rating": parse_rating}
+                 "rating": parse_rating,
+                 "annotation": parse_annotation}
 
     result.update(parse_attributes(attribs, event))
     result.update(parse_elements(elements, inner_els, event))

diff --git a/musicbrainzngs/musicbrainz.py b/musicbrainzngs/musicbrainz.py
@@ -51,7 +51,7 @@
         "aliases", "annotation"
     ] + RELATION_INCLUDES + TAG_INCLUDES + RATING_INCLUDES,
     'place': ["aliases", "annotation"] + RELATION_INCLUDES + TAG_INCLUDES,
-    'event': ["aliases"] + TAG_INCLUDES + RATING_INCLUDES + RELATION_INCLUDES,
+    'event': ["aliases", "annotation"] + TAG_INCLUDES + RATING_INCLUDES + RELATION_INCLUDES,
     'recording': [
         "artists", "releases", # Subqueries
         "discids", "media", "artist-credits", "isrcs",
@@ -83,16 +83,17 @@
     'collection': ['releases'],
 }
 VALID_BROWSE_INCLUDES = {
-    'artist': ["aliases"] + TAG_INCLUDES + RATING_INCLUDES + RELATION_INCLUDES,
-    'event': ["aliases"] + TAG_INCLUDES + RATING_INCLUDES + RELATION_INCLUDES,
-    'label': ["aliases"] + TAG_INCLUDES + RATING_INCLUDES + RELATION_INCLUDES,
-    'recording': ["artist-credits", "isrcs", "work-level-rels"] + TAG_INCLUDES + RATING_INCLUDES + RELATION_INCLUDES,
-    'release': ["artist-credits", "labels", "recordings", "isrcs",
+    'artist': ["annotation", "aliases"] + TAG_INCLUDES + RATING_INCLUDES + RELATION_INCLUDES,
+    'event': ["annotation", "aliases"] + TAG_INCLUDES + RATING_INCLUDES + RELATION_INCLUDES,
+    'label': ["annotation", "aliases"] + TAG_INCLUDES + RATING_INCLUDES + RELATION_INCLUDES,
+    'recording': ["annotation", "artist-credits", "isrcs",
+                  "work-level-rels"] + TAG_INCLUDES + RATING_INCLUDES + RELATION_INCLUDES,
+    'release': ["annotation", "artist-credits", "labels", "recordings", "isrcs",
                 "release-groups", "media", "discids"] + RATING_INCLUDES + RELATION_INCLUDES,
-    'place': ["aliases"] + TAG_INCLUDES + RELATION_INCLUDES,
-    'release-group': ["artist-credits"] + TAG_INCLUDES + RATING_INCLUDES + RELATION_INCLUDES,
+    'place': ["annotation", "aliases"] + TAG_INCLUDES + RELATION_INCLUDES,
+    'release-group': ["annotation", "artist-credits"] + TAG_INCLUDES + RATING_INCLUDES + RELATION_INCLUDES,
     'url': RELATION_INCLUDES,
-    'work': ["aliases", "annotation"] + TAG_INCLUDES + RATING_INCLUDES + RELATION_INCLUDES,
+    'work': ["annotation", "aliases", "annotation"] + TAG_INCLUDES + RATING_INCLUDES + RELATION_INCLUDES,
 }
 
 #: These can be used to filter whenever releases are includes or browsed

diff --git a/test/data/artist/0383dadf-2a4e-4d10-a46a-e9e041da8eb3-annotation.xml b/test/data/artist/0383dadf-2a4e-4d10-a46a-e9e041da8eb3-annotation.xml
@@ -0,0 +1,4 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<metadata xmlns="http://musicbrainz.org/ns/mmd-2.0#"><artist id="0383dadf-2a4e-4d10-a46a-e9e041da8eb3" type="Group" type-id="e431f5f6-b5d2-343d-8b36-72607fffb74b"><name>Queen</name><sort-name>Queen</sort-name><annotation><text>'''Note:''' Please add any 2011 Remaster releases to either ''Universal Island Records'' or ''Hollywood Records'' and '''not''' '''''Island Records'''''. Any previously added releases which fall under this category should be edited.
+----
+See also [artist:9dcffd99-eca6-4a09-8dbd-c6c4ed2f20fd|Queen + Paul Rodgers] and [artist:7e07a487-5438-4128-b2d0-f0f18ac08384|Queen + Adam Lambert] for released from Brian May and Roger Taylor collaborating with other singers.</text></annotation><disambiguation>UK rock group</disambiguation><isni-list><isni>0000000115265309</isni></isni-list><country>GB</country><area id="8a754a16-0027-3a29-b6d7-2b40ea0481ed"><name>United Kingdom</name><sort-name>United Kingdom</sort-name><iso-3166-1-code-list><iso-3166-1-code>GB</iso-3166-1-code></iso-3166-1-code-list></area><begin-area id="f03d09b3-39dc-4083-afd6-159e3f0d462f"><name>London</name><sort-name>London</sort-name><disambiguation>Greater London, which includes the City of London</disambiguation></begin-area><life-span><begin>1970-06-27</begin></life-span></artist></metadata>
diff --git a/test/test_mbxml_artist.py b/test/test_mbxml_artist.py
@@ -31,3 +31,7 @@ def testArtistTargets(self):
         res = _common.open_and_parse_test_data(self.datadir, "b3785a55-2cf6-497d-b8e3-cfa21a36f997-artist-rels.xml")
         self.assertTrue('target-credit' in res['artist']['artist-relation-list'][0])
         self.assertEqual(res['artist']['artist-relation-list'][0]["target-credit"], "TAO")
+
+    def testArtistAnnotation(self):
+        res = _common.open_and_parse_test_data(self.datadir, "0383dadf-2a4e-4d10-a46a-e9e041da8eb3-annotation.xml")
+        self.assertTrue('Please add any 2011 Remaster releases' in res['artist']['annotation']['text'])