Skip to content

Commit 2618fc6

Browse files
committed
SEARCH-628: Unmarshalling rules for Primary Type
1 parent 54c8c24 commit 2618fc6

11 files changed

Lines changed: 17 additions & 50 deletions

File tree

mb-solr/src/main/java/org/musicbrainz/search/solrwriter/moxy/StringPrimaryTypeAdapter.java

Lines changed: 0 additions & 41 deletions
This file was deleted.

mb-solr/src/main/resources/oxml.xml

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -261,6 +261,7 @@
261261
<java-type name="ReleaseGroup">
262262
<java-attributes>
263263
<xml-element java-attribute="secondaryTypeList" xml-path="."/>
264+
<xml-element java-attribute="primaryType" xml-path="." />
264265
</java-attributes>
265266
</java-type>
266267
<java-type name="ReleaseGroup">
@@ -452,9 +453,6 @@
452453
<java-type name="ReleaseGroup">
453454
<java-attributes>
454455
<xml-element java-attribute="tagList" xml-path="."/>
455-
<xml-element java-attribute="primaryType" name="primary-type">
456-
<xml-java-type-adapter value="org.musicbrainz.search.solrwriter.moxy.StringPrimaryTypeAdapter"/>
457-
</xml-element>
458456
</java-attributes>
459457
</java-type>
460458
<java-type name="Medium$TrackList">
@@ -540,6 +538,12 @@
540538
<xml-element java-attribute="iso31663Code" name="iso-3166-3-codes"/>
541539
</java-attributes>
542540
</java-type>
541+
<java-type name="PrimaryType">
542+
<java-attributes>
543+
<xml-element java-attribute="content" name="primary-type" />
544+
<xml-attribute java-attribute="id" name="primary-type-id" />
545+
</java-attributes>
546+
</java-type>
543547
<java-type name="Alias">
544548
<xml-type prop-order=""/>
545549
<java-attributes>

mb-solr/src/test/resources/org/musicbrainz/search/solrwriter/recording-list.json

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,7 @@
5555
"release-group": {
5656
"id": "f5fd971c-a1a2-33da-95b0-473e0176df83",
5757
"primary-type": "Album",
58+
"primary-type-id": "f529b476-6e62-324f-b0aa-1f3e33d313fc",
5859
"secondary-types": [
5960
"Soundtrack"
6061
]
@@ -99,6 +100,7 @@
99100
"release-group": {
100101
"id": "f5fd971c-a1a2-33da-95b0-473e0176df83",
101102
"primary-type": "Album",
103+
"primary-type-id": "f529b476-6e62-324f-b0aa-1f3e33d313fc",
102104
"secondary-types": [
103105
"Soundtrack"
104106
]
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
<?xml version="1.0" encoding="UTF-8" standalone="yes"?><metadata xmlns="http://musicbrainz.org/ns/mmd-2.0#" xmlns:ns2="http://musicbrainz.org/ns/ext#-2.0"><recording-list count="1" offset="0"><recording id="3e0bdde6-74e0-46d4-855a-4493589fd6b2" ns2:score="100"><title>Roots and Beginnings</title><length>391000</length><artist-credit><name-credit><artist id="9b58672a-e68e-4972-956e-a8985a165a1f"><name>Howard Shore</name><sort-name>Shore, Howard</sort-name><alias-list><alias sort-name="Howard Shaw">Howard Shaw</alias><alias sort-name="H. Shore">H. Shore</alias><alias sort-name="Shore">Shore</alias></alias-list></artist></name-credit></artist-credit><release-list><release id="e7fe61ca-65b8-4abb-8723-1e7c1a5e0a49"><title>The Lord of the Rings: The Return of the King: The Complete Recordings</title><status>Official</status><release-group id="f5fd971c-a1a2-33da-95b0-473e0176df83" type="Soundtrack"><primary-type>Album</primary-type><secondary-type-list><secondary-type>Soundtrack</secondary-type></secondary-type-list></release-group><date>2007-11-20</date><country>US</country><release-event-list><release-event><date>2007-11-20</date><area id="489ce91b-6658-3307-9877-795b68554c98"><name>United States</name><sort-name>United States</sort-name><iso-3166-1-code-list><iso-3166-1-code>US</iso-3166-1-code></iso-3166-1-code-list></area></release-event></release-event-list><medium-list><track-count>106</track-count><medium><position>1</position><format>CD</format><track-list count="15" offset="0"><track id="fc51bf50-c329-3d93-bb1b-e4f55cf00db8"><number>1</number><title>Roots and Beginnings</title><length>391000</length></track></track-list></medium></medium-list></release><release id="e7fe61ca-65b8-4abb-8723-1e7c1a5e0a49"><title>The Lord of the Rings: The Return of the King: The Complete Recordings</title><status>Official</status><release-group id="f5fd971c-a1a2-33da-95b0-473e0176df83" type="Soundtrack"><primary-type>Album</primary-type><secondary-type-list><secondary-type>Soundtrack</secondary-type></secondary-type-list></release-group><date>2007-11-20</date><country>US</country><release-event-list><release-event><date>2007-11-20</date><area id="489ce91b-6658-3307-9877-795b68554c98"><name>United States</name><sort-name>United States</sort-name><iso-3166-1-code-list><iso-3166-1-code>US</iso-3166-1-code></iso-3166-1-code-list></area></release-event></release-event-list><medium-list><track-count>106</track-count><medium><position>5</position><format>DVD</format><track-list count="53" offset="0"><track id="af77973d-3ca5-38e3-89ff-c742c94f59af"><number>1</number><title>Roots and Beginnings</title><length>391000</length></track></track-list></medium></medium-list></release></release-list><isrc-list><isrc id="USRE10701410"/></isrc-list><tag-list><tag count="2"><name>soundtrack</name></tag></tag-list></recording></recording-list></metadata>
1+
<?xml version="1.0" encoding="UTF-8" standalone="yes"?><metadata xmlns="http://musicbrainz.org/ns/mmd-2.0#" xmlns:ns2="http://musicbrainz.org/ns/ext#-2.0"><recording-list count="1" offset="0"><recording id="3e0bdde6-74e0-46d4-855a-4493589fd6b2" ns2:score="100"><title>Roots and Beginnings</title><length>391000</length><artist-credit><name-credit><artist id="9b58672a-e68e-4972-956e-a8985a165a1f"><name>Howard Shore</name><sort-name>Shore, Howard</sort-name><alias-list><alias sort-name="Howard Shaw">Howard Shaw</alias><alias sort-name="H. Shore">H. Shore</alias><alias sort-name="Shore">Shore</alias></alias-list></artist></name-credit></artist-credit><release-list><release id="e7fe61ca-65b8-4abb-8723-1e7c1a5e0a49"><title>The Lord of the Rings: The Return of the King: The Complete Recordings</title><status>Official</status><release-group id="f5fd971c-a1a2-33da-95b0-473e0176df83" type="Soundtrack"><primary-type id="f529b476-6e62-324f-b0aa-1f3e33d313fc">Album</primary-type><secondary-type-list><secondary-type>Soundtrack</secondary-type></secondary-type-list></release-group><date>2007-11-20</date><country>US</country><release-event-list><release-event><date>2007-11-20</date><area id="489ce91b-6658-3307-9877-795b68554c98"><name>United States</name><sort-name>United States</sort-name><iso-3166-1-code-list><iso-3166-1-code>US</iso-3166-1-code></iso-3166-1-code-list></area></release-event></release-event-list><medium-list><track-count>106</track-count><medium><position>1</position><format>CD</format><track-list count="15" offset="0"><track id="fc51bf50-c329-3d93-bb1b-e4f55cf00db8"><number>1</number><title>Roots and Beginnings</title><length>391000</length></track></track-list></medium></medium-list></release><release id="e7fe61ca-65b8-4abb-8723-1e7c1a5e0a49"><title>The Lord of the Rings: The Return of the King: The Complete Recordings</title><status>Official</status><release-group id="f5fd971c-a1a2-33da-95b0-473e0176df83" type="Soundtrack"><primary-type id="f529b476-6e62-324f-b0aa-1f3e33d313fc">Album</primary-type><secondary-type-list><secondary-type>Soundtrack</secondary-type></secondary-type-list></release-group><date>2007-11-20</date><country>US</country><release-event-list><release-event><date>2007-11-20</date><area id="489ce91b-6658-3307-9877-795b68554c98"><name>United States</name><sort-name>United States</sort-name><iso-3166-1-code-list><iso-3166-1-code>US</iso-3166-1-code></iso-3166-1-code-list></area></release-event></release-event-list><medium-list><track-count>106</track-count><medium><position>5</position><format>DVD</format><track-list count="53" offset="0"><track id="af77973d-3ca5-38e3-89ff-c742c94f59af"><number>1</number><title>Roots and Beginnings</title><length>391000</length></track></track-list></medium></medium-list></release></release-list><isrc-list><isrc id="USRE10701410"/></isrc-list><tag-list><tag count="2"><name>soundtrack</name></tag></tag-list></recording></recording-list></metadata>
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
<recording xmlns="http://musicbrainz.org/ns/mmd-2.0#" id="3e0bdde6-74e0-46d4-855a-4493589fd6b2"><title>Roots and Beginnings</title><length>391000</length><artist-credit><name-credit><artist id="9b58672a-e68e-4972-956e-a8985a165a1f"><name>Howard Shore</name><sort-name>Shore, Howard</sort-name><alias-list><alias sort-name="Howard Shaw">Howard Shaw</alias><alias sort-name="H. Shore">H. Shore</alias><alias sort-name="Shore">Shore</alias></alias-list></artist></name-credit></artist-credit><release-list><release id="e7fe61ca-65b8-4abb-8723-1e7c1a5e0a49"><title>The Lord of the Rings: The Return of the King: The Complete Recordings</title><status>Official</status><release-group id="f5fd971c-a1a2-33da-95b0-473e0176df83" type="Soundtrack"><primary-type>Album</primary-type><secondary-type-list><secondary-type>Soundtrack</secondary-type></secondary-type-list></release-group><date>2007-11-20</date><country>US</country><release-event-list><release-event><date>2007-11-20</date><area id="489ce91b-6658-3307-9877-795b68554c98"><name>United States</name><sort-name>United States</sort-name><iso-3166-1-code-list><iso-3166-1-code>US</iso-3166-1-code></iso-3166-1-code-list></area></release-event></release-event-list><medium-list><track-count>106</track-count><medium><position>1</position><format>CD</format><track-list count="15" offset="0"><track id="fc51bf50-c329-3d93-bb1b-e4f55cf00db8"><number>1</number><title>Roots and Beginnings</title><length>391000</length></track></track-list></medium></medium-list></release><release id="e7fe61ca-65b8-4abb-8723-1e7c1a5e0a49"><title>The Lord of the Rings: The Return of the King: The Complete Recordings</title><status>Official</status><release-group id="f5fd971c-a1a2-33da-95b0-473e0176df83" type="Soundtrack"><primary-type>Album</primary-type><secondary-type-list><secondary-type>Soundtrack</secondary-type></secondary-type-list></release-group><date>2007-11-20</date><country>US</country><release-event-list><release-event><date>2007-11-20</date><area id="489ce91b-6658-3307-9877-795b68554c98"><name>United States</name><sort-name>United States</sort-name><iso-3166-1-code-list><iso-3166-1-code>US</iso-3166-1-code></iso-3166-1-code-list></area></release-event></release-event-list><medium-list><track-count>106</track-count><medium><position>5</position><format>DVD</format><track-list count="53" offset="0"><track id="af77973d-3ca5-38e3-89ff-c742c94f59af"><number>1</number><title>Roots and Beginnings</title><length>391000</length></track></track-list></medium></medium-list></release></release-list><isrc-list><isrc id="USRE10701410"/></isrc-list><tag-list><tag count="2"><name>soundtrack</name></tag></tag-list></recording>
1+
<recording xmlns="http://musicbrainz.org/ns/mmd-2.0#" id="3e0bdde6-74e0-46d4-855a-4493589fd6b2"><title>Roots and Beginnings</title><length>391000</length><artist-credit><name-credit><artist id="9b58672a-e68e-4972-956e-a8985a165a1f"><name>Howard Shore</name><sort-name>Shore, Howard</sort-name><alias-list><alias sort-name="Howard Shaw">Howard Shaw</alias><alias sort-name="H. Shore">H. Shore</alias><alias sort-name="Shore">Shore</alias></alias-list></artist></name-credit></artist-credit><release-list><release id="e7fe61ca-65b8-4abb-8723-1e7c1a5e0a49"><title>The Lord of the Rings: The Return of the King: The Complete Recordings</title><status>Official</status><release-group id="f5fd971c-a1a2-33da-95b0-473e0176df83" type="Soundtrack"><primary-type id="f529b476-6e62-324f-b0aa-1f3e33d313fc">Album</primary-type><secondary-type-list><secondary-type>Soundtrack</secondary-type></secondary-type-list></release-group><date>2007-11-20</date><country>US</country><release-event-list><release-event><date>2007-11-20</date><area id="489ce91b-6658-3307-9877-795b68554c98"><name>United States</name><sort-name>United States</sort-name><iso-3166-1-code-list><iso-3166-1-code>US</iso-3166-1-code></iso-3166-1-code-list></area></release-event></release-event-list><medium-list><track-count>106</track-count><medium><position>1</position><format>CD</format><track-list count="15" offset="0"><track id="fc51bf50-c329-3d93-bb1b-e4f55cf00db8"><number>1</number><title>Roots and Beginnings</title><length>391000</length></track></track-list></medium></medium-list></release><release id="e7fe61ca-65b8-4abb-8723-1e7c1a5e0a49"><title>The Lord of the Rings: The Return of the King: The Complete Recordings</title><status>Official</status><release-group id="f5fd971c-a1a2-33da-95b0-473e0176df83" type="Soundtrack"><primary-type id="f529b476-6e62-324f-b0aa-1f3e33d313fc">Album</primary-type><secondary-type-list><secondary-type>Soundtrack</secondary-type></secondary-type-list></release-group><date>2007-11-20</date><country>US</country><release-event-list><release-event><date>2007-11-20</date><area id="489ce91b-6658-3307-9877-795b68554c98"><name>United States</name><sort-name>United States</sort-name><iso-3166-1-code-list><iso-3166-1-code>US</iso-3166-1-code></iso-3166-1-code-list></area></release-event></release-event-list><medium-list><track-count>106</track-count><medium><position>5</position><format>DVD</format><track-list count="53" offset="0"><track id="af77973d-3ca5-38e3-89ff-c742c94f59af"><number>1</number><title>Roots and Beginnings</title><length>391000</length></track></track-list></medium></medium-list></release></release-list><isrc-list><isrc id="USRE10701410"/></isrc-list><tag-list><tag count="2"><name>soundtrack</name></tag></tag-list></recording>

mb-solr/src/test/resources/org/musicbrainz/search/solrwriter/release-group-list.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
"count": 1,
1010
"title": "Chiptunes = WIN \\m|♥|m/",
1111
"primary-type": "Album",
12+
"primary-type-id": "f529b476-6e62-324f-b0aa-1f3e33d313fc",
1213
"secondary-types": [
1314
"Compilation"
1415
],
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
<?xml version="1.0" encoding="UTF-8" standalone="yes"?><metadata xmlns="http://musicbrainz.org/ns/mmd-2.0#" xmlns:ns2="http://musicbrainz.org/ns/ext#-2.0"><release-group-list count="1" offset="0"><release-group id="268d91a0-1a7f-4600-bfd4-74d738eb7de4" type="Compilation" ns2:score="100"><title>Chiptunes = WIN \m|♥|m/</title><primary-type>Album</primary-type><secondary-type-list><secondary-type>Compilation</secondary-type></secondary-type-list><artist-credit><name-credit><artist id="89ad4ac3-39f7-470e-963a-56509c546377"><name>Various Artists</name><sort-name>Various Artists</sort-name><disambiguation>add compilations to this artist</disambiguation></artist></name-credit></artist-credit><release-list count="1"><release id="871f12ec-6259-4e93-b440-9108559c38b1"><title>Chiptunes = WIN \m|♥|m/</title><status>Official</status></release></release-list><tag-list><tag count="1"><name>electronic</name></tag></tag-list></release-group></release-group-list></metadata>
1+
<?xml version="1.0" encoding="UTF-8" standalone="yes"?><metadata xmlns="http://musicbrainz.org/ns/mmd-2.0#" xmlns:ns2="http://musicbrainz.org/ns/ext#-2.0"><release-group-list count="1" offset="0"><release-group id="268d91a0-1a7f-4600-bfd4-74d738eb7de4" type="Compilation" ns2:score="100"><title>Chiptunes = WIN \m|♥|m/</title><primary-type id="f529b476-6e62-324f-b0aa-1f3e33d313fc">Album</primary-type><secondary-type-list><secondary-type>Compilation</secondary-type></secondary-type-list><artist-credit><name-credit><artist id="89ad4ac3-39f7-470e-963a-56509c546377"><name>Various Artists</name><sort-name>Various Artists</sort-name><disambiguation>add compilations to this artist</disambiguation></artist></name-credit></artist-credit><release-list count="1"><release id="871f12ec-6259-4e93-b440-9108559c38b1"><title>Chiptunes = WIN \m|♥|m/</title><status>Official</status></release></release-list><tag-list><tag count="1"><name>electronic</name></tag></tag-list></release-group></release-group-list></metadata>
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
<release-group xmlns="http://musicbrainz.org/ns/mmd-2.0#" id="268d91a0-1a7f-4600-bfd4-74d738eb7de4" type="Compilation"><title>Chiptunes = WIN \m|♥|m/</title><primary-type>Album</primary-type><secondary-type-list><secondary-type>Compilation</secondary-type></secondary-type-list><artist-credit><name-credit><artist id="89ad4ac3-39f7-470e-963a-56509c546377"><name>Various Artists</name><sort-name>Various Artists</sort-name><disambiguation>add compilations to this artist</disambiguation></artist></name-credit></artist-credit><release-list count="1"><release id="871f12ec-6259-4e93-b440-9108559c38b1"><title>Chiptunes = WIN \m|♥|m/</title><status>Official</status></release></release-list><tag-list><tag count="1"><name>electronic</name></tag></tag-list></release-group>
1+
<release-group xmlns="http://musicbrainz.org/ns/mmd-2.0#" id="268d91a0-1a7f-4600-bfd4-74d738eb7de4" type="Compilation"><title>Chiptunes = WIN \m|♥|m/</title><primary-type id="f529b476-6e62-324f-b0aa-1f3e33d313fc">Album</primary-type><secondary-type-list><secondary-type>Compilation</secondary-type></secondary-type-list><artist-credit><name-credit><artist id="89ad4ac3-39f7-470e-963a-56509c546377"><name>Various Artists</name><sort-name>Various Artists</sort-name><disambiguation>add compilations to this artist</disambiguation></artist></name-credit></artist-credit><release-list count="1"><release id="871f12ec-6259-4e93-b440-9108559c38b1"><title>Chiptunes = WIN \m|♥|m/</title><status>Official</status></release></release-list><tag-list><tag count="1"><name>electronic</name></tag></tag-list></release-group>

mb-solr/src/test/resources/org/musicbrainz/search/solrwriter/release-list.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,7 @@
3636
"release-group": {
3737
"id": "268d91a0-1a7f-4600-bfd4-74d738eb7de4",
3838
"primary-type": "Album",
39+
"primary-type-id": "f529b476-6e62-324f-b0aa-1f3e33d313fc",
3940
"secondary-types": [
4041
"Compilation"
4142
]

0 commit comments

Comments
 (0)