Mayor refactor

(bzr r11950.1.36)

1 files changed, 49 insertions, 41 deletions

diff --git a/src/ui/tool/path-manipulator.cpp b/src/ui/tool/path-manipulator.cpp
index b650987bc..c4158931a 100644
--- a/src/ui/tool/path-manipulator.cpp
+++ b/src/ui/tool/path-manipulator.cpp
@@ -1182,31 +1182,8 @@ bool PathManipulator::isBSpline(){
     }
     return false;
 }
-double PathManipulator::BSplineMaxPosition(Node *n){
-    double nearestPointAt = 0;
-    Geom::D2< Geom::SBasis > SBasisInsideNodes;
-    SPCurve *lineInsideNodes = new SPCurve();
-    Node * frontNode = n->nodeToward(n->front());
-    if(frontNode){
-        lineInsideNodes->moveto(n->position());
-        lineInsideNodes->lineto(frontNode->position());
-        SBasisInsideNodes = lineInsideNodes->first_segment()->toSBasis();
-        nearestPointAt = Geom::nearest_point(n->front()->position(),*lineInsideNodes->first_segment());
-    }
-    Node * backNode = n->nodeToward(n->back());
-    if(backNode){
-        lineInsideNodes->reset();
-        lineInsideNodes->moveto(n->position());
-        lineInsideNodes->lineto(backNode->position());
-        SBasisInsideNodes = lineInsideNodes->first_segment()->toSBasis();
-        using std::max;
-        nearestPointAt = max(nearestPointAt,1-Geom::nearest_point(n->back()->position(),*lineInsideNodes->first_segment()));
-    }
-    return nearestPointAt;
-}
-
-Geom::Point PathManipulator::BSplineHandleReposition(Handle *h){
-    double nearestPointAt = 0;
+double PathManipulator::BSplineHandlePosition(Handle *h){
+    double pos = 0;
     Node *n = h->parent();
     Geom::D2< Geom::SBasis > SBasisInsideNodes;
     SPCurve *lineInsideNodes = new SPCurve();
@@ -1215,19 +1192,23 @@ Geom::Point PathManipulator::BSplineHandleReposition(Handle *h){
         lineInsideNodes->moveto(n->position());
         lineInsideNodes->lineto(nextNode->position());
         SBasisInsideNodes = lineInsideNodes->first_segment()->toSBasis();
-        nearestPointAt = Geom::nearest_point(n->front()->position(),*lineInsideNodes->first_segment());
+        pos = Geom::nearest_point(h->position(),*lineInsideNodes->first_segment());
     }
-    return SBasisInsideNodes.valueAt(nearestPointAt);
+    return pos;
 }
 
-Geom::Point PathManipulator::BSplineHandleRepositionFixed(Handle *h,double pos){
+Geom::Point PathManipulator::BSplineHandleReposition(Handle *h){
+    double pos = 0;
+    pos = this->BSplineHandlePosition(h);
+    return BSplineHandleReposition(h,pos);
+}
+
+Geom::Point PathManipulator::BSplineHandleReposition(Handle *h,double pos){
     Node *n = h->parent();
-    if(n->back()->position() == h->position())
-        pos = 1-pos;
     Geom::D2< Geom::SBasis > SBasisInsideNodes;
     SPCurve *lineInsideNodes = new SPCurve();
     Node * nextNode = n->nodeToward(h);
-    if(nextNode){
+    if(nextNode && pos != 0){
         lineInsideNodes->moveto(n->position());
         lineInsideNodes->lineto(nextNode->position());
         SBasisInsideNodes = lineInsideNodes->first_segment()->toSBasis();
@@ -1251,26 +1232,53 @@ void PathManipulator::_createGeometryFromControlPoints(bool alert_LPE)
             continue;
         }
         NodeList::iterator prev = subpath->begin();
+        //BSpline
+        float pos = NULL;
+        bool isBSpline = false;
+        if(this->isBSpline())
+            isBSpline = true;
+        if(isBSpline){
+            pos = BSplineHandlePosition(prev.ptr()->front());
+            prev.ptr()->front()->setPosition(BSplineHandleReposition(prev.ptr()->front(),pos));
+            if(pos == 0){
+                prev.ptr()->setPosition(BSplineHandleReposition(i.ptr()->front(),pos));
+            }
+        }
+        //BSpline End
         builder.moveTo(prev->position());
         for (NodeList::iterator i = ++subpath->begin(); i != subpath->end(); ++i) {
-            if (this->isBSpline()) {
-                float pos = BSplineMaxPosition(i.ptr());
-                i.ptr()->front()->setPosition(BSplineHandleRepositionFixed(i.ptr()->front(),pos));
-                i.ptr()->back()->setPosition(BSplineHandleRepositionFixed(i.ptr()->back(),pos));
+            //BSpline
+            if (isBSpline) {
+                pos = BSplineHandlePosition(i.ptr()->front());
+                if(pos == 0)
+                    pos = BSplineHandlePosition(i.ptr()->back());
+                i.ptr()->front()->setPosition(BSplineHandleReposition(i.ptr()->front(),pos));
+                i.ptr()->back()->setPosition(BSplineHandleReposition(i.ptr()->back(),pos));
+                if(pos == 0){
+                    i.ptr()->setPosition(BSplineHandleReposition(i.ptr()->front(),pos));
+                }
             }
-            //BSplie End
+            
+            //BSpline End
             build_segment(builder, prev.ptr(), i.ptr());
             prev = i;
         }
         if (subpath->closed()) {
             // Here we link the last and first node if the path is closed.
             // If the last segment is Bezier, we add it.
-            if (this->isBSpline()) {
-                float pos = BSplineMaxPosition(prev.ptr());
-                subpath->begin().ptr()->front()->setPosition(BSplineHandleRepositionFixed(subpath->begin().ptr()->front(),pos));
-                prev.ptr()->back()->setPosition(BSplineHandleRepositionFixed(prev.ptr()->back(),pos));
-            
+            //BSpline
+            if (isBSpline) {
+                pos = BSplineHandlePosition(prev.ptr()->front());
+                if(pos == 0)
+                    pos = BSplineHandlePosition(subpath->begin().ptr()->back());
+                subpath->begin().ptr()->front()->setPosition(BSplineHandleReposition(subpath->begin().ptr()->front(),pos));
+                prev.ptr()->back()->setPosition(BSplineHandleReposition(prev.ptr()->back(),pos));
+                if(pos == 0){
+                    subpath->begin().ptr()->setPosition(BSplineHandleReposition(subpath->begin().ptr()->front(),pos));
+                    prev.ptr()->setPosition(BSplineHandleReposition(prev.ptr()->back(),pos));
+                }
             }
+            //BSpline End
             if (!prev->front()->isDegenerate() || !subpath->begin()->back()->isDegenerate()) {
                 build_segment(builder, prev.ptr(), subpath->begin().ptr());
             }