Force ParseSpeed to 1 when fixer is used

Signed-off-by: Maxime Gervais <[email protected]>
g-maxime · Dec 15, 2024 · 3d838a6 · 3d838a6
1 parent 1e6e4e9
commit 3d838a6
Show file tree

Hide file tree

Showing 4 changed files with 56 additions and 7 deletions.
diff --git a/Source/CLI/CLI.cpp b/Source/CLI/CLI.cpp
@@ -17,6 +17,7 @@
 #include "Common/Reports.h"
 #include <Common/MediaConchLib.h>
 #include <ZenLib/ZtringList.h>
+#include <ZenLib/Ztring.h>
 #include <ZenLib/File.h>
 #include <ZenLib/Dir.h>
 
@@ -126,7 +127,7 @@ void __stdcall Event_CallBackFunction(unsigned char* Data_Content, size_t Data_S
                  force_analyze(false), mil_analyze(true),
                  watch_folder_recursive(true), create_policy_mode(false), file_information(false),
                  plugins_list_mode(false), list_watch_folders_mode(false), no_needs_files_mode(false),
-                 list_mode(false)
+                 list_mode(false), fixer(false)
     {
         format = MediaConchLib::format_Max;
     }
@@ -931,6 +932,23 @@ void __stdcall Event_CallBackFunction(unsigned char* Data_Content, size_t Data_S
             return CLI_RETURN_FINISH;
         }
 
+        ZenLib::Ztring str = ZenLib::Ztring().From_UTF8(key.c_str());
+        if (str.Compare(__T("File_TryToFix")))
+        {
+            fixer = true;
+            options.push_back(std::make_pair(key, value));
+            options.push_back(std::make_pair("file_parsespeed", "1"));
+        }
+        else if (fixer && (str.Compare(__T("File_ParseSpeed")) || str.Compare(__T("ParseSpeed"))))
+        {
+            ZenLib::Ztring str = ZenLib::Ztring().From_UTF8(value.c_str());
+            if (str != __T("1"))
+            {
+                STRINGOUT(ZenLib::Ztring().From_UTF8("ParseSpeed option must be set to 1 with File_TryToFix option."));
+                return CLI_RETURN_ERROR;
+            }
+        }
+
         options.push_back(std::make_pair(key, value));
         return CLI_RETURN_NONE;
     }

diff --git a/Source/CLI/CLI.h b/Source/CLI/CLI.h
@@ -111,6 +111,7 @@ namespace MediaConch
         bool                    list_watch_folders_mode;
         bool                    no_needs_files_mode;
         bool                    list_mode;
+        bool                    fixer;
     };
 
 }

diff --git a/Source/GUI/Qt/mainwindow.cpp b/Source/GUI/Qt/mainwindow.cpp
@@ -241,11 +241,14 @@ int MainWindow::add_file_to_list(const QString& file, const QString& path,
     uisettings.change_last_display(displays_list[display_i].toUtf8().data());
     uisettings.change_last_verbosity(verbosity_i);
 
-    int parsespeed_idx = options.lastIndexOf("file_parsespeed");
-    if (parsespeed_idx != -1 && parsespeed_idx < options.size())
-        uisettings.change_last_parsespeed(options[parsespeed_idx + 1].toStdString());
-    else
-        uisettings.change_last_parsespeed("");
+    if (!fixer)
+    {
+        int parsespeed_idx = options.lastIndexOf("file_parsespeed");
+        if (!fixer && parsespeed_idx != -1 && parsespeed_idx < options.size())
+            uisettings.change_last_parsespeed(options[parsespeed_idx + 1].toStdString());
+        else
+            uisettings.change_last_parsespeed("");
+    }
 
     std::string full_path = filepath;
 #ifdef WINDOWS
@@ -1414,12 +1417,13 @@ int MainWindow::analyze(const std::vector<std::string>& files, bool with_fixer,
     if (with_fixer)
     {
         options.push_back(std::make_pair("File_TryToFix", "1"));
+        options.push_back(std::make_pair("file_parsespeed", "1"));
         force = true;
     }
 
     for (size_t i = 0; i + 1 < opt.size();)
     {
-        if (opt[i] != "File_TryToFix")
+        if (opt[i] != "File_TryToFix" && opt[i] != "file_parsespeed")
             options.push_back(std::make_pair(opt[i], opt[i + 1]));
         i += 2;
     }

diff --git a/Source/Resource/html/js/checker/base.js b/Source/Resource/html/js/checker/base.js
@@ -8,6 +8,32 @@ var checker = (function() {
         bindings();
 
         //
+        var checkerUpload_parsespeed_val_save = $('#checkerUpload_parsespeed_selector').val()
+        $('#checkerUpload_fixer').on('click', function(e) {
+            if ($('#checkerUpload_fixer').is(':checked')) {
+                $('#checkerUpload_parsespeed_selector').prop( "disabled", true);
+                checkerUpload_parsespeed_val_save = $('#checkerUpload_parsespeed_selector').val()
+                $('#checkerUpload_parsespeed_selector').val('1');
+            }
+            else {
+                $('#checkerUpload_parsespeed_selector').prop( "disabled", false);
+                $('#checkerUpload_parsespeed_selector').val(checkerUpload_parsespeed_val_save);
+            }
+        })
+
+        var checkerRepository_parsespeed_val_save = $('#checkerRepository_parsespeed_selector').val()
+        $('#checkerRepository_fixer').on('click', function(e) {
+            if ($('#checkerRepository_fixer').is(':checked')) {
+                $('#checkerRepository_parsespeed_selector').prop( "disabled", true);
+                checkerRepository_parsespeed_val_save = $('#checkerRepository_parsespeed_selector').val()
+                $('#checkerRepository_parsespeed_selector').val('1');
+            }
+            else {
+                $('#checkerRepository_parsespeed_selector').prop( "disabled", false);
+                $('#checkerRepository_parsespeed_selector').val(checkerRepository_parsespeed_val_save);
+            }
+        })
+
         webpage.set_parse_speed($('.parsespeedList').val());
         $('.parsespeedList').change(function(){
             webpage.set_parse_speed($(this).val());