From c2c081c8e4b5350d449ffc25389ef90483078f52 Mon Sep 17 00:00:00 2001
From: Luke Pulverenti <luke.pulverenti@gmail.com>
Date: Sun, 28 Apr 2013 01:44:45 -0400
Subject: fixes #206 - Adding roles, deleting actors, changing names - not
 being picked up

---
 MediaBrowser.Controller/Entities/BaseItem.cs | 82 ++--------------------------
 1 file changed, 4 insertions(+), 78 deletions(-)

(limited to 'MediaBrowser.Controller/Entities')

diff --git a/MediaBrowser.Controller/Entities/BaseItem.cs b/MediaBrowser.Controller/Entities/BaseItem.cs
index 805cb7bdfa..eca0ea4629 100644
--- a/MediaBrowser.Controller/Entities/BaseItem.cs
+++ b/MediaBrowser.Controller/Entities/BaseItem.cs
@@ -31,6 +31,7 @@ namespace MediaBrowser.Controller.Entities
             Studios = new List<string>();
             People = new List<PersonInfo>();
             CriticReviews = new List<ItemReview>();
+            Taglines = new List<string>();
         }
 
         /// <summary>
@@ -880,10 +881,10 @@ namespace MediaBrowser.Controller.Entities
             OfficialRating = null;
             CustomRating = null;
             Overview = null;
-            Taglines = null;
+            Taglines.Clear();
             Language = null;
-            Studios = null;
-            Genres = null;
+            Studios.Clear();
+            Genres.Clear();
             CommunityRating = null;
             RunTimeTicks = null;
             AspectRatio = null;
@@ -1052,24 +1053,6 @@ namespace MediaBrowser.Controller.Entities
             return (DateTime.UtcNow - DateCreated).TotalDays < ConfigurationManager.Configuration.RecentItemDays;
         }
 
-        /// <summary>
-        /// Adds people to the item
-        /// </summary>
-        /// <param name="people">The people.</param>
-        /// <exception cref="System.ArgumentNullException"></exception>
-        public void AddPeople(IEnumerable<PersonInfo> people)
-        {
-            if (people == null)
-            {
-                throw new ArgumentNullException();
-            }
-
-            foreach (var person in people)
-            {
-                AddPerson(person);
-            }
-        }
-
         /// <summary>
         /// Adds a person to the item
         /// </summary>
@@ -1087,12 +1070,6 @@ namespace MediaBrowser.Controller.Entities
                 throw new ArgumentNullException();
             }
 
-            if (People == null)
-            {
-                People = new List<PersonInfo> { person };
-                return;
-            }
-
             // If the type is GuestStar and there's already an Actor entry, then update it to avoid dupes
             if (string.Equals(person.Type, PersonType.GuestStar, StringComparison.OrdinalIgnoreCase))
             {
@@ -1123,24 +1100,6 @@ namespace MediaBrowser.Controller.Entities
             }
         }
 
-        /// <summary>
-        /// Adds studios to the item
-        /// </summary>
-        /// <param name="studios">The studios.</param>
-        /// <exception cref="System.ArgumentNullException"></exception>
-        public void AddStudios(IEnumerable<string> studios)
-        {
-            if (studios == null)
-            {
-                throw new ArgumentNullException();
-            }
-
-            foreach (var name in studios)
-            {
-                AddStudio(name);
-            }
-        }
-
         /// <summary>
         /// Adds a studio to the item
         /// </summary>
@@ -1153,11 +1112,6 @@ namespace MediaBrowser.Controller.Entities
                 throw new ArgumentNullException("name");
             }
 
-            if (Studios == null)
-            {
-                Studios = new List<string>();
-            }
-
             if (!Studios.Contains(name, StringComparer.OrdinalIgnoreCase))
             {
                 Studios.Add(name);
@@ -1176,11 +1130,6 @@ namespace MediaBrowser.Controller.Entities
                 throw new ArgumentNullException("name");
             }
 
-            if (Taglines == null)
-            {
-                Taglines = new List<string>();
-            }
-
             if (!Taglines.Contains(name, StringComparer.OrdinalIgnoreCase))
             {
                 Taglines.Add(name);
@@ -1222,11 +1171,6 @@ namespace MediaBrowser.Controller.Entities
                 throw new ArgumentNullException("name");
             }
 
-            if (Genres == null)
-            {
-                Genres = new List<string>();
-            }
-
             if (!Genres.Contains(name, StringComparer.OrdinalIgnoreCase))
             {
                 Genres.Add(name);
@@ -1256,24 +1200,6 @@ namespace MediaBrowser.Controller.Entities
             }
         }
 
-        /// <summary>
-        /// Adds genres to the item
-        /// </summary>
-        /// <param name="genres">The genres.</param>
-        /// <exception cref="System.ArgumentNullException"></exception>
-        public void AddGenres(IEnumerable<string> genres)
-        {
-            if (genres == null)
-            {
-                throw new ArgumentNullException();
-            }
-
-            foreach (var name in genres)
-            {
-                AddGenre(name);
-            }
-        }
-
         /// <summary>
         /// Marks the item as either played or unplayed
         /// </summary>
-- 
cgit v1.2.3