Fixes bug #478597 where the connector context crash due to asserting that

paths marked as connectors were always open.  Now it just treats them as
connectors if they are open, or normal objects otherwise.

diff --git a/src/connector-context.cpp b/src/connector-context.cpp
index 1263a9215086a068821fcbcf8179cca754d78f2b..f862abac4231bf8c6d451c423538048b7bf84858 100644 (file)
--- a/src/connector-context.cpp
+++ b/src/connector-context.cpp
@@ -1908,8 +1908,10 @@ static bool cc_item_is_shape(SPItem *item)
 bool cc_item_is_connector(SPItem *item)
 {
     if (SP_IS_PATH(item)) {
-        if (SP_PATH(item)->connEndPair.isAutoRoutingConn()) {
-            g_assert( SP_PATH(item)->original_curve ? !(SP_PATH(item)->original_curve->is_closed()) : !(SP_PATH(item)->curve->is_closed()) );
+        bool closed = SP_PATH(item)->original_curve ? SP_PATH(item)->original_curve->is_closed() : SP_PATH(item)->curve->is_closed();
+        if (SP_PATH(item)->connEndPair.isAutoRoutingConn() && !closed) {
+            // To be considered a connector, an object must be a non-closed 
+            // path that is marked with a "inkscape:connector-type" attribute.
             return true;
         }
     }