From 392e8e18915d0db26ca8a3840d5783cc81cd1c1c Mon Sep 17 00:00:00 2001
From: "zhimin.zeng" <zhimin.zeng@bambulab.com>
Date: Mon, 13 Jan 2025 17:18:00 +0800
Subject: [PATCH] FIX: back up the selected printer in prepare page

jira: none
Change-Id: Ibd163f78c4ce911a88459ecca96660ed4aa68ab2
---
 src/slic3r/GUI/Plater.cpp           | 5 +++++
 src/slic3r/GUI/Plater.hpp           | 1 +
 src/slic3r/GUI/PresetComboBoxes.cpp | 8 +++++++-
 src/slic3r/GUI/PresetComboBoxes.hpp | 3 +++
 4 files changed, 16 insertions(+), 1 deletion(-)

diff --git a/src/slic3r/GUI/Plater.cpp b/src/slic3r/GUI/Plater.cpp
index 27d4fc565..3c6638497 100644
--- a/src/slic3r/GUI/Plater.cpp
+++ b/src/slic3r/GUI/Plater.cpp
@@ -2885,6 +2885,11 @@ void Sidebar::sync_ams_list()
     notify_manager->bbl_show_seqprintinfo_notification(info_text);
 }
 
+PlaterPresetComboBox* Sidebar::printer_combox()
+{
+    return p->combo_printer;
+}
+
 ObjectList* Sidebar::obj_list()
 {
     // BBS
diff --git a/src/slic3r/GUI/Plater.hpp b/src/slic3r/GUI/Plater.hpp
index 62b38df5b..c092735fe 100644
--- a/src/slic3r/GUI/Plater.hpp
+++ b/src/slic3r/GUI/Plater.hpp
@@ -161,6 +161,7 @@ public:
     bool should_sync_extruder_list(MachineObject* obj);
     void update_sync_status(const MachineObject* obj);
 
+    PlaterPresetComboBox *  printer_combox();
     ObjectList*             obj_list();
     ObjectSettings*         obj_settings();
     ObjectLayers*           obj_layers();
diff --git a/src/slic3r/GUI/PresetComboBoxes.cpp b/src/slic3r/GUI/PresetComboBoxes.cpp
index b4c8fca64..7353f5a1e 100644
--- a/src/slic3r/GUI/PresetComboBoxes.cpp
+++ b/src/slic3r/GUI/PresetComboBoxes.cpp
@@ -182,7 +182,7 @@ void PresetComboBox::invalidate_selection()
 
 void PresetComboBox::validate_selection(bool predicate/*=false*/)
 {
-    if (predicate ||
+    if (predicate &&
         // just in case: mark m_last_selected as a first added element
         m_last_selected == INT_MAX)
         m_last_selected = GetCount() - 1;
@@ -290,24 +290,28 @@ wxString PresetComboBox::get_preset_item_name(unsigned int index)
     if (m_type == Preset::TYPE_PRINTER) {
         int idx = selected_connected_printer(index);
         if (idx < 0) {
+            m_selected_dev_id.clear();
             return GetString(index);
         }
         else {
             DeviceManager *dev = Slic3r::GUI::wxGetApp().getDeviceManager();
             if (!dev) {
                 assert(false);
+                m_selected_dev_id.clear();
                 return GetString(index);
             }
 
             std::map<std::string, MachineObject *> machine_list = dev->get_my_machine_list();
             if (machine_list.empty()) {
                 assert(false);
+                m_selected_dev_id.clear();
                 return GetString(index);
             }
 
             auto iter = m_backup_dev_list_sorted.begin();
             std::advance(iter, idx);
             if (iter != m_backup_dev_list_sorted.end() && machine_list.find(*iter) != machine_list.end()) {
+                m_selected_dev_id = *iter;
                 Preset* machine_preset = get_printer_preset(machine_list[*iter]);
                 if (machine_preset) {
                     dev->set_selected_machine(*iter);
@@ -317,6 +321,7 @@ wxString PresetComboBox::get_preset_item_name(unsigned int index)
         }
     }
 
+    m_selected_dev_id.clear();
     return GetString(index);
 }
 
@@ -476,6 +481,7 @@ void PresetComboBox::add_connected_printers(std::string selected, bool alias_nam
         boost::replace_all(printer_model, "Bambu Lab ", "");
         auto text = iter->second->dev_name + " (" + printer_model + ")";
         int item_id = Append(from_u8(text), wxNullBitmap, &m_first_printer_idx + std::distance(user_machine_list.begin(), iter));
+        validate_selection(m_selected_dev_id == iter->first);
     }
     m_last_printer_idx = GetCount();
 }
diff --git a/src/slic3r/GUI/PresetComboBoxes.hpp b/src/slic3r/GUI/PresetComboBoxes.hpp
index 86b884f1b..49d299da2 100644
--- a/src/slic3r/GUI/PresetComboBoxes.hpp
+++ b/src/slic3r/GUI/PresetComboBoxes.hpp
@@ -81,6 +81,8 @@ public:
     void set_filament_idx(const int extr_idx) { m_filament_idx = extr_idx; }
     int  get_filament_idx() const { return m_filament_idx; }
 
+    std::string get_selected_dev_id() const { return m_selected_dev_id; }
+
     // BBS
     wxString get_tooltip(const Preset& preset);
 
@@ -138,6 +140,7 @@ protected:
 
     std::vector<std::string> m_backup_dev_list;
     std::vector<std::string> m_backup_dev_list_sorted;
+    std::string              m_selected_dev_id;
 
     PrinterTechnology printer_technology {ptAny};