From 39b162419cd29596009944032398e1a0407c1208 Mon Sep 17 00:00:00 2001
From: Prinz23 <Prinz2311@gmail.com>
Date: Tue, 11 Jul 2017 10:29:55 +0100
Subject: [PATCH] Change editshow saving empty scene exceptions.

Change replace invalid scene exceptions table season column with -1.
Fix already saved invalid scene_exceptions.
---
 CHANGES.md                    | 1 +
 gui/slick/js/editShow.js      | 2 ++
 sickbeard/databases/mainDB.py | 9 +++++++++
 sickbeard/name_cache.py       | 3 ++-
 4 files changed, 14 insertions(+), 1 deletion(-)

diff --git a/CHANGES.md b/CHANGES.md
index d32f6dfc..7f59900b 100644
--- a/CHANGES.md
+++ b/CHANGES.md
@@ -62,6 +62,7 @@
 * Change catch show update task errors
 * Change simplify and update FreeBSD init script
 * Change only use newznab Api key if needed
+* Change editshow saving empty scene exceptions
 
 
 [develop changelog]
diff --git a/gui/slick/js/editShow.js b/gui/slick/js/editShow.js
index a40de222..0c30250a 100644
--- a/gui/slick/js/editShow.js
+++ b/gui/slick/js/editShow.js
@@ -67,6 +67,8 @@ $(document).ready(function () {
 		$('#SceneException').fadeIn('fast', 'linear');
 
 		var option = $('<option>');
+		if (null == sceneExSeason)
+			sceneExSeason = '-1';
 		option.val(sceneExSeason + '|' + sceneEx);
 		option.html((config.showIsAnime ? 'S' + ('-1' === sceneExSeason ? '*' : sceneExSeason) + ': ' : '') + sceneEx);
 
diff --git a/sickbeard/databases/mainDB.py b/sickbeard/databases/mainDB.py
index 44f0a5ee..c00b0b80 100644
--- a/sickbeard/databases/mainDB.py
+++ b/sickbeard/databases/mainDB.py
@@ -37,6 +37,7 @@ class MainSanityCheck(db.DBSanityCheck):
         self.fix_duplicate_episodes()
         self.fix_orphan_episodes()
         self.fix_unaired_episodes()
+        self.fix_scene_exceptions()
 
     def fix_duplicate_shows(self, column='indexer_id'):
 
@@ -159,6 +160,14 @@ class MainSanityCheck(db.DBSanityCheck):
         else:
             logger.log(u'No UNAIRED episodes, check passed')
 
+    def fix_scene_exceptions(self):
+
+        sql_results = self.connection.select(
+            'SELECT exception_id FROM scene_exceptions WHERE season = "null"')
+
+        if 0 < len(sql_results):
+            logger.log('Fixing invalid scene exceptions')
+            self.connection.action('UPDATE scene_exceptions SET season = -1 WHERE season = "null"')
 
 # ======================
 # = Main DB Migrations =
diff --git a/sickbeard/name_cache.py b/sickbeard/name_cache.py
index 1a5015af..e98645ca 100644
--- a/sickbeard/name_cache.py
+++ b/sickbeard/name_cache.py
@@ -18,6 +18,7 @@
 import threading
 import sickbeard
 from sickbeard import db
+from sickbeard.helpers import tryInt
 
 nameCache = {}
 nameCacheLock = threading.Lock()
@@ -83,7 +84,7 @@ def buildNameCache(show=None):
         if cache_results:
             for cache_result in cache_results:
                 indexer_id = int(cache_result['indexer_id'])
-                season = int(cache_result['season'])
+                season = tryInt(cache_result['season'], -1)
                 name = sickbeard.helpers.full_sanitizeSceneName(cache_result['show_name'])
                 nameCache[name] = [indexer_id, season]