From 9f346f271cf468e27cc8b1ac6abc9cd7a78cfd1d Mon Sep 17 00:00:00 2001
From: Luke Pulverenti <luke.pulverenti@gmail.com>
Date: Wed, 19 Feb 2014 13:50:37 -0500
Subject: [PATCH] rework image extraction settings

---
 dashboard-ui/metadataadvanced.html          |  8 --------
 dashboard-ui/scripts/metadataadvanced.js    |  4 ----
 dashboard-ui/scripts/wizardimagesettings.js | 21 +++++++++++++--------
 dashboard-ui/wizardimagesettings.html       |  4 ++--
 4 files changed, 15 insertions(+), 22 deletions(-)

diff --git a/dashboard-ui/metadataadvanced.html b/dashboard-ui/metadataadvanced.html
index 2dfdfd39dc..f54f351e1f 100644
--- a/dashboard-ui/metadataadvanced.html
+++ b/dashboard-ui/metadataadvanced.html
@@ -20,14 +20,6 @@
                 </div>
 
                 <form id="advancedMetadataConfigurationForm">
-                    <ul data-role="listview" class="ulForm">
-                        <li>
-                            <label for="chkVIdeoImages">Enable video image extraction</label>
-                            <input id="chkVIdeoImages" name="chkVIdeoImages" type="checkbox" checked="checked" data-mini="true" />
-                            <div class="fieldDescription">This is for videos that don't already have images, and that we're uanble to find internet images for. This will add some additional time to the initial library scan but will result in a more pleasing presentation.</div>
-                        </li>
-                    </ul>
-                    <h2>Chapter Images</h2>
                     <ul data-role="listview" class="ulForm">
                         <li>
                             <label>Extract chapter images for:</label>
diff --git a/dashboard-ui/scripts/metadataadvanced.js b/dashboard-ui/scripts/metadataadvanced.js
index ce3342b89b..e0f12829b7 100644
--- a/dashboard-ui/scripts/metadataadvanced.js
+++ b/dashboard-ui/scripts/metadataadvanced.js
@@ -15,8 +15,6 @@
 
     load: function (page, config) {
 
-        $('#chkVIdeoImages', page).checked(config.EnableVideoImageExtraction).checkboxradio("refresh");
-
         $('#chkMovies', page).checked(config.EnableMovieChapterImageExtraction).checkboxradio("refresh");
         $('#chkEpisodes', page).checked(config.EnableEpisodeChapterImageExtraction).checkboxradio("refresh");
         $('#chkOtherVideos', page).checked(config.EnableOtherVideoChapterImageExtraction).checkboxradio("refresh");
@@ -55,8 +53,6 @@
 
         ApiClient.getServerConfiguration().done(function (config) {
 
-            config.EnableVideoImageExtraction = $('#chkVIdeoImages', form).checked();
-
             config.EnableMovieChapterImageExtraction = $('#chkMovies', form).checked();
             config.EnableEpisodeChapterImageExtraction = $('#chkEpisodes', form).checked();
             config.EnableOtherVideoChapterImageExtraction = $('#chkOtherVideos', form).checked();
diff --git a/dashboard-ui/scripts/wizardimagesettings.js b/dashboard-ui/scripts/wizardimagesettings.js
index 0de21355df..b9a8713b4c 100644
--- a/dashboard-ui/scripts/wizardimagesettings.js
+++ b/dashboard-ui/scripts/wizardimagesettings.js
@@ -4,21 +4,26 @@
 
         Dashboard.showLoadingMsg();
 
-        // After saving chapter task, now save server config
-        ApiClient.getServerConfiguration().done(function (config) {
+        $.ajax({
+            type: "POST",
+            url: ApiClient.getUrl("System/Configuration/VideoImageExtraction", { Enabled: $('#chkVideoImages', page).checked() })
 
-            config.EnableVideoImageExtraction = $('#chkVIdeoImages', page).checked();
-            config.ImageSavingConvention = $('#selectImageSavingConvention', page).val();
+        }).done(function () {
 
-            config.EnableMovieChapterImageExtraction = $('#chkMovies', page).checked();
 
-            ApiClient.updateServerConfiguration(config).done(function (result) {
+            // After saving chapter task, now save server config
+            ApiClient.getServerConfiguration().done(function (config) {
 
-                navigateToNextPage();
+                config.ImageSavingConvention = $('#selectImageSavingConvention', page).val();
+                config.EnableMovieChapterImageExtraction = $('#chkMovies', page).checked();
 
+                ApiClient.updateServerConfiguration(config).done(function (result) {
+
+                    navigateToNextPage();
+
+                });
             });
         });
-
     }
 
     function navigateToNextPage() {
diff --git a/dashboard-ui/wizardimagesettings.html b/dashboard-ui/wizardimagesettings.html
index 0b2a71409e..e63b7ee746 100644
--- a/dashboard-ui/wizardimagesettings.html
+++ b/dashboard-ui/wizardimagesettings.html
@@ -23,8 +23,8 @@
                 </div>
 
                 <div style="margin: 2em 0;">
-                    <label for="chkVIdeoImages">Enable video image extraction</label>
-                    <input id="chkVIdeoImages" name="chkVIdeoImages" type="checkbox" checked="checked" />
+                    <label for="chkVideoImages">Enable video image extraction</label>
+                    <input id="chkVideoImages" name="chkVideoImages" type="checkbox" checked="checked" />
                     <div class="fieldDescription">For videos that don't already have images, and that we're unable to find internet images for. This will add some additional time to the initial library scan but will result in a more pleasing presentation.</div>
                 </div>