[Part] Fix Part Boolean operations Refine incorrectly set for new users (#24256)
* [Part] Fix Part Boolean operations Refine incorrectly set for new users * [Part] update tests to specify Refine to be... ...false where necessary
This commit is contained in:
@@ -61,7 +61,7 @@ bool getRefineModelParameter()
|
||||
{
|
||||
Base::Reference<ParameterGrp> hGrp = App::GetApplication().GetUserParameter()
|
||||
.GetGroup("BaseApp")->GetGroup("Preferences")->GetGroup("Mod/Part/Boolean");
|
||||
return hGrp->GetBool("RefineModel", false);
|
||||
return hGrp->GetBool("RefineModel", true);
|
||||
}
|
||||
|
||||
}
|
||||
|
||||
@@ -31,6 +31,7 @@ protected:
|
||||
_fused = _doc->addObject<Part::Fuse>();
|
||||
_fused->Base.setValue(_boxes[0]);
|
||||
_fused->Tool.setValue(_boxes[1]);
|
||||
_fused->Refine.setValue(false);
|
||||
_fused->execute();
|
||||
_chamfer = _doc->addObject<Part::Chamfer>();
|
||||
}
|
||||
|
||||
@@ -31,6 +31,7 @@ protected:
|
||||
_fused = _doc->addObject<Part::Fuse>();
|
||||
_fused->Base.setValue(_boxes[0]);
|
||||
_fused->Tool.setValue(_boxes[1]);
|
||||
_fused->Refine.setValue(false);
|
||||
_fused->execute();
|
||||
_fillet = _doc->addObject<Part::Fillet>();
|
||||
}
|
||||
|
||||
@@ -64,6 +64,7 @@ TEST_F(FeatureMirroringTest, testYMirrorWithExistingElementMap)
|
||||
_fuse = _doc->addObject<Part::Fuse>();
|
||||
_fuse->Base.setValue(_boxes[0]);
|
||||
_fuse->Tool.setValue(_boxes[1]);
|
||||
_fuse->Refine.setValue(false);
|
||||
// Act
|
||||
_fuse->execute();
|
||||
_mirror->Source.setValue(_fuse);
|
||||
|
||||
@@ -240,7 +240,7 @@ TEST_F(FeaturePartFuseTest, testRefine)
|
||||
// Arrange
|
||||
_fuse->Base.setValue(_boxes[0]);
|
||||
_fuse->Tool.setValue(_boxes[1]);
|
||||
|
||||
_fuse->Refine.setValue(false);
|
||||
// Act
|
||||
_fuse->execute();
|
||||
Part::TopoShape ts = _fuse->Shape.getValue();
|
||||
|
||||
@@ -150,6 +150,7 @@ TEST_F(PartFeaturesTest, testRefine)
|
||||
auto _fuse = _doc->addObject<Part::Fuse>();
|
||||
_fuse->Base.setValue(_boxes[0]);
|
||||
_fuse->Tool.setValue(_boxes[3]);
|
||||
_fuse->Refine.setValue(false);
|
||||
_fuse->execute();
|
||||
Part::TopoShape fusedts = _fuse->Shape.getShape();
|
||||
auto _refine = _doc->addObject<Refine>();
|
||||
|
||||
@@ -32,6 +32,7 @@ TEST_F(FeaturePartMakeElementRefineTest, makeElementRefineBoxes)
|
||||
auto _fuse = _doc->addObject<Part::Fuse>();
|
||||
_fuse->Base.setValue(_boxes[0]);
|
||||
_fuse->Tool.setValue(_boxes[3]);
|
||||
_fuse->Refine.setValue(false);
|
||||
// Act
|
||||
_fuse->execute();
|
||||
Part::TopoShape ts = _fuse->Shape.getShape();
|
||||
|
||||
Reference in New Issue
Block a user