PartDesign: Transformed. Fix regression for Mirrored, and multiple fe…

…atures

src/Mod/PartDesign/App/FeatureTransformed.cpp

@@ -296,21 +296,28 @@ App::DocumentObjectExecReturn *Transformed::execute(void)
         bool overlapping = false;
 
         std::vector<gp_Trsf>::const_iterator t = transformations.begin();
-        ++t; // Skip first transformation, which is always the identity transformation
+        //++t; // Skip first transformation, which is always the identity transformation
+        bool first = true;
         for (; t != transformations.end(); ++t) {
             // Make an explicit copy of the shape because the "true" parameter to BRepBuilderAPI_Transform
             // seems to be pretty broken
             BRepBuilderAPI_Copy copy(origShape);
 
             shape = copy.Shape();
 
-            shape.Move(*t);
+            BRepBuilderAPI_Transform mkTrf(shape, *t, false); // No need to copy, now
+            if (!mkTrf.IsDone())
+                return new App::DocumentObjectExecReturn("Transformation failed", (*o));
+            shape = mkTrf.Shape();
+
             shapes.emplace_back(shape);
             builder.Add(compShape, shape);
 
-            if (overlapDetectionMode)
+            if (overlapDetectionMode && !first)
                 overlapping =  overlapping || (countSolids(TopoShape(origShape).fuse(shape))==1);
 
+            if (first)
+                first = false;
         }
 
         TopoDS_Shape toolShape;