kindred/create
1
1
Fork 0
Code Issues 37 Pull Requests Actions Packages Projects 9 Releases 2 Wiki Activity

allow to attach a sketch to any kind of planar surfaces, not only planes

Browse Source
This commit is contained in:
wmayer
2017-09-23 22:46:54 +02:00
parent 4f0f3e667a
commit 0d03ad13da
2 changed files with 25 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
src/Mod/Part/App/Attacher.cpp
View File

@@ -54,6 +54,7 @@
# include <GProp_PGProps.hxx>
# include <GProp_PrincipalProps.hxx>
# include <BRepGProp.hxx>
# include <GeomLib_IsPlanarSurface.hxx>
#endif
#include <BRepLProp_SLProps.hxx>
#include <GeomAPI_ProjectPointOnCurve.hxx>
@@ -1119,15 +1120,25 @@ Base::Placement AttachEngine3D::calculateAttachedPlacement(Base::Placement origP
if (face.IsNull())
throw Base::ValueError("Null face in AttachEngine3D::calculateAttachedPlacement()!");
gp_Pln plane;
BRepAdaptor_Surface adapt(face);
if (adapt.GetType() != GeomAbs_Plane)
throw Base::ValueError("No planar face in AttachEngine3D::calculateAttachedPlacement()!");
if (adapt.GetType() == GeomAbs_Plane) {
plane = adapt.Plane();
}
else {
TopLoc_Location loc;
Handle(Geom_Surface) surf = BRep_Tool::Surface(face, loc);
GeomLib_IsPlanarSurface check(surf);
if (check.IsPlanar())
plane = check.Plan();
else
throw Base::ValueError("No planar face in AttachEngine3D::calculateAttachedPlacement()!");
}
bool Reverse = false;
if (face.Orientation() == TopAbs_REVERSED)
Reverse = true;
gp_Pln plane = adapt.Plane();
Standard_Boolean ok = plane.Direct();
if (!ok) {
// toggle if plane has a left-handed coordinate system

15
src/Mod/PartDesign/Gui/Command.cpp
View File

@@ -27,7 +27,10 @@
# include <TopoDS_Face.hxx>
# include <TopoDS.hxx>
# include <BRepAdaptor_Surface.hxx>
# include <BRep_Tool.hxx>
# include <TopExp_Explorer.hxx>
# include <TopLoc_Location.hxx>
# include <GeomLib_IsPlanarSurface.hxx>
# include <QMessageBox>
# include <Inventor/nodes/SoCamera.h>
#endif
@@ -398,10 +401,14 @@ void CmdPartDesignNewSketch::activated(int iMsg)
}
BRepAdaptor_Surface adapt(face);
if (adapt.GetType() != GeomAbs_Plane){
QMessageBox::warning(Gui::getMainWindow(), QObject::tr("No planar support"),
QObject::tr("You need a planar face as support for a sketch!"));
return;
if (adapt.GetType() != GeomAbs_Plane) {
TopLoc_Location loc;
Handle(Geom_Surface) surf = BRep_Tool::Surface(face, loc);
if (surf.IsNull() || !GeomLib_IsPlanarSurface(surf).IsPlanar()) {
QMessageBox::warning(Gui::getMainWindow(), QObject::tr("No planar support"),
QObject::tr("You need a planar face as support for a sketch!"));
return;
}
}
supportString = FaceFilter.Result[0][0].getAsPropertyLinkSubString();