diff --git a/src/sp-conn-end.cpp b/src/sp-conn-end.cpp
index 0b420a98e09ec62a74ccadb8f9d1c972f4cc19f4..07dd852f230fe988b809d812b4ad7fc215e5fc7d 100644 (file)
--- a/src/sp-conn-end.cpp
+++ b/src/sp-conn-end.cpp
#include "2geom/path-intersection.h"
#include "2geom/path-intersection.h"
-static void change_endpts(SPCurve *const curve, Geom::Point const h2endPt[2]);
+static void change_endpts(SPCurve *const curve, double const endPos[2]);
SPConnEnd::SPConnEnd(SPObject *const owner) :
ref(owner),
href(NULL),
SPConnEnd::SPConnEnd(SPObject *const owner) :
ref(owner),
href(NULL),
+ // Default to center connection endpoint
+ type(ConnPointDefault),
+ id(4),
_changed_connection(),
_delete_connection(),
_transformed_connection()
_changed_connection(),
_delete_connection(),
_transformed_connection()
@@ -39,45 +42,37 @@ get_nearest_common_ancestor(SPObject const *const obj, SPItem const *const objs[
}
}
-static bool try_get_intersect_point_with_item_recursive(SPCurve *conn_curve, SPItem& item,
- const Geom::Matrix& item_transform, const bool at_start, double* intersect_pos,
- unsigned *intersect_index) {
-
- double initial_pos = (at_start) ? 0.0 : std::numeric_limits<double>::max();
+static bool try_get_intersect_point_with_item_recursive(Geom::PathVector& conn_pv, SPItem* item,
+ const Geom::Matrix& item_transform, double& intersect_pos) {
+ double initial_pos = intersect_pos;
// if this is a group...
// if this is a group...
- if (SP_IS_GROUP(&item)) {
- SPGroup* group = SP_GROUP(&item);
-
+ if (SP_IS_GROUP(item)) {
+ SPGroup* group = SP_GROUP(item);
+
// consider all first-order children
// consider all first-order children
- double child_pos = initial_pos;
- unsigned child_index;
+ double child_pos = 0.0;
for (GSList const* i = sp_item_group_item_list(group); i != NULL; i = i->next) {
SPItem* child_item = SP_ITEM(i->data);
for (GSList const* i = sp_item_group_item_list(group); i != NULL; i = i->next) {
SPItem* child_item = SP_ITEM(i->data);
- try_get_intersect_point_with_item_recursive(conn_curve, *child_item,
- item_transform * child_item->transform, at_start, &child_pos, &child_index);
- if (fabs(initial_pos - child_pos) > fabs(initial_pos - *intersect_pos)) {
- // It is further away from the initial point than the current intersection
- // point (i.e. the "outermost" intersection), so use this one.
- *intersect_pos = child_pos;
- *intersect_index = child_index;
- }
+ try_get_intersect_point_with_item_recursive(conn_pv, child_item,
+ item_transform * child_item->transform, child_pos);
+ if (intersect_pos < child_pos)
+ intersect_pos = child_pos;
}
}
- return *intersect_pos != initial_pos;
+ return intersect_pos != initial_pos;
}
}
- // if this is a shape...
- if (!SP_IS_SHAPE(&item)) return false;
+ // if this is not a shape, nothing to be done
+ if (!SP_IS_SHAPE(item)) return false;
// make sure it has an associated curve
// make sure it has an associated curve
- SPCurve* item_curve = sp_shape_get_curve(SP_SHAPE(&item));
+ SPCurve* item_curve = SP_SHAPE(item)->getCurve();
if (!item_curve) return false;
// apply transformations (up to common ancestor)
item_curve->transform(item_transform);
const Geom::PathVector& curve_pv = item_curve->get_pathvector();
if (!item_curve) return false;
// apply transformations (up to common ancestor)
item_curve->transform(item_transform);
const Geom::PathVector& curve_pv = item_curve->get_pathvector();
- const Geom::PathVector& conn_pv = conn_curve->get_pathvector();
Geom::CrossingSet cross = crossings(conn_pv, curve_pv);
// iterate over all Crossings
for (Geom::CrossingSet::const_iterator i = cross.begin(); i != cross.end(); i++) {
Geom::CrossingSet cross = crossings(conn_pv, curve_pv);
// iterate over all Crossings
for (Geom::CrossingSet::const_iterator i = cross.begin(); i != cross.end(); i++) {
@@ -85,18 +80,14 @@ static bool try_get_intersect_point_with_item_recursive(SPCurve *conn_curve, SPI
for (Geom::Crossings::const_iterator i = cr.begin(); i != cr.end(); i++) {
const Geom::Crossing& cr_pt = *i;
for (Geom::Crossings::const_iterator i = cr.begin(); i != cr.end(); i++) {
const Geom::Crossing& cr_pt = *i;
- if (fabs(initial_pos - cr_pt.ta) > fabs(initial_pos - *intersect_pos)) {
- // It is further away from the initial point than the current intersection
- // point (i.e. the "outermost" intersection), so use this one.
- *intersect_pos = cr_pt.ta;
- *intersect_index = cr_pt.a;
- }
+ if ( intersect_pos < cr_pt.ta)
+ intersect_pos = cr_pt.ta;
}
}
item_curve->unref();
}
}
item_curve->unref();
- return *intersect_pos != initial_pos;
+ return intersect_pos != initial_pos;
}
}
@@ -104,22 +95,36 @@ static bool try_get_intersect_point_with_item_recursive(SPCurve *conn_curve, SPI
// and the item given. If the item is a group, then the component items are considered.
// The transforms given should be to a common ancestor of both the path and item.
//
// and the item given. If the item is a group, then the component items are considered.
// The transforms given should be to a common ancestor of both the path and item.
//
-static bool try_get_intersect_point_with_item(SPPath& conn, SPItem& item,
- const Geom::Matrix& item_transform, const Geom::Matrix& conn_transform,
- const bool at_start, double* intersect_pos, unsigned *intersect_index) {
-
- // We start with the intersection point either at the beginning or end of the
- // path, depending on whether we are considering the source or target endpoint.
- *intersect_pos = (at_start) ? 0.0 : std::numeric_limits<double>::max();
+static bool try_get_intersect_point_with_item(SPPath* conn, SPItem* item,
+ const Geom::Matrix& item_transform, const Geom::Matrix& conn_transform,
+ const bool at_start, double& intersect_pos) {
// Copy the curve and apply transformations up to common ancestor.
// Copy the curve and apply transformations up to common ancestor.
- SPCurve* conn_curve = conn.curve->copy();
+ SPCurve* conn_curve = conn->curve->copy();
conn_curve->transform(conn_transform);
conn_curve->transform(conn_transform);
+ Geom::PathVector conn_pv = conn_curve->get_pathvector();
+
+ // If this is not the starting point, use Geom::Path::reverse() to reverse the path
+ if (!at_start)
+ {
+ // connectors are actually a single path, so consider the first element from a Geom::PathVector
+ conn_pv[0] = conn_pv[0].reverse();
+ }
+
+ // We start with the intersection point at the beginning of the path
+ intersect_pos = 0.0;
+
// Find the intersection.
// Find the intersection.
- bool result = try_get_intersect_point_with_item_recursive(conn_curve, item, item_transform,
- at_start, intersect_pos, intersect_index);
-
+ bool result = try_get_intersect_point_with_item_recursive(conn_pv, item, item_transform, intersect_pos);
+
+ if (!result)
+ // No intersection point has been found (why?)
+ // just default to connector end
+ intersect_pos = 0;
+ // If not at the starting point, recompute position with respect to original path
+ if (!at_start)
+ intersect_pos = conn_pv[0].size() - intersect_pos;
// Free the curve copy.
conn_curve->unref();
// Free the curve copy.
conn_curve->unref();
static void
static void
-sp_conn_end_move_compensate(Geom::Matrix const */*mp*/, SPItem */*moved_item*/,
- SPPath *const path,
- bool const updatePathRepr = true)
+sp_conn_get_route_and_redraw(SPPath *const path,
+ const bool updatePathRepr = true)
{
{
- // TODO: SPItem::getBounds gives the wrong result for some objects
- // that have internal representations that are updated later
- // by the sp_*_update functions, e.g., text.
- sp_document_ensure_up_to_date(path->document);
-
// Get the new route around obstacles.
// Get the new route around obstacles.
- path->connEndPair.reroutePath();
+ bool rerouted = path->connEndPair.reroutePathFromLibavoid();
+ if (!rerouted) {
+ return;
+ }
SPItem *h2attItem[2];
path->connEndPair.getAttachedItems(h2attItem);
SPItem *h2attItem[2];
path->connEndPair.getAttachedItems(h2attItem);
@@ -147,72 +149,80 @@ sp_conn_end_move_compensate(Geom::Matrix const */*mp*/, SPItem */*moved_item*/,
SPObject const *const ancestor = get_nearest_common_ancestor(path_item, h2attItem);
Geom::Matrix const path2anc(i2anc_affine(path_item, ancestor));
SPObject const *const ancestor = get_nearest_common_ancestor(path_item, h2attItem);
Geom::Matrix const path2anc(i2anc_affine(path_item, ancestor));
- Geom::Point endPts[2] = { *(path->curve->first_point()), *(path->curve->last_point()) };
-
+ // Set sensible values incase there the connector ends are not
+ // attached to any shapes.
+ Geom::PathVector conn_pv = path->curve->get_pathvector();
+ double endPos[2] = { 0, conn_pv[0].size() };
+
+ SPConnEnd** _connEnd = path->connEndPair.getConnEnds();
for (unsigned h = 0; h < 2; ++h) {
for (unsigned h = 0; h < 2; ++h) {
- if (h2attItem[h]) {
- // For each attached object, change the corresponding point to be
- // at the outermost intersection with the object's path.
- double intersect_pos;
- unsigned intersect_index;
+ if (h2attItem[h] && _connEnd[h]->type == ConnPointDefault && _connEnd[h]->id == ConnPointPosCC) {
Geom::Matrix h2i2anc = i2anc_affine(h2attItem[h], ancestor);
Geom::Matrix h2i2anc = i2anc_affine(h2attItem[h], ancestor);
- if ( try_get_intersect_point_with_item(*path, *h2attItem[h], h2i2anc, path2anc,
- (h == 0), &intersect_pos, &intersect_index) ) {
- const Geom::PathVector& curve = path->curve->get_pathvector();
- endPts[h] = curve[intersect_index].pointAt(intersect_pos);
- }
+ try_get_intersect_point_with_item(path, h2attItem[h], h2i2anc, path2anc,
+ (h == 0), endPos[h]);
}
}
}
}
- change_endpts(path->curve, endPts);
+ change_endpts(path->curve, endPos);
if (updatePathRepr) {
if (updatePathRepr) {
- path->requestDisplayUpdate(SP_OBJECT_MODIFIED_FLAG);
path->updateRepr();
path->updateRepr();
+ path->requestDisplayUpdate(SP_OBJECT_MODIFIED_FLAG);
}
}
}
}
-// TODO: This triggering of makeInvalidPath could be cleaned up to be
-// another option passed to move_compensate.
+
static void
static void
-sp_conn_end_shape_move_compensate(Geom::Matrix const *mp, SPItem *moved_item,
+sp_conn_end_shape_move(Geom::Matrix const */*mp*/, SPItem */*moved_item*/,
SPPath *const path)
{
if (path->connEndPair.isAutoRoutingConn()) {
SPPath *const path)
{
if (path->connEndPair.isAutoRoutingConn()) {
- path->connEndPair.makePathInvalid();
+ path->connEndPair.tellLibavoidNewEndpoints();
}
}
- sp_conn_end_move_compensate(mp, moved_item, path);
}
void
}
void
-sp_conn_adjust_invalid_path(SPPath *const path)
+sp_conn_reroute_path(SPPath *const path)
{
{
- sp_conn_end_move_compensate(NULL, NULL, path);
+ if (path->connEndPair.isAutoRoutingConn()) {
+ path->connEndPair.tellLibavoidNewEndpoints();
+ }
}
}
+
void
void
-sp_conn_adjust_path(SPPath *const path)
+sp_conn_reroute_path_immediate(SPPath *const path)
{
if (path->connEndPair.isAutoRoutingConn()) {
{
if (path->connEndPair.isAutoRoutingConn()) {
- path->connEndPair.makePathInvalid();
+ bool processTransaction = true;
+ path->connEndPair.tellLibavoidNewEndpoints(processTransaction);
}
// Don't update the path repr or else connector dragging is slowed by
// constant update of values to the xml editor, and each step is also
// needlessly remembered by undo/redo.
bool const updatePathRepr = false;
}
// Don't update the path repr or else connector dragging is slowed by
// constant update of values to the xml editor, and each step is also
// needlessly remembered by undo/redo.
bool const updatePathRepr = false;
- sp_conn_end_move_compensate(NULL, NULL, path, updatePathRepr);
+ sp_conn_get_route_and_redraw(path, updatePathRepr);
+}
+
+void sp_conn_redraw_path(SPPath *const path)
+{
+ sp_conn_get_route_and_redraw(path);
}
static void
}
static void
-change_endpts(SPCurve *const curve, Geom::Point const h2endPt[2])
+change_endpts(SPCurve *const curve, double const endPos[2])
{
{
-#if 0
- curve->reset();
- curve->moveto(h2endPt[0]);
- curve->lineto(h2endPt[1]);
-#else
- curve->move_endpoints(h2endPt[0], h2endPt[1]);
-#endif
+ // Use Geom::Path::portion to cut the curve at the end positions
+ if (endPos[0] > endPos[1])
+ {
+ // Path is "negative", reset the curve and return
+ curve->reset();
+ return;
+ }
+ const Geom::Path& old_path = curve->get_pathvector()[0];
+ Geom::PathVector new_path_vector;
+ new_path_vector.push_back(old_path.portion(endPos[0], endPos[1]));
+ curve->set_pathvector(new_path_vector);
}
static void
}
static void
@@ -221,9 +231,10 @@ sp_conn_end_deleted(SPObject *, SPObject *const owner, unsigned const handle_ix)
// todo: The first argument is the deleted object, or just NULL if
// called by sp_conn_end_detach.
g_return_if_fail(handle_ix < 2);
// todo: The first argument is the deleted object, or just NULL if
// called by sp_conn_end_detach.
g_return_if_fail(handle_ix < 2);
- char const *const attr_str[] = {"inkscape:connection-start",
- "inkscape:connection-end"};
- SP_OBJECT_REPR(owner)->setAttribute(attr_str[handle_ix], NULL);
+ char const * const attr_strs[] = {"inkscape:connection-start", "inkscape:connection-start-point",
+ "inkscape:connection-end", "inkscape:connection-end-point"};
+ SP_OBJECT_REPR(owner)->setAttribute(attr_strs[2*handle_ix], NULL);
+ SP_OBJECT_REPR(owner)->setAttribute(attr_strs[2*handle_ix+1], NULL);
/* I believe this will trigger sp_conn_end_href_changed. */
}
/* I believe this will trigger sp_conn_end_href_changed. */
}
}
void
}
void
-SPConnEnd::setAttacherHref(gchar const *value)
+SPConnEnd::setAttacherHref(gchar const *value, SPPath* /*path*/)
{
if ( value && href && ( strcmp(value, href) == 0 ) ) {
/* No change, do nothing. */
{
if ( value && href && ( strcmp(value, href) == 0 ) ) {
/* No change, do nothing. */
- } else {
- g_free(href);
- href = NULL;
- if (value) {
- // First, set the href field, because sp_conn_end_href_changed will need it.
+ }
+ else
+ {
+ if (!value)
+ {
+ ref.detach();
+ g_free(href);
+ href = NULL;
+ }
+ else
+ {
+ bool validRef = true;
href = g_strdup(value);
href = g_strdup(value);
-
// Now do the attaching, which emits the changed signal.
try {
ref.attach(Inkscape::URI(value));
} catch (Inkscape::BadURIException &e) {
/* TODO: Proper error handling as per
// Now do the attaching, which emits the changed signal.
try {
ref.attach(Inkscape::URI(value));
} catch (Inkscape::BadURIException &e) {
/* TODO: Proper error handling as per
- * http://www.w3.org/TR/SVG11/implnote.html#ErrorProcessing. (Also needed for
- * sp-use.) */
+ * http://www.w3.org/TR/SVG11/implnote.html#ErrorProcessing. (Also needed for
+ * sp-use.) */
g_warning("%s", e.what());
g_warning("%s", e.what());
+ validRef = false;
+ }
+
+ if ( !validRef )
+ {
ref.detach();
ref.detach();
+ g_free(href);
+ href = NULL;
+ }
+ }
+ }
+}
+
+void
+SPConnEnd::setAttacherEndpoint(gchar const *value, SPPath* /*path*/)
+{
+
+ /* References to the connection points have the following format
+ <t><id>, where t is the type of the point, which
+ can be either "d" for default or "u" for user-defined, and
+ id is the local (inside the item) id of the connection point.
+ In the case of default points id represents the position on the
+ item (i.e. Top-Left, Center-Center, etc.).
+ */
+
+ bool changed = false;
+ ConnPointType newtype = type;
+
+ if (!value)
+ {
+ // Default to center endpoint
+ type = ConnPointDefault;
+ id = 4;
+ }
+ else
+ {
+ switch (value[0])
+ {
+ case 'd':
+ if ( newtype != ConnPointDefault )
+ {
+ newtype = ConnPointDefault;
+ changed = true;
+ }
+ break;
+ case 'u':
+ if ( newtype != ConnPointUserDefined)
+ {
+ newtype = ConnPointUserDefined;
+ changed = true;
+ }
+ break;
+ default:
+ g_warning("Bad reference to a connection point.");
+ }
+
+ int newid = (int) g_ascii_strtod( value+1, 0 );
+ if ( id != newid )
+ {
+ id = newid;
+ changed = true;
+ }
+
+ // We have to verify that the reference to the
+ // connection point is a valid one.
+
+ if ( changed )
+ {
+
+ // Get the item the connector is attached to
+ SPItem* item = ref.getObject();
+ if ( item )
+ {
+ if (!item->avoidRef->isValidConnPointId( newtype, newid ) )
+ {
+ g_warning("Bad reference to a connection point.");
+ }
+ else
+ {
+ type = newtype;
+ id = newid;
+ }
+ /* // Update the connector
+ if (path->connEndPair.isAutoRoutingConn()) {
+ path->connEndPair.tellLibavoidNewEndpoints();
+ }
+ */
}
}
- } else {
- ref.detach();
}
}
}
}
}
}
= SP_OBJECT(refobj)->connectDelete(sigc::bind(sigc::ptr_fun(&sp_conn_end_deleted),
SP_OBJECT(path), handle_ix));
connEnd._transformed_connection
= SP_OBJECT(refobj)->connectDelete(sigc::bind(sigc::ptr_fun(&sp_conn_end_deleted),
SP_OBJECT(path), handle_ix));
connEnd._transformed_connection
- = SP_ITEM(refobj)->connectTransformed(sigc::bind(sigc::ptr_fun(&sp_conn_end_shape_move_compensate),
+ = SP_ITEM(refobj)->connectTransformed(sigc::bind(sigc::ptr_fun(&sp_conn_end_shape_move),
path));
}
}
path));
}
}