Add UI for unpack_check and make the default value "on" again.

interfaces/Classic/templates/config_switches.tmpl

@@ -41,6 +41,9 @@
     <label><input type="checkbox" name="pause_on_pwrar" value="1" <!--#if $pause_on_pwrar > 0 then "checked=1" else ""#--> /> <strong>$T('opt-pause_on_pwrar')</strong></label><br>
     $T('explain-pause_on_pwrar')<br>
     <br>
+    <label><input type="checkbox" name="unpack_check" value="1" <!--#if $unpack_check > 0 then "checked=1" else ""#--> /> <strong>$T('opt-unpack_check')</strong></label><br>
+    $T('explain-unpack_check')<br>
+    <br>
     <strong>$T('opt-no_dupes'):</strong><br>
     $T('explain-no_dupes')<br>
     <input class="radio" type="radio" name="no_dupes" value="0" <!--#if $no_dupes == 0 then 'checked="1"' else ""#--> /> $T('nodupes-off')

interfaces/Plush/templates/config_switches.tmpl

@@ -224,6 +224,13 @@
           <span class="component-desc">$T('explain-sfv_check')</span>
         </label>
       </div>
+      <div class="field-pair alt">
+        <input type="checkbox" name="sfv_check" id="unpack_check" value="1" <!--#if $unpack_check > 0 then "checked=1" else ""#--> />
+        <label class="clearfix" for="unpack_check">
+          <span class="component-title">$T('opt-unpack_check')</span>
+          <span class="component-desc">$T('explain-unpack_check')</span>
+        </label>
+      </div>
       <!--#if $nt#-->
       <div class="field-pair">
         <input type="checkbox" name="par2_multicore" id="par2_multicore" value="1" <!--#if $par2_multicore > 0 then "checked=1" else ""#--> />

interfaces/smpl/templates/config_switches.tmpl

@@ -68,6 +68,11 @@
   <span class="tips">$T('explain-sfv_check')</span></label>
 <br class="clear" />
 
+  <label><span class="label">$T('opt-unpack_check'):</span>
+  <input class="radio" type="checkbox" name="unpack_check" value="1" <!--#if $unpack_check > 0 then 'checked="1"' else ""#--> />
+  <span class="tips">$T('explain-unpack_check')</span></label>
+<br class="clear" />
+
   <label><span class="label">$T('opt-folder_rename'):</span>
   <input class="radio" type="checkbox" name="folder_rename" value="1" <!--#if $folder_rename > 0 then 'checked="1"' else ""#--> />
   <span class="tips">$T('explain-folder_rename')</span></label>

sabnzbd/cfg.py

@@ -199,7 +199,7 @@ enable_https = OptionBool('misc', 'enable_https', False)
 
 language = OptionStr('misc', 'language', 'en')
 ssl_type = OptionStr('misc', 'ssl_type', 'v23')
-unpack_check = OptionBool('misc', 'unpack_check', False)
+unpack_check = OptionBool('misc', 'unpack_check', True)
 no_penalties = OptionBool('misc', 'no_penalties', False)
 
 # Internal options, not saved in INI file

sabnzbd/interface.py

@@ -1122,7 +1122,8 @@ SWITCH_LIST = \
              'enable_par_cleanup', 'auto_sort', 'check_new_rel', 'auto_disconnect',
              'safe_postproc', 'no_dupes', 'replace_spaces', 'replace_dots', 'replace_illegal', 'auto_browser',
              'ignore_samples', 'pause_on_post_processing', 'quick_check', 'nice', 'ionice',
-             'ssl_type', 'pre_script', 'pause_on_pwrar', 'ampm', 'sfv_check', 'folder_rename'
+             'ssl_type', 'pre_script', 'pause_on_pwrar', 'ampm', 'sfv_check', 'folder_rename',
+             'unpack_check'
              )
 
 #------------------------------------------------------------------------------

sabnzbd/skintext.py

@@ -365,6 +365,8 @@ SKIN_TEXT = {
     'nodupes-pause' : TT('Pause'), #: Three way switch for duplicates
     'opt-sfv_check' : TT('Enable SFV-based checks'),
     'explain-sfv_check' : TT('Do an extra verification based on SFV files.'),
+    'opt-unpack_check' : TT('Check result of unpacking'),
+    'explain-unpack_check' : TT('Check result of unpacking (needs to be off for some file systems).'),
     'opt-folder_rename' : TT('Enable folder rename'),
     'explain-folder_rename' : TT('Use temporary names during post processing. Disable when your system doesn\'t handle that properly.'),
     'opt-dirscan_opts' : TT('Default Post-Processing'),