From 3f2da1441f2d9df8b02ee39500b13f777eefc4c9 Mon Sep 17 00:00:00 2001
From: geogolem <geogolem@gmail.com>
Date: Fri, 3 Mar 2017 14:25:03 -0500
Subject: [PATCH 1/3] always check exclusions with tmdbid

---
 .../NetImport/NetImportSearchService.cs       | 42 ++++++++++++-------
 1 file changed, 28 insertions(+), 14 deletions(-)

diff --git a/src/NzbDrone.Core/NetImport/NetImportSearchService.cs b/src/NzbDrone.Core/NetImport/NetImportSearchService.cs
index 2c3f257fc9..a8ddb396c0 100644
--- a/src/NzbDrone.Core/NetImport/NetImportSearchService.cs
+++ b/src/NzbDrone.Core/NetImport/NetImportSearchService.cs
@@ -25,11 +25,13 @@ public class NetImportSearchService : IFetchNetImport, IExecute<NetImportSyncCom
         private readonly ISearchForNewMovie _movieSearch;
         private readonly IRootFolderService _rootFolder;
         private readonly IConfigService _configService;
-        
+        private readonly IProvideMovieInfo _movieInfo;
+
 
         public NetImportSearchService(INetImportFactory netImportFactory, IMovieService movieService,
-            ISearchForNewMovie movieSearch, IRootFolderService rootFolder, IConfigService configService, Logger logger)
+            ISearchForNewMovie movieSearch, IRootFolderService rootFolder, IConfigService configService, IProvideMovieInfo movieInfo, Logger logger)
         {
+            _movieInfo = movieInfo;
             _netImportFactory = netImportFactory;
             _movieService = movieService;
             _movieSearch = movieSearch;
@@ -142,23 +144,35 @@ public void Execute(NetImportSyncCommand message)
             foreach (var movie in movies)
             {
                 bool shouldAdd = true;
-                if (importExclusions != null)
+                var mapped = _movieSearch.MapMovieToTmdbMovie(movie);
+                if (mapped != null)
                 {
-                    foreach (var exclusion in importExclusions)
+                    if (importExclusions != null)
                     {
-                        if (exclusion == movie.ImdbId || exclusion == movie.TmdbId.ToString())
+                        foreach (var exclusion in importExclusions)
                         {
-                            _logger.Info("Movie: {0} was found but will not be added because {1} was found on your exclusion list",movie, exclusion);
-                            shouldAdd = false;
-                            break;
+                            int tmdbId;
+                            if (exclusion.StartsWith("tt"))
+                            {
+                                var result = _movieInfo.GetMovieInfo(exclusion);
+                                tmdbId = result.TmdbId;
+                            }
+                            else
+                            {
+                                tmdbId = Int32.Parse(exclusion);
+                            }
+                            if (tmdbId == movie.TmdbId)
+                            {
+                                _logger.Info("Movie: {0} was found but will not be added because {1} was found on your exclusion list", movie, exclusion);
+                                shouldAdd = false;
+                                break;
+                            }
                         }
                     }
-                }
-
-                var mapped = _movieSearch.MapMovieToTmdbMovie(movie);
-                if ((mapped != null) && shouldAdd)
-                {
-                    _movieService.AddMovie(mapped);
+                    if (shouldAdd)
+                    {
+                        _movieService.AddMovie(mapped);
+                    }
                 }
             }
         }

From a3c0f4cb3f0e4b862fc4c1a962882167b0bbfad9 Mon Sep 17 00:00:00 2001
From: geogolem <geogolem@gmail.com>
Date: Fri, 3 Mar 2017 19:31:38 -0500
Subject: [PATCH 2/3] also use TMDBID on list sync

---
 src/NzbDrone.Core/NetImport/NetImportSearchService.cs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/NzbDrone.Core/NetImport/NetImportSearchService.cs b/src/NzbDrone.Core/NetImport/NetImportSearchService.cs
index a8ddb396c0..b616b293e6 100644
--- a/src/NzbDrone.Core/NetImport/NetImportSearchService.cs
+++ b/src/NzbDrone.Core/NetImport/NetImportSearchService.cs
@@ -94,7 +94,7 @@ public void Execute(NetImportSyncCommand message)
                     bool foundMatch = false;
                     foreach (var listedMovie in listedMovies)
                     {
-                        if (movie.ImdbId == listedMovie.ImdbId)
+                        if (movie.TmdbId == listedMovie.TmdbId)
                         {
                             foundMatch = true;
                             break;

From 3dd14c72c828a5da7842774245b2cd1616b1fda8 Mon Sep 17 00:00:00 2001
From: geogolem <geogolem@gmail.com>
Date: Sun, 5 Mar 2017 02:02:24 -0500
Subject: [PATCH 3/3] store titleSlug in tags for exclusions and always use
 TMDBID

---
 .../NetImport/NetImportSearchService.cs       | 37 +++++---------
 src/NzbDrone.Core/Tv/MovieService.cs          |  6 +--
 src/UI/JsLibraries/bootstrap.tagsinput.js     | 51 ++++++++++++++++---
 .../NetImport/Options/NetImportOptionsView.js |  9 +++-
 .../Options/NetImportOptionsViewTemplate.hbs  |  4 +-
 5 files changed, 72 insertions(+), 35 deletions(-)

diff --git a/src/NzbDrone.Core/NetImport/NetImportSearchService.cs b/src/NzbDrone.Core/NetImport/NetImportSearchService.cs
index b616b293e6..0bfe92b67f 100644
--- a/src/NzbDrone.Core/NetImport/NetImportSearchService.cs
+++ b/src/NzbDrone.Core/NetImport/NetImportSearchService.cs
@@ -25,13 +25,11 @@ public class NetImportSearchService : IFetchNetImport, IExecute<NetImportSyncCom
         private readonly ISearchForNewMovie _movieSearch;
         private readonly IRootFolderService _rootFolder;
         private readonly IConfigService _configService;
-        private readonly IProvideMovieInfo _movieInfo;
-
+        
 
         public NetImportSearchService(INetImportFactory netImportFactory, IMovieService movieService,
-            ISearchForNewMovie movieSearch, IRootFolderService rootFolder, IConfigService configService, IProvideMovieInfo movieInfo, Logger logger)
+            ISearchForNewMovie movieSearch, IRootFolderService rootFolder, IConfigService configService, Logger logger)
         {
-            _movieInfo = movieInfo;
             _netImportFactory = netImportFactory;
             _movieService = movieService;
             _movieSearch = movieSearch;
@@ -144,24 +142,15 @@ public void Execute(NetImportSyncCommand message)
             foreach (var movie in movies)
             {
                 bool shouldAdd = true;
-                var mapped = _movieSearch.MapMovieToTmdbMovie(movie);
-                if (mapped != null)
+                if (importExclusions != null)
                 {
-                    if (importExclusions != null)
+                    foreach (var exclusion in importExclusions)
                     {
-                        foreach (var exclusion in importExclusions)
+                        //var excludedTmdbId = exclusion.Substring(exclusion.LastIndexOf('-')+1);
+                        int excludedTmdbId;
+                        if (Int32.TryParse(exclusion.Substring(exclusion.LastIndexOf('-') + 1), out excludedTmdbId))
                         {
-                            int tmdbId;
-                            if (exclusion.StartsWith("tt"))
-                            {
-                                var result = _movieInfo.GetMovieInfo(exclusion);
-                                tmdbId = result.TmdbId;
-                            }
-                            else
-                            {
-                                tmdbId = Int32.Parse(exclusion);
-                            }
-                            if (tmdbId == movie.TmdbId)
+                            if (excludedTmdbId == movie.TmdbId)
                             {
                                 _logger.Info("Movie: {0} was found but will not be added because {1} was found on your exclusion list", movie, exclusion);
                                 shouldAdd = false;
@@ -169,10 +158,12 @@ public void Execute(NetImportSyncCommand message)
                             }
                         }
                     }
-                    if (shouldAdd)
-                    {
-                        _movieService.AddMovie(mapped);
-                    }
+                }
+
+                var mapped = _movieSearch.MapMovieToTmdbMovie(movie);
+                if ((mapped != null) && shouldAdd)
+                {
+                    _movieService.AddMovie(mapped);
                 }
             }
         }
diff --git a/src/NzbDrone.Core/Tv/MovieService.cs b/src/NzbDrone.Core/Tv/MovieService.cs
index 110f82a401..d535a1dc4d 100644
--- a/src/NzbDrone.Core/Tv/MovieService.cs
+++ b/src/NzbDrone.Core/Tv/MovieService.cs
@@ -244,11 +244,11 @@ public void DeleteMovie(int movieId, bool deleteFiles, bool addExclusion = false
             {
                 if (_configService.ImportExclusions.Empty())
                 {
-                    _configService.ImportExclusions = movie.ImdbId;
+                    _configService.ImportExclusions = movie.TitleSlug;
                 }
-                else if (!_configService.ImportExclusions.Contains(movie.ImdbId) && !_configService.ImportExclusions.Contains(movie.TmdbId.ToString()))
+                else if (!_configService.ImportExclusions.Contains(movie.TitleSlug))
                 {
-                    _configService.ImportExclusions += ',' + movie.ImdbId;
+                    _configService.ImportExclusions += ',' + movie.TitleSlug;
                 }
             }
             _movieRepository.Delete(movieId);
diff --git a/src/UI/JsLibraries/bootstrap.tagsinput.js b/src/UI/JsLibraries/bootstrap.tagsinput.js
index 93e7548a45..c48401bd8e 100644
--- a/src/UI/JsLibraries/bootstrap.tagsinput.js
+++ b/src/UI/JsLibraries/bootstrap.tagsinput.js
@@ -95,6 +95,37 @@
         }
       }
 
+	  /*
+      var itemValue = self.options.itemValue(item),
+          itemText = self.options.itemText(item),
+          tagClass = self.options.tagClass(item);
+
+      // Ignore items allready added
+      var existing = $.grep(self.itemsArray, function(item) { return self.options.itemValue(item) === itemValue; } )[0];
+      if (existing && !self.options.allowDuplicates) {
+        // Invoke onTagExists
+        if (self.options.onTagExists) {
+          var $existingTag = $(".tag", self.$container).filter(function() { return $(this).data("item") === existing; });
+          self.options.onTagExists(item, $existingTag);
+        }
+        return;
+      }
+
+      // if length greater than limit
+      if (self.items().toString().length + item.length + 1 > self.options.maxInputLength)
+        return;
+	  */
+      // raise beforeItemAdd arg
+      var beforeItemAddEvent = $.Event('beforeItemAdd', { item: item, cancel: false });
+      self.$element.trigger(beforeItemAddEvent);
+      if (beforeItemAddEvent.cancel)
+        return;
+
+      // register item in internal array and map
+      //self.itemsArray.push(item);
+
+	  // read var beforeItemAddEvent with new value
+	  var item = beforeItemAddEvent.item; // Get text from event (BeforeItemAddEvent)	  
       var itemValue = self.options.itemValue(item),
           itemText = self.options.itemText(item),
           tagClass = self.options.tagClass(item);
@@ -114,14 +145,22 @@
       if (self.items().toString().length + item.length + 1 > self.options.maxInputLength)
         return;
 
-      // raise beforeItemAdd arg
-      var beforeItemAddEvent = $.Event('beforeItemAdd', { item: item, cancel: false });
-      self.$element.trigger(beforeItemAddEvent);
-      if (beforeItemAddEvent.cancel)
-        return;
 
-      // register item in internal array and map
+	  // register item in internal array and map
       self.itemsArray.push(item);
+    	  
+	  if (beforeItemAddEvent.tagClass !== undefined){ var tagClass = beforeItemAddEvent.tagClass; }
+	  if (item != undefined){
+	  	var items = item.toString().split(',');	  
+  	  	if (items.length > 1) {			  
+   	  		for (var i = 0; i < items.length; i++) {							
+	  			this.add(items[i], true);
+ 	  		}
+	  		if (!dontPushVal)
+  	  			self.pushVal(self.options.triggerChange);
+      			return;
+	  		}	  
+	  }
 
       // add a tag element
       var $tag = $('<span class="tag ' + htmlEncode(tagClass) + '">' + htmlEncode(itemText) + '<span data-role="remove"></span></span>');
diff --git a/src/UI/Settings/NetImport/Options/NetImportOptionsView.js b/src/UI/Settings/NetImport/Options/NetImportOptionsView.js
index 13c5fc8e97..9b28af1158 100644
--- a/src/UI/Settings/NetImport/Options/NetImportOptionsView.js
+++ b/src/UI/Settings/NetImport/Options/NetImportOptionsView.js
@@ -90,12 +90,19 @@ var view = Marionette.ItemView.extend({
             });
             promise.success(function(response) {
                 event.cancel=false;
+
+				//var newText = response['tmdbId']+'-';
+				//if (event.item.startsWith('tt')) {
+				//	newText = newText+'['+event.item+']';
+				//}
+				event.item = response['titleSlug'];//+' ('+response['year']+')-'+response['tmdbId'];
             });
 
             promise.error(function(request, status, error) {
                 event.cancel = true;
                 window.alert(event.item+' is not a valid! Must be valid tt#### IMDB ID or #### TMDB ID');
-            });
+			});
+			return event;
         });
     },
 
diff --git a/src/UI/Settings/NetImport/Options/NetImportOptionsViewTemplate.hbs b/src/UI/Settings/NetImport/Options/NetImportOptionsViewTemplate.hbs
index 1cfc5b6dcf..bd429de47f 100644
--- a/src/UI/Settings/NetImport/Options/NetImportOptionsViewTemplate.hbs
+++ b/src/UI/Settings/NetImport/Options/NetImportOptionsViewTemplate.hbs
@@ -32,11 +32,11 @@
 	          </div>
 		  <div class="form-group">
 		                <label class="col-sm-3 control-label">Import Exclusions</label>
-				<div class="col-sm-1 col-sm-push-2 help-inline">
+				<div class="col-sm-1 col-sm-push-4 help-inline">
 				<i class="icon-sonarr-form-warning" title="Movies in this field will not be imported even if they exist on your lists."/>
 				<i class="icon-sonarr-form-info" title="Comma separated imdbid or tmdbid: tt0120915,216138,tt0121765"/>
 				</div>
-				<div class="col-sm-2 col-sm-pull-1">
+				<div class="col-sm-4 col-sm-pull-1">
 				                
 				                <input type="text" name="importExclusions" class="form-control x-import-exclusions"/>
 				</div>