MGG-1402 [#ccc] svgpathtools:bugfix: Skip the end path in a polyline if no dedicated end path is provided

References: MGG-1402, MGG-1448

Introduce an 'is_polygon' flag in 'polygon2pathd' function to handle the end path in polyline and polygon

if path string or is_polygon flag is enableed then append path given in input
if path string and is_polygon flag is enable then append 'z' path

Signed-off-by: Harikrushna Parmar <harikrushna.parmar@nxp.com>

Author: HKPTech

svgpathtools/svg_to_paths.py

@@ -93,7 +93,7 @@ def polyline2pathd(polyline, is_polygon=False):
     # The `parse_path` call ignores redundant 'z' (closure) commands
     # e.g. `parse_path('M0 0L100 100Z') == parse_path('M0 0L100 100L0 0Z')`
     # This check ensures that an n-point polygon is converted to an n-Line path.
-    if is_polygon and closed:
+    if is_polygon or closed:
         points.append(points[0])
 
     d = 'M' + 'L'.join('{0} {1}'.format(x,y) for x,y in points)
@@ -102,13 +102,13 @@ def polyline2pathd(polyline, is_polygon=False):
     return d
 
 
-def polygon2pathd(polyline):
+def polygon2pathd(polyline, is_polygon=True):
     """converts the string from a polygon points-attribute to a string 
     for a Path object d-attribute.
     Note:  For a polygon made from n points, the resulting path will be
     composed of n lines (even if some of these lines have length zero).
     """
-    return polyline2pathd(polyline, True)
+    return polyline2pathd(polyline, is_polygon)
 
 
 def rect2pathd(rect):
@@ -230,7 +230,7 @@ def dom2dict(element):
     # path strings, add to list
     if convert_polygons_to_paths:
         pgons = [dom2dict(el) for el in doc.getElementsByTagName('polygon')]
-        d_strings += [polygon2pathd(pg) for pg in pgons]
+        d_strings += [polygon2pathd(pg, True) for pg in pgons]
         attribute_dictionary_list += pgons
 
     if convert_lines_to_paths: