From f55b36b8a63b41c313bb26d18fe1c6763e18e377 Mon Sep 17 00:00:00 2001
From: Mario Voigt <mario.voigt@stadtfabrikanten.org>
Date: Mon, 18 Oct 2021 16:52:55 +0200
Subject: [PATCH] fix back in create links

---
 .../fablabchemnitz/create_links/create_links.inx  |  2 +-
 .../fablabchemnitz/create_links/create_links.py   | 15 ++++++---------
 2 files changed, 7 insertions(+), 10 deletions(-)

diff --git a/extensions/fablabchemnitz/create_links/create_links.inx b/extensions/fablabchemnitz/create_links/create_links.inx
index f560adb1..85f08dc3 100644
--- a/extensions/fablabchemnitz/create_links/create_links.inx
+++ b/extensions/fablabchemnitz/create_links/create_links.inx
@@ -50,7 +50,7 @@
 					</param>
 					<param name="keep_selected" type="bool" gui-text="Keep selected elements">false</param>
 					<param name="no_convert" type="bool" gui-text="Do not create output path(s) (cosmetic style only)">false</param>
-					<param name="breakapart" type="bool" gui-text="Break apart output path(s) into segments" gui-description="Performs CTRL + SHIFT + K to break the new output path into it's parts">false</param>
+					<param name="breakapart" type="bool" gui-text="Break apart output path(s) into segments" gui-description="Performs CTRL + SHIFT + K to break the new output path into it's parts. Recommended to enable because default break apart of Inkscape might produce pointy paths.">true</param>
 					<param name="show_info" type="bool" gui-text="Print length, pattern and filtering information/errors" gui-description="Warning: might freeze Inkscape forever if you have a lot of nodes because we create too much print output. Use for debugging only!">false</param>
 					<param name="skip_errors" type="bool" gui-text="Skip errors">false</param>
 				</vbox>
diff --git a/extensions/fablabchemnitz/create_links/create_links.py b/extensions/fablabchemnitz/create_links/create_links.py
index 43f52d69..d432e28f 100644
--- a/extensions/fablabchemnitz/create_links/create_links.py
+++ b/extensions/fablabchemnitz/create_links/create_links.py
@@ -59,7 +59,7 @@ class LinksCreator(inkex.EffectExtension):
         pars.add_argument("--length_filter_unit", default="mm", help="Length filter unit")
         pars.add_argument("--keep_selected", type=inkex.Boolean, default=False, help="Keep selected elements")
         pars.add_argument("--no_convert", type=inkex.Boolean, default=False, help="Do not create segments (cosmetic gaps only)")
-        pars.add_argument("--breakapart", type=inkex.Boolean, default=False, help="Performs CTRL + SHIFT + K to break the new output path into it's parts")
+        pars.add_argument("--breakapart", type=inkex.Boolean, default=True, help="Performs CTRL + SHIFT + K to break the new output path into it's parts. Recommended to enable because default break apart of Inkscape might produce pointy paths.")
         pars.add_argument("--show_info", type=inkex.Boolean, default=False, help="Print some length and pattern information")
         pars.add_argument("--skip_errors", type=inkex.Boolean, default=False, help="Skip errors")
 
@@ -283,15 +283,12 @@ class LinksCreator(inkex.EffectExtension):
                             length = length - dash
                             idash = (idash + 1) % len(dashes)
                             dash = dashes[idash]
-                        if sub[-1] != sub[i] and sub[i][0] != sub[i][1]: #avoid pointy paths
-                            if idash % 2:
-                                new.append([sub[i]])
-                            else:
-                                new[-1].append(sub[i])
+                        if idash % 2:
+                            new.append([sub[i]])
+                        else:
+                            new[-1].append(sub[i])
                         i += 1
-                if new[-1][0] == new[-1][0]: #avoid pointy paths
-                    new.remove(new[-1])
-   
+
                 style.pop('stroke-dasharray')
                 element.pop('sodipodi:type')
                 csp = CubicSuperPath(new)