rework storage of PresentationUniqueKey

MediaBrowser.Controller/Entities/Audio/MusicArtist.cs

@@ -169,13 +169,9 @@ namespace MediaBrowser.Controller.Entities.Audio
             list.Add("Artist-" + (item.Name ?? string.Empty).RemoveDiacritics());
             return list;
         }
-
+        public override string CreatePresentationUniqueKey()
-        public override string PresentationUniqueKey
         {
-            get
+            return "Artist-" + (Name ?? string.Empty).RemoveDiacritics();
-            {
-                return "Artist-" + (Name ?? string.Empty).RemoveDiacritics();
-            }
         }
         protected override bool GetBlockUnratedValue(UserPolicy config)
         {

MediaBrowser.Controller/Entities/Audio/MusicGenre.cs

@@ -18,13 +18,9 @@ namespace MediaBrowser.Controller.Entities.Audio
             list.Insert(0, GetType().Name + "-" + (Name ?? string.Empty).RemoveDiacritics());
             return list;
         }
-
+        public override string CreatePresentationUniqueKey()
-        public override string PresentationUniqueKey
         {
-            get
+            return GetUserDataKeys()[0];
-            {
-                return GetUserDataKeys()[0];
-            }
         }
 
         [IgnoreDataMember]

MediaBrowser.Controller/Entities/BaseItem.cs

@@ -1194,10 +1194,17 @@ namespace MediaBrowser.Controller.Entities
             get { return null; }
         }
 
-        [IgnoreDataMember]
+        public virtual string CreatePresentationUniqueKey()
-        public virtual string PresentationUniqueKey
         {
-            get { return Id.ToString("N"); }
+            return Id.ToString("N");
+        }
+
+        [IgnoreDataMember]
+        public string PresentationUniqueKey { get; set; }
+
+        public string GetPresentationUniqueKey()
+        {
+            return PresentationUniqueKey ?? CreatePresentationUniqueKey();
         }
 
         public virtual bool RequiresRefresh()

MediaBrowser.Controller/Entities/GameGenre.cs

@@ -16,12 +16,9 @@ namespace MediaBrowser.Controller.Entities
             return list;
         }
 
-        public override string PresentationUniqueKey
+        public override string CreatePresentationUniqueKey()
         {
-            get
+            return GetUserDataKeys()[0];
-            {
-                return GetUserDataKeys()[0];
-            }
         }
 
         /// <summary>

MediaBrowser.Controller/Entities/Genre.cs

@@ -19,13 +19,9 @@ namespace MediaBrowser.Controller.Entities
             list.Insert(0, GetType().Name + "-" + (Name ?? string.Empty).RemoveDiacritics());
             return list;
         }
-
+        public override string CreatePresentationUniqueKey()
-        public override string PresentationUniqueKey
         {
-            get
+            return GetUserDataKeys()[0];
-            {
-                return GetUserDataKeys()[0];
-            }
         }
 
         /// <summary>

MediaBrowser.Controller/Entities/Person.cs

@@ -26,13 +26,9 @@ namespace MediaBrowser.Controller.Entities
             list.Insert(0, GetType().Name + "-" + (Name ?? string.Empty).RemoveDiacritics());
             return list;
         }
-
+        public override string CreatePresentationUniqueKey()
-        public override string PresentationUniqueKey
         {
-            get
+            return GetUserDataKeys()[0];
-            {
-                return GetUserDataKeys()[0];
-            }
         }
 
         public PersonLookupInfo GetLookupInfo()

MediaBrowser.Controller/Entities/Studio.cs

@@ -18,13 +18,9 @@ namespace MediaBrowser.Controller.Entities
             list.Insert(0, GetType().Name + "-" + (Name ?? string.Empty).RemoveDiacritics());
             return list;
         }
-
+        public override string CreatePresentationUniqueKey()
-        public override string PresentationUniqueKey
         {
-            get
+            return GetUserDataKeys()[0];
-            {
-                return GetUserDataKeys()[0];
-            }
         }
 
         /// <summary>

MediaBrowser.Controller/Entities/TV/Season.cs

@@ -114,22 +114,18 @@ namespace MediaBrowser.Controller.Entities.TV
             }
         }
 
-        [IgnoreDataMember]
+        public override string CreatePresentationUniqueKey()
-        public override string PresentationUniqueKey
         {
-            get
+            if (IndexNumber.HasValue)
             {
-                if (IndexNumber.HasValue)
+                var series = Series;
+                if (series != null)
                 {
-                    var series = Series;
+                    return series.PresentationUniqueKey + "-" + (IndexNumber ?? 0).ToString("000");
-                    if (series != null)
-                    {
-                        return series.PresentationUniqueKey + "-" + (IndexNumber ?? 0).ToString("000");
-                    }
                 }
-
-                return base.PresentationUniqueKey;
             }
+
+            return base.CreatePresentationUniqueKey();
         }
 
         /// <summary>

MediaBrowser.Controller/Entities/TV/Series.cs

@@ -96,19 +96,30 @@ namespace MediaBrowser.Controller.Entities.TV
             }
         }
 
-        [IgnoreDataMember]
+        public override string CreatePresentationUniqueKey()
-        public override string PresentationUniqueKey
         {
-            get
+            var userdatakeys = GetUserDataKeys();
-            {
-                var userdatakeys = GetUserDataKeys();
 
-                if (userdatakeys.Count > 1)
+            if (userdatakeys.Count > 1)
-                {
+            {
-                    return userdatakeys[0];
+                return AddLibrariesToPresentationUniqueKey(userdatakeys[0]);
-                }
-                return base.PresentationUniqueKey;
             }
+            return base.CreatePresentationUniqueKey();
+        }
+
+        private string AddLibrariesToPresentationUniqueKey(string key)
+        {
+            return key;
+            //var folders = LibraryManager.GetCollectionFolders(this)
+            //    .Select(i => i.Id.ToString("N"))
+            //    .ToArray();
+
+            //if (folders.Length == 0)
+            //{
+            //    return key;
+            //}
+
+            //return key + "-" + string.Join("-", folders);
         }
 
         private static string GetUniqueSeriesKey(BaseItem series)
@@ -117,7 +128,7 @@ namespace MediaBrowser.Controller.Entities.TV
             {
                 return series.Id.ToString("N");
             }
-            return series.PresentationUniqueKey;
+            return series.GetPresentationUniqueKey();
         }
 
         public override int GetChildCount(User user)

MediaBrowser.Controller/Entities/UserViewBuilder.cs

@@ -1198,7 +1198,7 @@ namespace MediaBrowser.Controller.Entities
         {
             var user = query.User;
 
-            items = items.DistinctBy(i => i.PresentationUniqueKey, StringComparer.OrdinalIgnoreCase);
+            items = items.DistinctBy(i => i.GetPresentationUniqueKey(), StringComparer.OrdinalIgnoreCase);
 
             if (query.SortBy.Length > 0)
             {

MediaBrowser.Controller/Entities/Video.cs

@@ -44,18 +44,14 @@ namespace MediaBrowser.Controller.Entities
             }
         }
 
-        [IgnoreDataMember]
+        public override string CreatePresentationUniqueKey()
-        public override string PresentationUniqueKey
         {
-            get
+            if (!string.IsNullOrWhiteSpace(PrimaryVersionId))
             {
-                if (!string.IsNullOrWhiteSpace(PrimaryVersionId))
+                return PrimaryVersionId;
-                {
-                    return PrimaryVersionId;
-                }
-
-                return base.PresentationUniqueKey;
             }
+
+            return base.CreatePresentationUniqueKey();
         }
 
         [IgnoreDataMember]

MediaBrowser.Server.Implementations/Persistence/SqliteItemRepository.cs

@@ -412,7 +412,8 @@ namespace MediaBrowser.Server.Implementations.Persistence
             "SeasonName",
             "SeasonId",
             "SeriesId",
-            "SeriesSortName"
+            "SeriesSortName",
+            "PresentationUniqueKey"
         };
 
         private readonly string[] _mediaStreamSaveColumns =
@@ -918,7 +919,7 @@ namespace MediaBrowser.Server.Implementations.Persistence
                         _saveItemCommand.GetParameter(index++).Value = GetCleanValue(item.Name);
                     }
 
-                    _saveItemCommand.GetParameter(index++).Value = item.PresentationUniqueKey;
+                    _saveItemCommand.GetParameter(index++).Value = item.GetPresentationUniqueKey();
                     _saveItemCommand.GetParameter(index++).Value = item.SlugName;
                     _saveItemCommand.GetParameter(index++).Value = item.OriginalTitle;
 
@@ -1454,6 +1455,12 @@ namespace MediaBrowser.Server.Implementations.Persistence
             }
             index++;
 
+            if (!reader.IsDBNull(index))
+            {
+                item.PresentationUniqueKey = reader.GetString(index);
+            }
+            index++;
+
             return item;
         }
 

MediaBrowser.Server.Implementations/TV/TVSeriesManager.cs

@@ -152,7 +152,7 @@ namespace MediaBrowser.Server.Implementations.TV
             {
                 return series.Id.ToString("N");
             }
-            return series.PresentationUniqueKey;
+            return series.GetPresentationUniqueKey();
         }
 
         /// <summary>