Skip to content

Commit

Permalink
[Part TopoShapeExpansion] move shape transform from make element copy…
Browse files Browse the repository at this point in the history
… into ruled surface function since it does not work well when copying vertex objects
  • Loading branch information
mwganson authored and chennes committed Feb 3, 2025
1 parent 488d931 commit d5b0f68
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/Mod/Part/App/TopoShapeExpansion.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -2203,8 +2203,8 @@ TopoShape& TopoShape::makeElementRuledSurface(const std::vector<TopoShape>& shap
// if both shapes are sub-elements of one common shape then the fill
// algorithm leads to problems if the shape has set a placement. The
// workaround is to copy the sub-shape
S1 = S1.makeElementCopy();
S2 = S2.makeElementCopy();
S1.setTransform(S1.getTransform());
S2.setTransform(S2.getTransform());

if (orientation == 0) {
// Automatic
Expand Down Expand Up @@ -3427,7 +3427,7 @@ TopoShape::makeElementCopy(const TopoShape& shape, const char* op, bool copyGeom

TopoShape tmp(shape);
tmp.setShape(BRepBuilderAPI_Copy(shape.getShape(), copyGeom, copyMesh).Shape(), false);
tmp.setTransform(shape.getTransform());

if (op || (shape.Tag && shape.Tag != Tag)) {
setShape(tmp._Shape);
initCache();
Expand Down

0 comments on commit d5b0f68

Please sign in to comment.