From f4acc2ad6111ca82c5e212bc2cb5d99ff05bb06e Mon Sep 17 00:00:00 2001
From: Luke Pulverenti <luke.pulverenti@gmail.com>
Date: Fri, 13 Mar 2015 11:54:20 -0400
Subject: [PATCH 1/4] add AlbumArtists to item dto's

---
 dashboard-ui/edititemmetadata.html       |  3 ++-
 dashboard-ui/scripts/edititemmetadata.js | 16 ++++++++++++--
 dashboard-ui/scripts/librarybrowser.js   | 27 ++++++++++++++++++------
 dashboard-ui/scripts/playlist.js         |  2 +-
 4 files changed, 37 insertions(+), 11 deletions(-)

diff --git a/dashboard-ui/edititemmetadata.html b/dashboard-ui/edititemmetadata.html
index 4f6546b4e5..df7f141a6c 100644
--- a/dashboard-ui/edititemmetadata.html
+++ b/dashboard-ui/edititemmetadata.html
@@ -73,8 +73,9 @@
                                 <input type="text" id="txtArtist" placeholder="${LabelArtistsHelp}" />
                             </div>
                             <div data-role="fieldcontain" id="fldAlbumArtist" style="display: none;">
-                                <label for="txtAlbumArtist">${LabelAlbumArtist}</label>
+                                <label for="txtAlbumArtist">${LabelAlbumArtists}</label>
                                 <input type="text" id="txtAlbumArtist" />
+                                <input type="text" id="txtArtist" placeholder="${LabelArtistsHelp}" />
                             </div>
                             <div data-role="fieldcontain" id="fldAlbum" style="display: none;">
                                 <label for="txtAlbum">${LabelAlbum}</label>
diff --git a/dashboard-ui/scripts/edititemmetadata.js b/dashboard-ui/scripts/edititemmetadata.js
index fb6ebde48e..a3966737de 100644
--- a/dashboard-ui/scripts/edititemmetadata.js
+++ b/dashboard-ui/scripts/edititemmetadata.js
@@ -501,7 +501,9 @@
         $('#txtAirsBeforeEpisode', page).val(('AirsBeforeEpisodeNumber' in item) ? item.AirsBeforeEpisodeNumber : "");
 
         $('#txtAlbum', page).val(item.Album || "");
-        $('#txtAlbumArtist', page).val(item.AlbumArtist || "");
+
+        var albumArtists = item.AlbumArtists || [];
+        $('#txtAlbumArtist', page).val(albumArtists.join(';'));
 
         $('#selectDisplayOrder', page).val(item.DisplayOrder).selectmenu('refresh');
 
@@ -802,6 +804,16 @@
             .jstree("delete_node", '#' + id);
     }
 
+    function getAlbumArtists(form) {
+
+        return $('#txtAlbumArtist', form).val().split(';').map(function (a) {
+
+            return {
+                Name: a
+            };
+        });
+    }
+
     function editItemMetadataPage() {
 
         var self = this;
@@ -834,7 +846,7 @@
                 DisplayOrder: $('#selectDisplayOrder', form).val(),
                 Players: $('#txtPlayers', form).val(),
                 Album: $('#txtAlbum', form).val(),
-                AlbumArtist: $('#txtAlbumArtist', form).val(),
+                AlbumArtist: getAlbumArtists(form),
                 Artists: $('#txtArtist', form).val().split(';'),
                 Metascore: $('#txtMetascore', form).val(),
                 AwardSummary: $('#txtAwardSummary', form).val(),
diff --git a/dashboard-ui/scripts/librarybrowser.js b/dashboard-ui/scripts/librarybrowser.js
index 08823d41f5..67a6f9dcf6 100644
--- a/dashboard-ui/scripts/librarybrowser.js
+++ b/dashboard-ui/scripts/librarybrowser.js
@@ -231,6 +231,24 @@
             return html;
         },
 
+        getAlbumArtistLinksHtml: function (artists, cssClass) {
+
+            var html = [];
+
+            for (var i = 0, length = artists.length; i < length; i++) {
+
+                var artist = artists[i];
+
+                var css = cssClass ? (' class="' + cssClass + '"') : '';
+                html.push('<a' + css + ' href="itembynamedetails.html?context=music&id=' + artist.Id + '">' + artist.Name + '</a>');
+
+            }
+
+            html = html.join(' / ');
+
+            return html;
+        },
+
         showPlayMenu: function (positionTo, itemId, itemType, isFolder, mediaType, resumePositionTicks, showAddToPlaylist) {
 
             var externalPlayers = ExternalPlayer.getExternalPlayers();
@@ -1744,10 +1762,8 @@
 
             var contextParam = context ? ('&context=' + context) : '';
 
-            if (item.AlbumArtist && item.Type == "Audio") {
-                html.push('<a class="detailPageParentLink" href="itembynamedetails.html?context=music&musicartist=' + ApiClient.encodeName(item.AlbumArtist) + contextParam + '">' + item.AlbumArtist + '</a>');
-            } else if (item.AlbumArtist && item.Type == "MusicAlbum") {
-                html.push('<a class="detailPageParentLink" href="itembynamedetails.html?context=music&musicartist=' + ApiClient.encodeName(item.AlbumArtist) + contextParam + '">' + item.AlbumArtist + '</a>');
+            if (item.AlbumArtists) {
+                html.push(LibraryBrowser.getAlbumArtistLinksHtml(item.AlbumArtists, "detailPageParentLink"));
             } else if (item.Artists && item.Artists.length && item.Type == "MusicVideo") {
                 html.push('<a class="detailPageParentLink" href="itembynamedetails.html?context=music&musicartist=' + ApiClient.encodeName(item.Artists[0]) + contextParam + '">' + item.Artists[0] + '</a>');
             } else if (item.SeriesName && item.Type == "Episode") {
@@ -1769,11 +1785,8 @@
             } else if (item.Album && item.Type == "MusicVideo" && item.AlbumId) {
                 html.push('<a class="detailPageParentLink" href="itemdetails.html?id=' + item.AlbumId + contextParam + '">' + item.Album + '</a>');
 
-            } else if (item.AlbumArtist && item.Type == "MusicAlbum") {
-
             } else if (item.Album) {
                 html.push(item.Album);
-
             }
 
             if (html.length) {
diff --git a/dashboard-ui/scripts/playlist.js b/dashboard-ui/scripts/playlist.js
index 81750ec790..35b47d278a 100644
--- a/dashboard-ui/scripts/playlist.js
+++ b/dashboard-ui/scripts/playlist.js
@@ -42,7 +42,7 @@
 
             html += '<td>';
             if (item.AlbumArtist) {
-                html += LibraryBrowser.getArtistLinksHtml([item.AlbumArtist]);
+                html += LibraryBrowser.getAlbumArtistLinksHtml(item.AlbumArtists || []);
             }
             html += '</td>';
 

From 66813c9b271751519c0247f933e6f1ca2019a66b Mon Sep 17 00:00:00 2001
From: Luke Pulverenti <luke.pulverenti@gmail.com>
Date: Fri, 13 Mar 2015 13:25:28 -0400
Subject: [PATCH 2/4] add ArtistItems to api output

---
 dashboard-ui/edititemmetadata.html           |  3 +-
 dashboard-ui/scripts/edititemmetadata.js     | 26 +++++++++++++----
 dashboard-ui/scripts/itembynamedetailpage.js | 18 ++++++------
 dashboard-ui/scripts/itemdetailpage.js       |  6 ++--
 dashboard-ui/scripts/librarybrowser.js       | 30 ++++++--------------
 dashboard-ui/scripts/mediaplayer.js          |  4 +--
 dashboard-ui/scripts/playlist.js             |  4 +--
 7 files changed, 46 insertions(+), 45 deletions(-)

diff --git a/dashboard-ui/edititemmetadata.html b/dashboard-ui/edititemmetadata.html
index df7f141a6c..e619286183 100644
--- a/dashboard-ui/edititemmetadata.html
+++ b/dashboard-ui/edititemmetadata.html
@@ -74,8 +74,7 @@
                             </div>
                             <div data-role="fieldcontain" id="fldAlbumArtist" style="display: none;">
                                 <label for="txtAlbumArtist">${LabelAlbumArtists}</label>
-                                <input type="text" id="txtAlbumArtist" />
-                                <input type="text" id="txtArtist" placeholder="${LabelArtistsHelp}" />
+                                <input type="text" id="txtAlbumArtist" placeholder="${LabelArtistsHelp}" />
                             </div>
                             <div data-role="fieldcontain" id="fldAlbum" style="display: none;">
                                 <label for="txtAlbum">${LabelAlbum}</label>
diff --git a/dashboard-ui/scripts/edititemmetadata.js b/dashboard-ui/scripts/edititemmetadata.js
index a3966737de..b4e202e2be 100644
--- a/dashboard-ui/scripts/edititemmetadata.js
+++ b/dashboard-ui/scripts/edititemmetadata.js
@@ -502,13 +502,19 @@
 
         $('#txtAlbum', page).val(item.Album || "");
 
-        var albumArtists = item.AlbumArtists || [];
-        $('#txtAlbumArtist', page).val(albumArtists.join(';'));
+        $('#txtAlbumArtist', page).val((item.AlbumArtists || []).map(function (a) {
+
+            return a.Name;
+
+        }).join(';'));
 
         $('#selectDisplayOrder', page).val(item.DisplayOrder).selectmenu('refresh');
 
-        var artists = item.Artists || [];
-        $('#txtArtist', page).val(artists.join(';'));
+        $('#txtArtist', page).val((item.ArtistItems || []).map(function (a) {
+
+            return a.Name;
+
+        }).join(';'));
 
         var date;
 
@@ -814,6 +820,16 @@
         });
     }
 
+    function getArtists(form) {
+
+        return $('#txtArtist', form).val().split(';').map(function (a) {
+
+            return {
+                Name: a
+            };
+        });
+    }
+
     function editItemMetadataPage() {
 
         var self = this;
@@ -847,7 +863,7 @@
                 Players: $('#txtPlayers', form).val(),
                 Album: $('#txtAlbum', form).val(),
                 AlbumArtist: getAlbumArtists(form),
-                Artists: $('#txtArtist', form).val().split(';'),
+                ArtistItems: getArtists(form),
                 Metascore: $('#txtMetascore', form).val(),
                 AwardSummary: $('#txtAwardSummary', form).val(),
                 Overview: $('#txtOverview', form).val(),
diff --git a/dashboard-ui/scripts/itembynamedetailpage.js b/dashboard-ui/scripts/itembynamedetailpage.js
index b40bb96eb7..5d03d0cbc6 100644
--- a/dashboard-ui/scripts/itembynamedetailpage.js
+++ b/dashboard-ui/scripts/itembynamedetailpage.js
@@ -229,7 +229,7 @@
                 MediaTypes: "",
                 IncludeItemTypes: "MusicVideo",
                 PersonTypes: "",
-                Artists: ""
+                ArtistIds: ""
             });
 
         });
@@ -241,7 +241,7 @@
                 MediaTypes: "",
                 IncludeItemTypes: "Movie",
                 PersonTypes: "",
-                Artists: ""
+                ArtistIds: ""
             });
 
         });
@@ -253,7 +253,7 @@
                 MediaTypes: "",
                 IncludeItemTypes: "Series",
                 PersonTypes: "",
-                Artists: ""
+                ArtistIds: ""
             });
         });
 
@@ -264,7 +264,7 @@
                 MediaTypes: "",
                 IncludeItemTypes: "Trailer",
                 PersonTypes: "",
-                Artists: ""
+                ArtistIds: ""
             });
         });
 
@@ -275,7 +275,7 @@
                 IncludeItemTypes: "",
                 MediaTypes: "Game",
                 PersonTypes: "",
-                Artists: ""
+                ArtistIds: ""
             });
         });
 
@@ -286,7 +286,7 @@
                 MediaTypes: "",
                 IncludeItemTypes: "Episode",
                 PersonTypes: "",
-                Artists: ""
+                ArtistIds: ""
             });
         });
 
@@ -297,7 +297,7 @@
                 MediaTypes: "",
                 IncludeItemTypes: "MusicAlbum",
                 PersonTypes: "",
-                Artists: ""
+                ArtistIds: ""
             });
         });
 
@@ -307,7 +307,7 @@
                 MediaTypes: "",
                 IncludeItemTypes: "Audio",
                 PersonTypes: "",
-                Artists: ""
+                ArtistIds: ""
             });
         });
     }
@@ -381,7 +381,7 @@
             query.Studios = currentItem.Name;
         }
         else if (currentItem.Type == "MusicArtist") {
-            query.Artists = currentItem.Name;
+            query.ArtistIds = currentItem.Id;
         }
     }
 
diff --git a/dashboard-ui/scripts/itemdetailpage.js b/dashboard-ui/scripts/itemdetailpage.js
index 041dfa6ef2..4d38cce679 100644
--- a/dashboard-ui/scripts/itemdetailpage.js
+++ b/dashboard-ui/scripts/itemdetailpage.js
@@ -406,8 +406,8 @@
             $('#players', page).hide();
         }
 
-        if (item.Artists && item.Artists.length && item.Type != "MusicAlbum") {
-            $('#artist', page).show().html(getArtistLinksHtml(item.Artists, context)).trigger('create');
+        if (item.ArtistItems && item.ArtistItems.length && item.Type != "MusicAlbum") {
+            $('#artist', page).show().html(getArtistLinksHtml(item.ArtistItems, context)).trigger('create');
         } else {
             $('#artist', page).hide();
         }
@@ -512,7 +512,7 @@
 
             var artist = artists[i];
 
-            html.push('<a class="textlink" href="itembynamedetails.html?context=' + context + '&musicartist=' + ApiClient.encodeName(artist) + '">' + artist + '</a>');
+            html.push('<a class="textlink" href="itembynamedetails.html?context=' + context + '&id=' + artist.Id + '">' + artist.Name + '</a>');
 
         }
 
diff --git a/dashboard-ui/scripts/librarybrowser.js b/dashboard-ui/scripts/librarybrowser.js
index 67a6f9dcf6..b4456a9009 100644
--- a/dashboard-ui/scripts/librarybrowser.js
+++ b/dashboard-ui/scripts/librarybrowser.js
@@ -214,24 +214,7 @@
             return counts.join(' • ');
         },
 
-        getArtistLinksHtml: function (artists) {
-
-            var html = [];
-
-            for (var i = 0, length = artists.length; i < length; i++) {
-
-                var artist = artists[i];
-
-                html.push('<a href="itembynamedetails.html?context=music&musicartist=' + ApiClient.encodeName(artist) + '">' + artist + '</a>');
-
-            }
-
-            html = html.join(' / ');
-
-            return html;
-        },
-
-        getAlbumArtistLinksHtml: function (artists, cssClass) {
+        getArtistLinksHtml: function (artists, cssClass) {
 
             var html = [];
 
@@ -815,7 +798,10 @@
                 textlines.push(displayName);
 
                 if (item.Type == 'Audio') {
-                    textlines.push(item.Artists.join(', ') || '&nbsp;');
+                    textlines.push(item.ArtistItems.map(function (a) {
+                        return a.Name;
+
+                    }).join(', ') || '&nbsp;');
                 }
 
                 if (item.Type == 'Game') {
@@ -1763,9 +1749,9 @@
             var contextParam = context ? ('&context=' + context) : '';
 
             if (item.AlbumArtists) {
-                html.push(LibraryBrowser.getAlbumArtistLinksHtml(item.AlbumArtists, "detailPageParentLink"));
-            } else if (item.Artists && item.Artists.length && item.Type == "MusicVideo") {
-                html.push('<a class="detailPageParentLink" href="itembynamedetails.html?context=music&musicartist=' + ApiClient.encodeName(item.Artists[0]) + contextParam + '">' + item.Artists[0] + '</a>');
+                html.push(LibraryBrowser.getArtistLinksHtml(item.AlbumArtists, "detailPageParentLink"));
+            } else if (item.ArtistItems && item.ArtistItems.length && item.Type == "MusicVideo") {
+                html.push(LibraryBrowser.getArtistLinksHtml(item.ArtistItems, "detailPageParentLink"));
             } else if (item.SeriesName && item.Type == "Episode") {
 
                 html.push('<a class="detailPageParentLink" href="itemdetails.html?id=' + item.SeriesId + contextParam + '">' + item.SeriesName + '</a>');
diff --git a/dashboard-ui/scripts/mediaplayer.js b/dashboard-ui/scripts/mediaplayer.js
index 4b2b59ef96..e796af7cd4 100644
--- a/dashboard-ui/scripts/mediaplayer.js
+++ b/dashboard-ui/scripts/mediaplayer.js
@@ -423,7 +423,7 @@
             else if (firstItem.Type == "MusicArtist") {
 
                 promise = self.getItemsForPlayback({
-                    Artists: firstItem.Name,
+                    ArtistIds: firstItem.Id,
                     Filters: "IsNotFolder",
                     Recursive: true,
                     SortBy: "SortName",
@@ -938,7 +938,7 @@
                 if (item.Type == "MusicArtist") {
 
                     query.MediaTypes = "Audio";
-                    query.Artists = item.Name;
+                    query.ArtistIds = item.Id;
 
                 }
                 else if (item.Type == "MusicGenre") {
diff --git a/dashboard-ui/scripts/playlist.js b/dashboard-ui/scripts/playlist.js
index 35b47d278a..c05663158c 100644
--- a/dashboard-ui/scripts/playlist.js
+++ b/dashboard-ui/scripts/playlist.js
@@ -37,12 +37,12 @@
             html += '</td>';
 
             html += '<td>';
-            html += LibraryBrowser.getArtistLinksHtml(item.Artists || []);
+            html += LibraryBrowser.getArtistLinksHtml(item.ArtistItems || []);
             html += '</td>';
 
             html += '<td>';
             if (item.AlbumArtist) {
-                html += LibraryBrowser.getAlbumArtistLinksHtml(item.AlbumArtists || []);
+                html += LibraryBrowser.getArtistLinksHtml(item.AlbumArtists || []);
             }
             html += '</td>';
 

From 7a5667a19813cfe13b3bfe3510a5c4c86fe1a663 Mon Sep 17 00:00:00 2001
From: Luke Pulverenti <luke.pulverenti@gmail.com>
Date: Fri, 13 Mar 2015 14:45:03 -0400
Subject: [PATCH 3/4] query for content by studio id

---
 dashboard-ui/scripts/itembynamedetailpage.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dashboard-ui/scripts/itembynamedetailpage.js b/dashboard-ui/scripts/itembynamedetailpage.js
index 5d03d0cbc6..d9a4530176 100644
--- a/dashboard-ui/scripts/itembynamedetailpage.js
+++ b/dashboard-ui/scripts/itembynamedetailpage.js
@@ -378,7 +378,7 @@
             query.Genres = currentItem.Name;
         }
         else if (currentItem.Type == "Studio") {
-            query.Studios = currentItem.Name;
+            query.StudioIds = currentItem.Id;
         }
         else if (currentItem.Type == "MusicArtist") {
             query.ArtistIds = currentItem.Id;

From 99bfe82587ad6fd22e27c1a19fdc4bc6a209c733 Mon Sep 17 00:00:00 2001
From: Luke Pulverenti <luke.pulverenti@gmail.com>
Date: Sat, 14 Mar 2015 00:50:23 -0400
Subject: [PATCH 4/4] add dynamic images

---
 dashboard-ui/scripts/indexpage.js | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/dashboard-ui/scripts/indexpage.js b/dashboard-ui/scripts/indexpage.js
index e23ffec121..2d0e21d821 100644
--- a/dashboard-ui/scripts/indexpage.js
+++ b/dashboard-ui/scripts/indexpage.js
@@ -462,7 +462,7 @@
         switch (index) {
 
             case 0:
-                return 'smalllibrarytiles-automobile';
+                return 'librarytiles-automobile';
             case 1:
                 return 'resume';
             case 2:
@@ -493,6 +493,9 @@
         else if (section == 'smalllibrarytiles-automobile') {
             Sections.loadLibraryTiles(elem, userId, 'homePageSmallBackdrop', index, true);
         }
+        else if (section == 'librarytiles-automobile') {
+            Sections.loadLibraryTiles(elem, userId, 'backdrop', index, true);
+        }
         else if (section == 'librarybuttons') {
             Sections.loadlibraryButtons(elem, userId, index);
         }