aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLuke Pulverenti <luke.pulverenti@gmail.com>2017-10-28 00:20:18 -0400
committerLuke Pulverenti <luke.pulverenti@gmail.com>2017-10-28 00:20:18 -0400
commit5106819a4b9bdcc12535f47cbdc447970140f479 (patch)
tree9a7c073b070f072d7b6f2ddd0f7a30623dc765f9
parent7cd1416d276cfede3de315cde434b5d78936e011 (diff)
show album artist in album identify wizard
-rw-r--r--MediaBrowser.Model/Providers/RemoteSearchResult.cs2
-rw-r--r--MediaBrowser.Providers/Music/MusicBrainzAlbumProvider.cs163
2 files changed, 165 insertions, 0 deletions
diff --git a/MediaBrowser.Model/Providers/RemoteSearchResult.cs b/MediaBrowser.Model/Providers/RemoteSearchResult.cs
index 72b6632e4..fdff7ba46 100644
--- a/MediaBrowser.Model/Providers/RemoteSearchResult.cs
+++ b/MediaBrowser.Model/Providers/RemoteSearchResult.cs
@@ -34,6 +34,8 @@ namespace MediaBrowser.Model.Providers
public string GameSystem { get; set; }
public string Overview { get; set; }
+ public RemoteSearchResult AlbumArtist { get; set; }
+
public RemoteSearchResult()
{
ProviderIds = new Dictionary<string, string>(StringComparer.OrdinalIgnoreCase);
diff --git a/MediaBrowser.Providers/Music/MusicBrainzAlbumProvider.cs b/MediaBrowser.Providers/Music/MusicBrainzAlbumProvider.cs
index b54d14935..3befca428 100644
--- a/MediaBrowser.Providers/Music/MusicBrainzAlbumProvider.cs
+++ b/MediaBrowser.Providers/Music/MusicBrainzAlbumProvider.cs
@@ -117,6 +117,17 @@ namespace MediaBrowser.Providers.Music
ProductionYear = i.Year
};
+ if (i.Artists.Count > 0)
+ {
+ result.AlbumArtist = new RemoteSearchResult
+ {
+ SearchProviderName = Name,
+ Name = i.Artists[0].Item1
+ };
+
+ result.AlbumArtist.SetProviderId(MetadataProviders.MusicBrainzArtist, i.Artists[0].Item2);
+ }
+
if (!string.IsNullOrWhiteSpace(i.ReleaseId))
{
result.SetProviderId(MetadataProviders.MusicBrainzAlbum, i.ReleaseId);
@@ -285,6 +296,8 @@ namespace MediaBrowser.Providers.Music
public string Overview;
public int? Year;
+ public List<Tuple<string, string>> Artists = new List<Tuple<string, string>>();
+
public static List<ReleaseResult> Parse(XmlReader reader)
{
reader.MoveToContent();
@@ -419,6 +432,32 @@ namespace MediaBrowser.Providers.Music
reader.Skip();
break;
}
+ case "artist-credit":
+ {
+ // TODO
+
+ /*
+ * <artist-credit>
+<name-credit>
+<artist id="e225cda5-882d-4b80-b8a3-b36d7175b1ea">
+<name>SARCASTIC+ZOOKEEPER</name>
+<sort-name>SARCASTIC+ZOOKEEPER</sort-name>
+</artist>
+</name-credit>
+</artist-credit>
+ */
+ using (var subReader = reader.ReadSubtree())
+ {
+ var artist = ParseArtistCredit(subReader);
+
+ if (artist != null)
+ {
+ result.Artists.Add(artist);
+ }
+ }
+
+ break;
+ }
default:
{
reader.Skip();
@@ -436,6 +475,130 @@ namespace MediaBrowser.Providers.Music
}
}
+ private static Tuple<string, string> ParseArtistCredit(XmlReader reader)
+ {
+ reader.MoveToContent();
+ reader.Read();
+
+ // http://stackoverflow.com/questions/2299632/why-does-xmlreader-skip-every-other-element-if-there-is-no-whitespace-separator
+
+ // Loop through each element
+ while (!reader.EOF && reader.ReadState == ReadState.Interactive)
+ {
+ if (reader.NodeType == XmlNodeType.Element)
+ {
+ switch (reader.Name)
+ {
+ case "name-credit":
+ {
+ using (var subReader = reader.ReadSubtree())
+ {
+ return ParseArtistNameCredit(subReader);
+ }
+ }
+ default:
+ {
+ reader.Skip();
+ break;
+ }
+ }
+ }
+ else
+ {
+ reader.Read();
+ }
+ }
+
+ return null;
+ }
+
+ private static Tuple<string, string> ParseArtistNameCredit(XmlReader reader)
+ {
+ reader.MoveToContent();
+ reader.Read();
+
+ string name = null;
+
+ // http://stackoverflow.com/questions/2299632/why-does-xmlreader-skip-every-other-element-if-there-is-no-whitespace-separator
+
+ // Loop through each element
+ while (!reader.EOF && reader.ReadState == ReadState.Interactive)
+ {
+ if (reader.NodeType == XmlNodeType.Element)
+ {
+ switch (reader.Name)
+ {
+ case "artist":
+ {
+ var id = reader.GetAttribute("id");
+ using (var subReader = reader.ReadSubtree())
+ {
+ return ParseArtistArtistCredit(subReader, id);
+ }
+ }
+ default:
+ {
+ reader.Skip();
+ break;
+ }
+ }
+ }
+ else
+ {
+ reader.Read();
+ }
+ }
+
+ if (string.IsNullOrWhiteSpace(name))
+ {
+ return null;
+ }
+
+ return new Tuple<string, string>(name, null);
+ }
+
+ private static Tuple<string, string> ParseArtistArtistCredit(XmlReader reader, string artistId)
+ {
+ reader.MoveToContent();
+ reader.Read();
+
+ string name = null;
+
+ // http://stackoverflow.com/questions/2299632/why-does-xmlreader-skip-every-other-element-if-there-is-no-whitespace-separator
+
+ // Loop through each element
+ while (!reader.EOF && reader.ReadState == ReadState.Interactive)
+ {
+ if (reader.NodeType == XmlNodeType.Element)
+ {
+ switch (reader.Name)
+ {
+ case "name":
+ {
+ name = reader.ReadElementContentAsString();
+ break;
+ }
+ default:
+ {
+ reader.Skip();
+ break;
+ }
+ }
+ }
+ else
+ {
+ reader.Read();
+ }
+ }
+
+ if (string.IsNullOrWhiteSpace(name))
+ {
+ return null;
+ }
+
+ return new Tuple<string, string>(name, artistId);
+ }
+
private async Task<string> GetReleaseIdFromReleaseGroupId(string releaseGroupId, CancellationToken cancellationToken)
{
var url = string.Format("/ws/2/release?release-group={0}", releaseGroupId);