Merge pull request #10033 from jffmichi/fix_flipedge

[Path] fix flipEdge not taking into account FirstParameter and LastParameter…
FreeCAD · Aug 21, 2023 · 9382255 · 9382255
2 parents 13284dd + 32e1082
commit 9382255
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/src/Mod/Path/Path/Geom.py b/src/Mod/Path/Path/Geom.py
@@ -676,7 +676,7 @@ def flipEdge(edge):
             poles, mults, knots, perio, degree, weights, ratio
         )
 
-        return Part.Edge(flipped)
+        return Part.Edge(flipped, ma + mi - edge.LastParameter, ma + mi - edge.FirstParameter)
     elif type(edge.Curve) == Part.OffsetCurve:
         return edge.reversed()
 

diff --git a/src/Mod/Path/PathTests/TestPathGeom.py b/src/Mod/Path/PathTests/TestPathGeom.py
@@ -872,6 +872,9 @@ def test75(self):
         edge = Part.Edge(spline)
         self.assertEdgeShapesMatch(edge, Path.Geom.flipEdge(edge))
 
+        edge = Part.Edge(spline, 4, 12)
+        self.assertEdgeShapesMatch(edge, Path.Geom.flipEdge(edge))
+
         edge = Part.Edge(
             Part.BSplineCurve(
                 [