From 2e387b1ce8b79faebd3e8e9da421882b8b561db3 Mon Sep 17 00:00:00 2001
From: JackDandy <JackDandy@users.noreply.github.com>
Date: Tue, 3 Jun 2014 18:43:37 +0100
Subject: [PATCH] Expose "Date Style"... "Use System Default" only when
 FuzzyMoment is off.

If the Date Style "Use System Default" is selected and FuzzyMoments is clicked on, then an appropriate Date Style is chosen from explicit defaults.  This change prevents issues where moments() cannot interpret or guess locale date formats.
---
 gui/slick/interfaces/default/config_general.tmpl | 39 +++++++++++++++---------
 gui/slick/js/config.js                           | 33 +++++++++++++++++++-
 gui/slick/js/fuzzyMoment.js                      |  4 +--
 sickbeard/webserve.py                            |  3 +-
 4 files changed, 61 insertions(+), 18 deletions(-)

diff --git a/gui/slick/interfaces/default/config_general.tmpl b/gui/slick/interfaces/default/config_general.tmpl
index a96c236..59c3359 100644
--- a/gui/slick/interfaces/default/config_general.tmpl
+++ b/gui/slick/interfaces/default/config_general.tmpl
@@ -370,31 +370,42 @@
                     <fieldset class="component-group-list" style="width:670px">
 
                         <div class="field-pair">
-                            <input type="checkbox" name="fuzzy_dating" id="fuzzy_dating" #if $sickbeard.FUZZY_DATING == True then "checked=\"checked\"" else ""#/>
+                            <input type="checkbox" name="fuzzy_dating" id="fuzzy_dating" class="viewIf datePresets" #if $sickbeard.FUZZY_DATING == True then "checked=\"checked\"" else ""#/>
                             <label class="clearfix" for="fuzzy_dating">
                                 <span class="component-title">Display Fuzzy Dates</span>
                                 <span class="component-desc">E.g "Last Thu", "On Tue" and move the absolute date into time tooltips?</span>
                             </label>
                         </div>
-
-                        <div class="field-pair">
-                            <input type="checkbox" name="trim_zero" id="trim_zero" #if $sickbeard.TRIM_ZERO == True then "checked=\"checked\"" else ""#/>
-                            <label class="clearfix" for="trim_zero">
-                                <span class="component-title">Trim Zero Padding</span>
-                                <span class="component-desc">Trim leading number "0" shown on hour of day and date of month?</span>
-                            </label>
+                        <div class="show_if_fuzzy_dating#if True == $sickbeard.FUZZY_DATING then '' else ' metadataDiv'#">
+                            <div class="field-pair">
+                                <input type="checkbox" name="trim_zero" id="trim_zero" #if $sickbeard.TRIM_ZERO == True then "checked=\"checked\"" else ""#/>
+                                <label class="clearfix" for="trim_zero">
+                                    <span class="component-title">Trim Zero Padding</span>
+                                    <span class="component-desc">Trim leading number "0" shown on hour of day and date of month?</span>
+                                </label>
+                            </div>
                         </div>
 
                         <div class="field-pair">
                             <label class="nocheck clearfix" for="date_presets">
                                 <span class="component-title">Date Style:</span>
                                 <span class="component-desc">
-                                    <select id="date_presets" name="date_preset">
-                                        <option value="%x" #if "%x" == $sickbeard.DATE_PRESET then "selected=\"selected\"" else ""#>Use System Default</option>
-                                        #for $cur_preset in $date_presets:
-                                            <option value="$cur_preset" #if $cur_preset == $sickbeard.DATE_PRESET then "selected=\"selected\"" else ""#>$datetime.datetime($datetime.datetime.now().year+1, 1, 1, 14, 30, 47).strftime($cur_preset)</option>
-                                        #end for
-                                    </select>
+                                    <div class="show_if_fuzzy_dating#if True == $sickbeard.FUZZY_DATING then '' else ' metadataDiv'#">
+                                        <select id="date_presets#if True == $sickbeard.FUZZY_DATING then '' else '_na'#" name="date_preset#if True == $sickbeard.FUZZY_DATING then '' else '_na'#">
+                                            #for $cur_preset in $date_presets:
+                                                <option value="$cur_preset" #if $cur_preset == $sickbeard.DATE_PRESET or ("%x" == $sickbeard.DATE_PRESET and "$cur_preset" == '%a, %b %d, %Y') then "selected=\"selected\"" else ""#>$datetime.datetime($datetime.datetime.now().year, 12, 31, 14, 30, 47).strftime($cur_preset)</option>
+                                            #end for
+                                        </select>
+                                    </div>
+                                    <div class="hide_if_fuzzy_dating#if True != $sickbeard.FUZZY_DATING then '' else ' metadataDiv'#">
+                                        <select id="date_presets#if True != $sickbeard.FUZZY_DATING then '' else '_na'#" name="date_preset#if True != $sickbeard.FUZZY_DATING then '' else '_na'#">
+                                            <option value="%x" #if "%x" == $sickbeard.DATE_PRESET then "selected=\"selected\"" else ""#>Use System Default</option>
+                                            #for $cur_preset in $date_presets:
+                                                <option value="$cur_preset" #if $cur_preset == $sickbeard.DATE_PRESET then "selected=\"selected\"" else ""#>$datetime.datetime($datetime.datetime.now().year, 12, 31, 14, 30, 47).strftime($cur_preset)</option>
+                                            #end for
+                                        </select>
+                                    </div>
+                                    <div id="date_use_system_default" class="metadataDiv"></div>
                                 </span>
                             </label>
                         </div>
diff --git a/gui/slick/js/config.js b/gui/slick/js/config.js
index 709d3ac..e9fafc5 100644
--- a/gui/slick/js/config.js
+++ b/gui/slick/js/config.js
@@ -9,7 +9,38 @@ $(document).ready(function(){
             $('#content_'+$(this).attr('id')).fadeIn("fast", "linear");
         else
             $('#content_'+$(this).attr('id')).fadeOut("fast", "linear");
-  });
+    });
+
+    $(".viewIf").click(function() {
+        if ($(this).prop('checked')) {
+            $('.hide_if_'+$(this).attr('id')).css('display','none');
+            $('.show_if_'+$(this).attr('id')).fadeIn("fast", "linear");
+        } else {
+            $('.show_if_'+$(this).attr('id')).css('display','none');
+            $('.hide_if_'+$(this).attr('id')).fadeIn("fast", "linear");
+        }
+    });
+
+    $(".datePresets").click(function() {
+        var def = $('#date_presets').val()
+        if ($(this).prop('checked') && '%x' == def) {
+            def = '%a, %b %d, %Y'
+            $('#date_use_system_default').html('1')
+        } else if (!$(this).prop('checked') && '1' == $('#date_use_system_default').html())
+            def = '%x'
+
+        $('#date_presets').attr('name', 'date_preset_old')
+        $('#date_presets').attr('id', 'date_presets_old')
+
+        $('#date_presets_na').attr('name', 'date_preset')
+        $('#date_presets_na').attr('id', 'date_presets')
+
+        $('#date_presets_old').attr('name', 'date_preset_na')
+        $('#date_presets_old').attr('id', 'date_presets_na')
+
+        if (def)
+            $('#date_presets').val(def)
+    });
 
     // bind 'myForm' and provide a simple callback function 
     $('#configForm').ajaxForm({
diff --git a/gui/slick/js/fuzzyMoment.js b/gui/slick/js/fuzzyMoment.js
index f1812dc..0b24597 100644
--- a/gui/slick/js/fuzzyMoment.js
+++ b/gui/slick/js/fuzzyMoment.js
@@ -108,8 +108,8 @@
             airdate = airdatetime.clone().hour(0).minute(0).second(0).millisecond(0),
             today = moment({}),
             day = Math.abs(airdate.diff(today, 'days')),
-            week = airdate.diff(today, 'week'), isPast = week < 0, week = Math.abs(week),
-            titleThis = false, qTipTime = false
+            week = Math.abs(weekdiff = airdate.diff(today, 'week')), isPast = weekdiff < 0,
+            titleThis = false, qTipTime = false,
             result = (0 == week ? airdatetime.calendar() : '');
 
         if (/\bOn\b/i.test(result)) {
diff --git a/sickbeard/webserve.py b/sickbeard/webserve.py
index 6631564..fe356ef 100644
--- a/sickbeard/webserve.py
+++ b/sickbeard/webserve.py
@@ -1047,7 +1047,7 @@ class ConfigGeneral:
                     web_password=None, version_notify=None, enable_https=None, https_cert=None, https_key=None,
                     handle_reverse_proxy=None, sort_article=None, auto_update=None, proxy_setting=None,
                     anon_redirect=None, git_path=None, calendar_unprotected=None,
-                    fuzzy_dating=None, trim_zero=None, date_preset=None, time_preset=None,
+                    fuzzy_dating=None, trim_zero=None, date_preset=None, date_preset_na=None, time_preset=None,
                     indexer_timeout=None):
 
         results = []
@@ -1081,6 +1081,7 @@ class ConfigGeneral:
 
         if date_preset:
             sickbeard.DATE_PRESET = date_preset
+            discarded_na_data = date_preset_na
 
         if indexer_default:
             sickbeard.INDEXER_DEFAULT = config.to_int(indexer_default)