From bdac98db4b8d8962fa3d9deea200c42899c01b73 Mon Sep 17 00:00:00 2001
From: KontiSR <sickrage@headshots.nl>
Date: Mon, 15 Sep 2014 15:01:18 +0200
Subject: [PATCH] Implemented the queuing functionality also for Failed
 downloads. ajaxEpSearch.js now also processes the clicks for Retries. SO
 merged those together and removed ajaxEpRetry.js.

---
 gui/slick/interfaces/default/displayShow.tmpl |  1 -
 gui/slick/js/ajaxEpRetry.js                   | 51 -------------------
 gui/slick/js/ajaxEpSearch.js                  |  2 +-
 gui/slick/js/displayShow.js                   |  1 -
 sickbeard/search_queue.py                     |  5 ++
 sickbeard/webserve.py                         | 27 +++-------
 6 files changed, 14 insertions(+), 73 deletions(-)
 delete mode 100644 gui/slick/js/ajaxEpRetry.js

diff --git a/gui/slick/interfaces/default/displayShow.tmpl b/gui/slick/interfaces/default/displayShow.tmpl
index f38aa998..be41bd28 100644
--- a/gui/slick/interfaces/default/displayShow.tmpl
+++ b/gui/slick/interfaces/default/displayShow.tmpl
@@ -26,7 +26,6 @@
 <script type="text/javascript" src="$sbRoot/js/sceneExceptionsTooltip.js?$sbPID"></script>
 <script type="text/javascript" src="$sbRoot/js/ajaxEpSearch.js?$sbPID"></script>
 <script type="text/javascript" src="$sbRoot/js/ajaxEpSubtitles.js?$sbPID"></script>
-<script type="text/javascript" src="$sbRoot/js/ajaxEpRetry.js?$sbPID"></script>
 <script type="text/javascript" charset="utf-8">
 <!--
 \$(document).ready(function(){
diff --git a/gui/slick/js/ajaxEpRetry.js b/gui/slick/js/ajaxEpRetry.js
deleted file mode 100644
index 1f10c3e5..00000000
--- a/gui/slick/js/ajaxEpRetry.js
+++ /dev/null
@@ -1,51 +0,0 @@
-(function () {
-
-    $.ajaxEpRetry = {
-        defaults: {
-            size:               16,
-            colorRow:           false,
-            loadingImage:       'loading16_dddddd.gif',
-            noImage:            'no16.png',
-            yesImage:           'yes16.png'
-        }
-    };
-
-    $.fn.ajaxEpRetry = function (options) {
-        options = $.extend({}, $.ajaxEpRetry.defaults, options);
-
-        $('.epRetry').click(function () {
-            if ( !confirm("Mark download as bad and retry?") )
-                return false;
-
-            var parent = $(this).parent();
-
-            // put the ajax spinner (for non white bg) placeholder while we wait
-            parent.empty();
-            parent.append($("<img/>").attr({"src": sbRoot + "/images/" + options.loadingImage, "height": options.size, "alt": "", "title": "loading"}));
-
-            $.getJSON($(this).attr('href'), function (data) {
-                // if they failed then just put the red X
-                if (data.result == 'failure') {
-                    img_name = options.noImage;
-                    img_result = 'failed';
-
-                // if the snatch was successful then apply the corresponding class and fill in the row appropriately
-                } else {
-                    img_name = options.yesImage;
-                    img_result = 'success';
-                    // color the row
-                    if (options.colorRow) {
-                        parent.parent().removeClass('skipped wanted qual good unaired snatched').addClass('snatched');
-                    }
-                }
-
-                // put the corresponding image as the result for the the row
-                parent.empty();
-                parent.append($("<img/>").attr({"src": sbRoot + "/images/" + img_name, "height": options.size, "alt": img_result, "title": img_result}));
-            });
-
-            // don't follow the link
-            return false;
-        });
-    };
-})();
diff --git a/gui/slick/js/ajaxEpSearch.js b/gui/slick/js/ajaxEpSearch.js
index ffe6f6f0..c3c33bdc 100644
--- a/gui/slick/js/ajaxEpSearch.js
+++ b/gui/slick/js/ajaxEpSearch.js
@@ -123,7 +123,7 @@ function disableLink(el) {
 	$.fn.ajaxEpSearch = function(options){
 		options = $.extend({}, $.ajaxEpSearch.defaults, options);
 		
-	    $('.epSearch').click(function(event){
+	    $('.epSearch, .epRetry').click(function(event){
 	    	event.preventDefault();
 	        
 	    	// Check if we have disabled the click
diff --git a/gui/slick/js/displayShow.js b/gui/slick/js/displayShow.js
index 0abf94d9..ec77a195 100644
--- a/gui/slick/js/displayShow.js
+++ b/gui/slick/js/displayShow.js
@@ -1,7 +1,6 @@
 $(document).ready(function () {
 
     $('#sbRoot').ajaxEpSearch({'colorRow': true});
-    //$('#sbRoot').ajaxEpRetry({'colorRow': true});
 
     $('#sbRoot').ajaxEpSubtitlesSearch();
 
diff --git a/sickbeard/search_queue.py b/sickbeard/search_queue.py
index 27b429dd..6d196258 100644
--- a/sickbeard/search_queue.py
+++ b/sickbeard/search_queue.py
@@ -241,12 +241,14 @@ class FailedQueueItem(generic_queue.QueueItem):
         self.show = show
         self.segment = segment
         self.success = None
+        self.started = None
 
     def run(self):
         generic_queue.QueueItem.run(self)
 
         try:
             logger.log(u"Marking episode as bad: [" + self.segment.prettyName() + "]")
+            self.started = True
             failed_history.markFailed(self.segment)
 
             (release, provider) = failed_history.findRelease(self.segment)
@@ -271,6 +273,9 @@ class FailedQueueItem(generic_queue.QueueItem):
                 logger.log(u"No valid episode found to retry for: [" + self.segment.prettyName() + "]")
         except Exception:
             logger.log(traceback.format_exc(), logger.DEBUG)
+            
+        ### Keep a list with the 100 last executed searches
+        fifo(MANUAL_SEARCH_HISTORY, self, MANUAL_SEARCH_HISTORY_SIZE)
 
         if self.success is None:
             self.success = False
diff --git a/sickbeard/webserve.py b/sickbeard/webserve.py
index eeb32fc7..f391a544 100644
--- a/sickbeard/webserve.py
+++ b/sickbeard/webserve.py
@@ -4510,26 +4510,15 @@ class Home(MainHandler):
         ep_queue_item = search_queue.FailedQueueItem(ep_obj.show, ep_obj)
         sickbeard.searchQueueScheduler.action.add_item(ep_queue_item)  # @UndefinedVariable
 
-        # wait until the queue item tells us whether it worked or not
-        while ep_queue_item.success is None:  # @UndefinedVariable
-            time.sleep(cpu_presets[sickbeard.CPU_PRESET])
-
-        # return the correct json value
         if ep_queue_item.success:
-            # Find the quality class for the episode
-            quality_class = Quality.qualityStrings[Quality.UNKNOWN]
-            ep_status, ep_quality = Quality.splitCompositeStatus(ep_obj.status)
-            for x in (SD, HD720p, HD1080p):
-                if ep_quality in Quality.splitQuality(x)[0]:
-                    quality_class = qualityPresetStrings[x]
-                    break
-
-            return json.dumps({'result': statusStrings[ep_obj.status],
-                               'quality': quality_class
-            })
-
-        return json.dumps({'result': 'failure'})
-
+            return returnManualSearchResult(ep_queue_item)
+        if not ep_queue_item.started and ep_queue_item.success is None:
+            return json.dumps({'result': 'success'}) #I Actually want to call it queued, because the search hasnt been started yet!
+        if ep_queue_item.started and ep_queue_item.success is None:
+            return json.dumps({'result': 'success'})
+        else:
+            return json.dumps({'result': 'failure'})
+        
 
 class UI(MainHandler):
     def add_message(self):