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

Ellipse variable name change to create consistency

Browse Source 
Change variable names of the Ellipse from Angle0 to Angle1 to create consistency with all other named angle variables
This commit is contained in:
Sebastian
2022-02-11 06:50:28 +01:00
committed by wmayer
parent 1e33533539
commit 2158eebc88
4 changed files with 25 additions and 25 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
src/Mod/Part/App/PrimitiveFeature.cpp
View File

@@ -1009,10 +1009,10 @@ Ellipse::Ellipse()
{
ADD_PROPERTY(MajorRadius,(4.0f));
ADD_PROPERTY(MinorRadius,(4.0f));
ADD_PROPERTY(Angle0,(0.0f));
Angle0.setConstraints(&angleRange);
ADD_PROPERTY(Angle1,(360.0f));
ADD_PROPERTY(Angle1,(0.0f));
Angle1.setConstraints(&angleRange);
ADD_PROPERTY(Angle2,(360.0f));
Angle2.setConstraints(&angleRange);
}
Ellipse::~Ellipse()
@@ -1021,8 +1021,8 @@ Ellipse::~Ellipse()
short Ellipse::mustExecute() const
{
if (Angle0.isTouched() ||
Angle1.isTouched() ||
if (Angle1.isTouched() ||
Angle2.isTouched() ||
MajorRadius.isTouched() ||
MinorRadius.isTouched())
return 1;
@@ -1040,8 +1040,8 @@ App::DocumentObjectExecReturn *Ellipse::execute(void)
ellipse.SetMajorRadius(this->MajorRadius.getValue());
ellipse.SetMinorRadius(this->MinorRadius.getValue());
BRepBuilderAPI_MakeEdge clMakeEdge(ellipse, Base::toRadians<double>(this->Angle0.getValue()),
Base::toRadians<double>(this->Angle1.getValue()));
BRepBuilderAPI_MakeEdge clMakeEdge(ellipse, Base::toRadians<double>(this->Angle1.getValue()),
Base::toRadians<double>(this->Angle2.getValue()));
const TopoDS_Edge& edge = clMakeEdge.Edge();
this->Shape.setValue(edge);
@@ -1051,7 +1051,7 @@ App::DocumentObjectExecReturn *Ellipse::execute(void)
void Ellipse::onChanged(const App::Property* prop)
{
if (!isRestoring()) {
if (prop == &MajorRadius || prop == &MinorRadius || prop == &Angle0 || prop == &Angle1){
if (prop == &MajorRadius || prop == &MinorRadius || prop == &Angle1 || prop == &Angle2){
try {
App::DocumentObjectExecReturn *ret = recompute();
delete ret;