From 33b8c869483da9654b8365f92de1a6ade8564880 Mon Sep 17 00:00:00 2001
From: Luke Pulverenti <luke.pulverenti@gmail.com>
Date: Thu, 27 Jun 2013 14:40:50 -0400
Subject: [PATCH] hide people tab for ibn items

---
 dashboard-ui/edititemimages.html         |  2 +-
 dashboard-ui/edititemmetadata.html       | 21 ++++++++++-----------
 dashboard-ui/scripts/edititemimages.js   |  6 ++++++
 dashboard-ui/scripts/edititemmetadata.js |  6 ++++++
 4 files changed, 23 insertions(+), 12 deletions(-)

diff --git a/dashboard-ui/edititemimages.html b/dashboard-ui/edititemimages.html
index 0cb91e04bc..bf89bfc269 100644
--- a/dashboard-ui/edititemimages.html
+++ b/dashboard-ui/edititemimages.html
@@ -14,7 +14,7 @@
                 <br />
                 <div data-role="controlgroup" data-type="horizontal" class="localnav" data-mini="true">
                     <a href="#" data-role="button" onclick="Dashboard.navigate('edititemmetadata.html', true);">Metadata</a>
-                    <a href="#" data-role="button" onclick="Dashboard.navigate('edititempeople.html', true);">People</a>
+                    <a id="peopleTab" style="display: none;" href="#" data-role="button" onclick="Dashboard.navigate('edititempeople.html', true);">People</a>
                     <a href="#" data-role="button" class="ui-btn-active">Images</a>
                 </div>
 
diff --git a/dashboard-ui/edititemmetadata.html b/dashboard-ui/edititemmetadata.html
index 6d79159b28..7cf538d6fb 100644
--- a/dashboard-ui/edititemmetadata.html
+++ b/dashboard-ui/edititemmetadata.html
@@ -13,7 +13,7 @@
                 <br />
                 <div data-role="controlgroup" data-type="horizontal" class="localnav" data-mini="true">
                     <a href="#" data-role="button" class="ui-btn-active">Metadata</a>
-                    <a href="#" data-role="button" onclick="Dashboard.navigate('edititempeople.html', true);">People</a>
+                    <a id="peopleTab" style="display: none;" href="#" data-role="button" onclick="Dashboard.navigate('edititempeople.html', true);">People</a>
                     <a href="#" data-role="button" onclick="Dashboard.navigate('edititemimages.html', true);">Images</a>
                 </div>
 
@@ -95,7 +95,7 @@
                                     <label for="selectAirDays">Air Days:</label>
                                     <div data-role="editableListviewContainer" style="width: 78%; display: inline-block;">
                                         <div>
-                                            <select name="selectAirDays" Id="selectAirDays" data-mini="true">
+                                            <select name="selectAirDays" id="selectAirDays" data-mini="true">
                                                 <option value=''></option>
                                                 <option value='Monday'>Monday</option>
                                                 <option value='Tuesday'>Tuesday</option>
@@ -104,12 +104,12 @@
                                                 <option value='Friday'>Friday</option>
                                                 <option value='Saturday'>Saturday</option>
                                                 <option value='Sunday'>Sunday</option>
-                                            </select>  
-                                            <a data-role="button" data-theme="b" data-inline="true" data-mini="true" data-icon="plus"  onclick="EditItemMetadataPage.addElementToEditableListview(this,EditItemMetadataPage.sortDaysOfTheWeek)">Add</a>
+                                            </select>
+                                            <a data-role="button" data-theme="b" data-inline="true" data-mini="true" data-icon="plus" onclick="EditItemMetadataPage.addElementToEditableListview(this,EditItemMetadataPage.sortDaysOfTheWeek)">Add</a>
                                         </div>
                                         <ul data-role="listview" data-inset="true" data-split-icon="delete" id="listAirDays"></ul>
                                     </div>
-                                    
+
                                 </div>
                                 <div data-role="fieldcontain" id="fldAirTime" style="display: none;">
                                     <label for="txtAirTime">Air Time:</label>
@@ -149,8 +149,8 @@
                             <h3>Genres</h3>
                             <div data-role="editableListviewContainer">
                                 <div data-role="fieldcontain">
-                                    <input type="text" data-mini="true" />    
-                                    <a data-role="button" data-theme="b" data-inline="true" data-mini="true" data-icon="plus"  onclick="editItemMetadataPage.addElementToEditableListview(this)">Add</a>
+                                    <input type="text" data-mini="true" />
+                                    <a data-role="button" data-theme="b" data-inline="true" data-mini="true" data-icon="plus" onclick="editItemMetadataPage.addElementToEditableListview(this)">Add</a>
                                 </div>
                                 <ul data-role="listview" data-inset="true" data-split-icon="delete" id="listGenres"></ul>
                             </div>
@@ -159,7 +159,7 @@
                             <h3>Studios</h3>
                             <div data-role="editableListviewContainer">
                                 <div data-role="fieldcontain">
-                                    <input type="text" data-mini="true" />    
+                                    <input type="text" data-mini="true" />
                                     <a data-role="button" data-theme="b" data-inline="true" data-mini="true" data-icon="plus" onclick="EditItemMetadataPage.addElementToEditableListview(this)">Add</a>
                                 </div>
                                 <ul data-role="listview" data-inset="true" data-split-icon="delete" id="listStudios"></ul>
@@ -169,7 +169,7 @@
                             <h3>Tags</h3>
                             <div data-role="editableListviewContainer">
                                 <div data-role="fieldcontain">
-                                    <input type="text" data-mini="true" />    
+                                    <input type="text" data-mini="true" />
                                     <a data-role="button" data-theme="b" data-inline="true" data-mini="true" data-icon="plus" onclick="EditItemMetadataPage.addElementToEditableListview(this)">Add</a>
                                 </div>
                                 <ul data-role="listview" data-inset="true" data-split-icon="delete" id="listTags"></ul>
@@ -216,11 +216,10 @@
                             <h3>Internet Provider Settings</h3>
                             <div>
                                 <div data-role="fieldcontain">
-                                    <input type="checkbox" name="enableInternetProviders" id="enableInternetProviders" onchange="EditItemMetadataPage.setProviderSettingsContainerVisibility(this)"/>
+                                    <input type="checkbox" name="enableInternetProviders" id="enableInternetProviders" onchange="EditItemMetadataPage.setProviderSettingsContainerVisibility(this)" />
                                     <label for="enableInternetProviders">Enable internet metadata providers for this item.</label>
                                 </div>
                                 <div id="providerSettingsContainer" style="display: none">
-                                    
                                 </div>
                             </div>
                         </div>
diff --git a/dashboard-ui/scripts/edititemimages.js b/dashboard-ui/scripts/edititemimages.js
index 7689521890..a319c8223d 100644
--- a/dashboard-ui/scripts/edititemimages.js
+++ b/dashboard-ui/scripts/edititemimages.js
@@ -52,6 +52,12 @@
             LibraryBrowser.renderName(item, $('.itemName', page), true);
             LibraryBrowser.renderParentName(item, $('.parentName', page));
 
+            if (item.Type == "Person" || item.Type == "Studio" || item.Type == "MusicGenre" || item.Type == "Genre" || item.Type == "Artist") {
+                $('#peopleTab', page).hide();
+            } else {
+                $('#peopleTab', page).show();
+            }
+
             ApiClient.getItemImageInfos(currentItem.Id).done(function (imageInfos) {
                 renderStandardImages(page, item, imageInfos);
                 renderBackdrops(page, item, imageInfos);
diff --git a/dashboard-ui/scripts/edititemmetadata.js b/dashboard-ui/scripts/edititemmetadata.js
index ae272df623..b5ce383ebe 100644
--- a/dashboard-ui/scripts/edititemmetadata.js
+++ b/dashboard-ui/scripts/edititemmetadata.js
@@ -63,6 +63,12 @@
 
             setFieldVisibilities(page, item);
             fillItemInfo(page, item);
+            
+            if (item.Type == "Person" || item.Type == "Studio" || item.Type == "MusicGenre" || item.Type == "Genre" || item.Type == "Artist") {
+                $('#peopleTab', page).hide();
+            } else {
+                $('#peopleTab', page).show();
+            }
 
             Dashboard.hideLoadingMsg();
         });