From ce6eff0a22e5827596b27e22a5aa06315b54575e Mon Sep 17 00:00:00 2001
From: "Zheng, Lei" <realthunder.dev@gmail.com>
Date: Fri, 6 Mar 2020 13:28:40 +0800
Subject: [PATCH] PartDesign: improve SubShapeBinder make command

---
 src/Mod/PartDesign/Gui/Command.cpp | 46 ++++++++++++------------------
 1 file changed, 18 insertions(+), 28 deletions(-)

diff --git a/src/Mod/PartDesign/Gui/Command.cpp b/src/Mod/PartDesign/Gui/Command.cpp
index e9b8d7f696..d776531367 100644
--- a/src/Mod/PartDesign/Gui/Command.cpp
+++ b/src/Mod/PartDesign/Gui/Command.cpp
@@ -383,36 +383,26 @@ void CmdPartDesignSubShapeBinder::activated(int iMsg)
         pcActiveBody = PartDesignGui::getBody(false,true,true,&binderParent,&binderSub);
         FeatName = getUniqueObjectName("Binder",pcActiveBody);
     }
-    Base::Matrix4D mat;
-    if(values.size()==1 && binderParent && binderParent!=binder) {
-        App::DocumentObject *obj = values.begin()->first;
-        auto subs = values.begin()->second;
-        App::DocumentObject *sobj = 0;
-        App::DocumentObject *parent = 0;
-        std::string parentSub = binderSub;
-        for(auto &sub : subs) {
-            auto link = obj;
-            auto linkSub = binderSub;
-            auto res = binderParent->resolveRelativeLink(linkSub,link,sub);
-            if(!sobj) {
-                sobj = link;
-                parent = res;
-                parentSub = linkSub;
-            }else if(sobj!=link || parent!=res) {
-                QMessageBox::critical(Gui::getMainWindow(), QObject::tr("SubShapeBinder"),
-                        QObject::tr("Cannot link to more than one object"));
-                return;
+    if(binderParent && binderParent!=binder) {
+        decltype(values) links;
+        for(auto &v : values) {
+            App::DocumentObject *obj = v.first;
+            if(obj == pcActiveBody)
+                continue;
+            if(obj != binderParent) {
+                auto &subs = links[obj];
+                subs.insert(subs.end(),v.second.begin(),v.second.end());
+                continue;
+            }
+            for(auto &sub : v.second) {
+                auto link = obj;
+                auto linkSub = binderSub;
+                binderParent->resolveRelativeLink(linkSub,link,sub);
+                if(link && link != pcActiveBody)
+                    links[link].push_back(sub);
             }
         }
-        if(sobj) {
-            values.clear();
-            values[sobj] = std::move(subs);
-        }
-        if(parent) {
-            binderParent = parent;
-            binderSub = parentSub;
-            binderParent->getSubObject(binderSub.c_str(),0,&mat);
-        }
+        values = std::move(links);
     }
         
     try {