From 5a5ec56328ef3a6ac551543e17a0f0b0e0492cef Mon Sep 17 00:00:00 2001
From: Luke Pulverenti <luke.pulverenti@gmail.com>
Date: Sun, 4 Jan 2015 09:34:15 -0500
Subject: [PATCH] adjust imdb id parsing

---
 .../EntryPoints/UsageEntryPoint.cs            |  3 +-
 .../Library/PathExtensions.cs                 |  9 ++--
 .../Library/Resolvers/Movies/MovieResolver.cs | 43 ++++++++++---------
 3 files changed, 29 insertions(+), 26 deletions(-)

diff --git a/MediaBrowser.Server.Implementations/EntryPoints/UsageEntryPoint.cs b/MediaBrowser.Server.Implementations/EntryPoints/UsageEntryPoint.cs
index 0e99ee673e..dac0b0d150 100644
--- a/MediaBrowser.Server.Implementations/EntryPoints/UsageEntryPoint.cs
+++ b/MediaBrowser.Server.Implementations/EntryPoints/UsageEntryPoint.cs
@@ -29,13 +29,14 @@ namespace MediaBrowser.Server.Implementations.EntryPoints
 
         private readonly ConcurrentDictionary<Guid, ClientInfo> _apps = new ConcurrentDictionary<Guid, ClientInfo>();
 
-        public UsageEntryPoint(ILogger logger, IApplicationHost applicationHost, INetworkManager networkManager, IHttpClient httpClient, ISessionManager sessionManager)
+        public UsageEntryPoint(ILogger logger, IApplicationHost applicationHost, INetworkManager networkManager, IHttpClient httpClient, ISessionManager sessionManager, IUserManager userManager)
         {
             _logger = logger;
             _applicationHost = applicationHost;
             _networkManager = networkManager;
             _httpClient = httpClient;
             _sessionManager = sessionManager;
+            _userManager = userManager;
 
             _sessionManager.SessionStarted += _sessionManager_SessionStarted;
         }
diff --git a/MediaBrowser.Server.Implementations/Library/PathExtensions.cs b/MediaBrowser.Server.Implementations/Library/PathExtensions.cs
index 822f41da22..6c0e3237e8 100644
--- a/MediaBrowser.Server.Implementations/Library/PathExtensions.cs
+++ b/MediaBrowser.Server.Implementations/Library/PathExtensions.cs
@@ -33,13 +33,12 @@ namespace MediaBrowser.Server.Implementations.Library
                 return str.Substring(start, end - start);
             }
             // for imdbid we also accept pattern matching
-            if (attrib == "imdbid")
+            if (string.Equals(attrib, "imdbid", StringComparison.OrdinalIgnoreCase))
             {
-              Regex imdbPattern = new Regex("tt\\d{7}");
-              var m = imdbPattern.Match(str);
-              return m.Success ? m.Value : null;            
+                var m = Regex.Match(str, "tt\\d{7}", RegexOptions.IgnoreCase);
+                return m.Success ? m.Value : null;
             }
-            
+
             return null;
         }
     }
diff --git a/MediaBrowser.Server.Implementations/Library/Resolvers/Movies/MovieResolver.cs b/MediaBrowser.Server.Implementations/Library/Resolvers/Movies/MovieResolver.cs
index 5ecf7759d0..441fada6ab 100644
--- a/MediaBrowser.Server.Implementations/Library/Resolvers/Movies/MovieResolver.cs
+++ b/MediaBrowser.Server.Implementations/Library/Resolvers/Movies/MovieResolver.cs
@@ -19,7 +19,8 @@ namespace MediaBrowser.Server.Implementations.Library.Resolvers.Movies
     /// </summary>
     public class MovieResolver : BaseVideoResolver<Video>, IMultiItemResolver
     {
-        public MovieResolver(ILibraryManager libraryManager) : base(libraryManager)
+        public MovieResolver(ILibraryManager libraryManager)
+            : base(libraryManager)
         {
         }
 
@@ -39,7 +40,7 @@ namespace MediaBrowser.Server.Implementations.Library.Resolvers.Movies
             }
         }
 
-        public MultiItemResolverResult ResolveMultiple(Folder parent, 
+        public MultiItemResolverResult ResolveMultiple(Folder parent,
             List<FileSystemInfo> files,
             string collectionType,
             IDirectoryService directoryService)
@@ -237,25 +238,27 @@ namespace MediaBrowser.Server.Implementations.Library.Resolvers.Movies
         /// <param name="item">The item.</param>
         private void SetProviderIdsFromPath(Video item)
         {
-            //we need to only look at the name of this actual item (not parents)
-            var justName = item.IsInMixedFolder ? Path.GetFileName(item.Path) : Path.GetFileName(item.ContainingFolderPath);
-
-            // check for tmdb id
-            var tmdbid = justName.GetAttributeValue("tmdbid");
-
-            if (!string.IsNullOrEmpty(tmdbid))
+            if (item is Movie || item is MusicVideo)
             {
-                item.SetProviderId(MetadataProviders.Tmdb, tmdbid);
+                //we need to only look at the name of this actual item (not parents)
+                var justName = item.IsInMixedFolder ? Path.GetFileName(item.Path) : Path.GetFileName(item.ContainingFolderPath);
+
+                // check for tmdb id
+                var tmdbid = justName.GetAttributeValue("tmdbid");
+
+                if (!string.IsNullOrEmpty(tmdbid))
+                {
+                    item.SetProviderId(MetadataProviders.Tmdb, tmdbid);
+                }
+
+                // check for imdb id - we use full media path, as we can assume, that this will match in any use case (wither id in parent dir or in file name)
+                var imdbid = item.Path.GetAttributeValue("imdbid");
+
+                if (!string.IsNullOrEmpty(imdbid))
+                {
+                    item.SetProviderId(MetadataProviders.Imdb, imdbid);
+                }
             }
-
-            // check for imdb id - we use full media path, as we can assume, that this will match in any use case (wither id in parent dir or in file name)
-            var imdbid = item.Path.GetAttributeValue("imdbid");
-
-            if (!string.IsNullOrEmpty(imdbid))
-            {
-              item.SetProviderId(MetadataProviders.Imdb, imdbid);
-            }
-
         }
 
         /// <summary>
@@ -272,7 +275,7 @@ namespace MediaBrowser.Server.Implementations.Library.Resolvers.Movies
             where T : Video, new()
         {
             var multiDiscFolders = new List<FileSystemInfo>();
-            
+
             // Search for a folder rip
             foreach (var child in fileSystemEntries)
             {