From d8f5daf345d09e61a6e6fd48ad1d09d6d1b073fb Mon Sep 17 00:00:00 2001
From: Enrico Turri <enricoturri@seznam.cz>
Date: Tue, 24 Jul 2018 15:32:44 +0200
Subject: [PATCH] Fixed selection of object modified by gizmo

---
 xs/src/slic3r/GUI/GLCanvas3D.cpp | 33 +++++++++-----------------------
 xs/src/slic3r/GUI/GLCanvas3D.hpp |  2 +-
 2 files changed, 10 insertions(+), 25 deletions(-)

diff --git a/xs/src/slic3r/GUI/GLCanvas3D.cpp b/xs/src/slic3r/GUI/GLCanvas3D.cpp
index 819f26609..193ccd068 100644
--- a/xs/src/slic3r/GUI/GLCanvas3D.cpp
+++ b/xs/src/slic3r/GUI/GLCanvas3D.cpp
@@ -2715,7 +2715,7 @@ void GLCanvas3D::on_mouse(wxMouseEvent& evt)
         {
             update_gizmos_data();
             m_gizmos.start_dragging();
-            m_mouse.drag.gizmo_volume_idx = _get_first_selected_volume_id();
+            m_mouse.drag.gizmo_volume_idx = _get_first_selected_volume_id(selected_object_idx);
             m_dirty = true;
         }
         else
@@ -3736,32 +3736,17 @@ int GLCanvas3D::_get_first_selected_object_id() const
     return -1;
 }
 
-int GLCanvas3D::_get_first_selected_volume_id() const
+int GLCanvas3D::_get_first_selected_volume_id(int object_id) const
 {
-    if (m_print != nullptr)
-    {
-        int objects_count = (int)m_print->objects.size();
+    int volume_id = -1;
 
-        for (const GLVolume* vol : m_volumes.volumes)
-        {
-            if ((vol != nullptr) && vol->selected)
-            {
-                int object_id = vol->select_group_id / 1000000;
-                // Objects with object_id >= 1000 have a specific meaning, for example the wipe tower proxy.
-                if ((object_id < 10000) && (object_id < objects_count))
-                {
-                    int volume_id = 0;
-                    for (int i = 0; i < object_id; ++i)
-                    {
-                        const PrintObject* obj = m_print->objects[i];
-                        const ModelObject* model = obj->model_object();
-                        volume_id += model->instances.size();
-                    }
-                    return volume_id;
-                }
-            }
-        }
+    for (const GLVolume* vol : m_volumes.volumes)
+    {
+        ++volume_id;
+        if ((vol != nullptr) && vol->selected && (object_id == vol->select_group_id / 1000000))
+            return volume_id;
     }
+
     return -1;
 }
 
diff --git a/xs/src/slic3r/GUI/GLCanvas3D.hpp b/xs/src/slic3r/GUI/GLCanvas3D.hpp
index d52d69eeb..0d8cf8855 100644
--- a/xs/src/slic3r/GUI/GLCanvas3D.hpp
+++ b/xs/src/slic3r/GUI/GLCanvas3D.hpp
@@ -642,7 +642,7 @@ private:
     void _stop_timer();
 
     int _get_first_selected_object_id() const;
-    int _get_first_selected_volume_id() const;
+    int _get_first_selected_volume_id(int object_id) const;
 
     // Create 3D thick extrusion lines for a skirt and brim.
     // Adds a new Slic3r::GUI::3DScene::Volume to volumes.