Full refactor of the LPE now too much simple code

(bzr r14272.1.6)

1 files changed, 65 insertions, 55 deletions

diff --git a/src/live_effects/lpe-bendpath.cpp b/src/live_effects/lpe-bendpath.cpp
index 9b015245d..d80f8e33a 100644
--- a/src/live_effects/lpe-bendpath.cpp
+++ b/src/live_effects/lpe-bendpath.cpp
@@ -20,7 +20,13 @@
 #include <2geom/d2.h>
 #include <2geom/piecewise.h>
 
+#include "knot-holder-entity.h"
+#include "knotholder.h"
+
+#include <glibmm/i18n.h>
+
 #include <algorithm>
+
 using std::vector;
 
 
@@ -48,29 +54,33 @@ first) but I think we can first forget about them.
 namespace Inkscape {
 namespace LivePathEffect {
 
+Geom::PathVector bp_helper_path;
+namespace BeP {
+class KnotHolderEntityWidthBendPath : public LPEKnotHolderEntity {
+    public:
+        KnotHolderEntityWidthBendPath(LPEBendPath * effect) : LPEKnotHolderEntity(effect) {}
+        virtual void knot_set(Geom::Point const &p, Geom::Point const &origin, guint state);
+        virtual Geom::Point knot_get() const;
+    };
+} // BeP
 
 LPEBendPath::LPEBendPath(LivePathEffectObject *lpeobject) :
     Effect(lpeobject),
     bend_path(_("Bend path:"), _("Path along which to bend the original path"), "bendpath", &wr, this, "M0,0 L1,0"),
+    original_height(0.0),
     prop_scale(_("_Width:"), _("Width of the path"), "prop_scale", &wr, this, 1.0),
-    width(_("Width distance"), _("Change the width of bend path - <b>Ctrl+Alt+Click</b>: reset"), "width", &wr, this),
     scale_y_rel(_("W_idth in units of length"), _("Scale the width of the path in units of its length"), "scale_y_rel", &wr, this, false),
-    vertical_pattern(_("_Original path is vertical"), _("Rotates the original 90 degrees, before bending it along the bend path"), "vertical", &wr, this, false),
-    height(0.0),
-    original_height(0.0),
-    prop_scale_from_widget(1.0),
-    firstTime(false),
-    secondTime(false)
+    vertical_pattern(_("_Original path is vertical"), _("Rotates the original 90 degrees, before bending it along the bend path"), "vertical", &wr, this, false)
 {
     registerParameter( dynamic_cast<Parameter *>(&bend_path) );
     registerParameter( dynamic_cast<Parameter *>(&prop_scale) );
     registerParameter( dynamic_cast<Parameter *>(&scale_y_rel) );
     registerParameter( dynamic_cast<Parameter *>(&vertical_pattern) );
-    registerParameter( dynamic_cast<Parameter *>(&width) );
 
     prop_scale.param_set_digits(3);
     prop_scale.param_set_increments(0.01, 0.10);
 
+    _provides_knotholder_entities = true;
     concatenate_before_pwd2 = true;
 }
 
@@ -82,56 +92,11 @@ LPEBendPath::~LPEBendPath()
 void
 LPEBendPath::doBeforeEffect (SPLPEItem const* lpeitem)
 {
-    hp.clear();
     // get the item bounding box
     original_bbox(lpeitem);
     original_height = boundingbox_Y.max() - boundingbox_Y.min();
-    Geom::Path path_in = bend_path.get_pathvector().pathAt(Geom::PathVectorTime(0, 0, 0.0));
-    Geom::Point ptA = path_in.pointAt(Geom::PathTime(0, 0.0));
-    Geom::Point B = path_in.pointAt(Geom::PathTime(1, 0.0));
-    Geom::Curve const *first_curve = &path_in.curveAt(Geom::PathTime(0, 0.0));
-    Geom::CubicBezier const *cubic = dynamic_cast<Geom::CubicBezier const *>(&*first_curve);
-    Geom::Ray ray(ptA, B);
-    if (cubic) {
-        ray.setPoints(ptA,(*cubic)[1]);
-    }
-    
-    Geom::Angle first_curve_angle = ray.transformed(Geom::Rotate(Geom::deg_to_rad(90))).angle();
-    //This Hack is to fix a boring bug in the first call to the function, we have
-    //a wrong "ptA"
-    if(!firstTime && Geom::are_near(width, Geom::Point())){
-        firstTime = true;
-    } else if(firstTime && Geom::are_near(width, Geom::Point())){
-        Geom::Point default_point = Geom::Point::polar(first_curve_angle, original_height/2.0) + ptA;
-        prop_scale.param_set_value(1.0);
-        height = original_height;
-        width.param_setValue(default_point);
-        width.param_update_default(default_point);
-    } else {
-        double distance_knot =  Geom::distance(width , ptA);
-        width.param_setValue(Geom::Point::polar(first_curve_angle, distance_knot) + ptA);
-        height = distance_knot * 2;
-        if(secondTime){
-            prop_scale.param_set_value(1);
-            height = original_height;
-            width.param_setValue(Geom::Point::polar(first_curve_angle, height/2.0) + ptA);
-            secondTime = false;
-        } else if(prop_scale_from_widget == prop_scale){
-            prop_scale.param_set_value(height/original_height);
-        } else {
-            height = original_height * prop_scale;
-            width.param_setValue(Geom::Point::polar(first_curve_angle, height/2.0) + ptA);
-        }
-        if(firstTime){
-            firstTime = false;
-            secondTime = true;
-        }
-    }
-    prop_scale_from_widget = prop_scale;
-    Geom::Path hp_path(width);
-    hp_path.appendNew<Geom::LineSegment>(ptA);
-    hp.push_back(hp_path);
     SPLPEItem * item = const_cast<SPLPEItem*>(lpeitem);
+    
     item->apply_to_clippath(item);
     item->apply_to_mask(item);
 }
@@ -206,9 +171,54 @@ LPEBendPath::resetDefaults(SPItem const* item)
 void
 LPEBendPath::addCanvasIndicators(SPLPEItem const */*lpeitem*/, std::vector<Geom::PathVector> &hp_vec)
 {
-    hp_vec.push_back(hp);
+    hp_vec.push_back(bp_helper_path);
 }
 
+void 
+LPEBendPath::addKnotHolderEntities(KnotHolder *knotholder, SPDesktop *desktop, SPItem *item)
+{
+    KnotHolderEntity *e = new BeP::KnotHolderEntityWidthBendPath(this);
+    e->create(desktop, item, knotholder, Inkscape::CTRL_TYPE_UNKNOWN, _("Change the width"), SP_KNOT_SHAPE_CIRCLE);
+    knotholder->add(e);
+}
+
+namespace BeP {
+
+void 
+KnotHolderEntityWidthBendPath::knot_set(Geom::Point const &p, Geom::Point const& /*origin*/, guint state)
+{
+    LPEBendPath *lpe = dynamic_cast<LPEBendPath *> (_effect);
+
+    Geom::Point const s = snap_knot_position(p, state);
+    Geom::Path path_in = lpe->bend_path.get_pathvector().pathAt(Geom::PathVectorTime(0, 0, 0.0));
+    Geom::Point ptA = path_in.pointAt(Geom::PathTime(0, 0.0));
+    lpe->prop_scale.param_set_value(Geom::distance(s , ptA)/(lpe->original_height/2.0));
+    sp_lpe_item_update_patheffect (SP_LPE_ITEM(item), false, true);
+}
+
+Geom::Point 
+KnotHolderEntityWidthBendPath::knot_get() const
+{
+    LPEBendPath *lpe = dynamic_cast<LPEBendPath *> (_effect);
+
+    bp_helper_path.clear();
+    Geom::Path path_in = lpe->bend_path.get_pathvector().pathAt(Geom::PathVectorTime(0, 0, 0.0));
+    Geom::Point ptA = path_in.pointAt(Geom::PathTime(0, 0.0));
+    Geom::Point B = path_in.pointAt(Geom::PathTime(1, 0.0));
+    Geom::Curve const *first_curve = &path_in.curveAt(Geom::PathTime(0, 0.0));
+    Geom::CubicBezier const *cubic = dynamic_cast<Geom::CubicBezier const *>(&*first_curve);
+    Geom::Ray ray(ptA, B);
+    if (cubic) {
+        ray.setPoints(ptA,(*cubic)[1]);
+    }
+    Geom::Angle first_curve_angle = ray.transformed(Geom::Rotate(Geom::deg_to_rad(90))).angle();
+    Geom::Point result_point = Geom::Point::polar(first_curve_angle, (lpe->original_height * lpe->prop_scale)/2.0) + ptA;
+    Geom::Path hp_path(result_point);
+    hp_path.appendNew<Geom::LineSegment>(ptA);
+    bp_helper_path.push_back(hp_path);
+    return result_point;
+}
+} // namespace BeP
 } // namespace LivePathEffect
 } /* namespace Inkscape */