Fix for PresetCollection::load_external_preset()

"filament_vendor" parameter isn't used neither by the UI nor by the slicing core,
therefore erase it from config apply to avoid redundant "dirty" parameter in loaded preset.
This commit is contained in:
YuSanka 2023-11-16 12:37:39 +01:00
parent 3d438ea93a
commit 55e3bf4c0a

View File

@ -768,7 +768,7 @@ static bool profile_print_params_same(const DynamicPrintConfig &cfg_old, const D
// when comparing profiles for equality. Ignore them.
for (const char *key : { "compatible_prints", "compatible_prints_condition",
"compatible_printers", "compatible_printers_condition", "inherits",
"print_settings_id", "filament_settings_id", "sla_print_settings_id", "sla_material_settings_id", "printer_settings_id",
"print_settings_id", "filament_settings_id", "sla_print_settings_id", "sla_material_settings_id", "printer_settings_id", "filament_vendor",
"printer_model", "printer_variant", "default_print_profile", "default_filament_profile", "default_sla_print_profile", "default_sla_material_profile",
//FIXME remove the print host keys?
"print_host", "printhost_apikey", "printhost_cafile" })
@ -849,7 +849,7 @@ ExternalPreset PresetCollection::load_external_preset(
// Following keys are not used neither by the UI nor by the slicing core, therefore they are not important
// Erase them from config apply to avoid redundant "dirty" parameter in loaded preset.
for (const char* key : { "print_settings_id", "filament_settings_id", "sla_print_settings_id", "sla_material_settings_id", "printer_settings_id",
for (const char* key : { "print_settings_id", "filament_settings_id", "sla_print_settings_id", "sla_material_settings_id", "printer_settings_id", "filament_vendor",
"printer_model", "printer_variant", "default_print_profile", "default_filament_profile", "default_sla_print_profile", "default_sla_material_profile" })
keys.erase(std::remove(keys.begin(), keys.end(), key), keys.end());